Fix bug with trailing slashes in xdmod_host.

tests/integration/test_datawarehouse_integration.py

@@ -35,7 +35,7 @@
 VALID_DATE = '2020-01-01'
 VALID_DIMENSION = 'Resource'
 VALID_VALUES = {
-  'duration': 'Previous month',
+  'duration': 'Yesterday',
   'realm': 'Jobs',
   'metric': 'CPU Hours: Total',
   'dimension': VALID_DIMENSION,
@@ -44,7 +44,7 @@
   'aggregation_unit': 'Auto',
   'parameter': 'duration',
   'fields': ['Nodes'],
-  'show_progress': True,
+  'show_progress': False,
 }
 KEY_ERROR_TEST_VALUES_AND_MATCHES = {
     'duration': (INVALID_STR, 'Invalid value for `duration`'),
@@ -92,8 +92,16 @@
                 method + ':end_date',
             ]
             date_malformed_test_params += [
-                (method, 'start_date', {'duration': (INVALID_STR, VALID_DATE)}),
-                (method, 'end_date', {'duration': (VALID_DATE, INVALID_STR)}),
+                (
+                    method,
+                    'start_date',
+                    {'duration':(INVALID_STR, VALID_DATE)}
+                ),
+                (   
+                    method,
+                    'end_date',
+                    {'duration': (VALID_DATE, INVALID_STR)}
+                ),
             ]
             value_error_test_methods += [method]
     if 'filters' in METHOD_PARAMS[method]:
@@ -107,8 +115,11 @@
 
 
 @pytest.fixture(scope='module')
-def dw_methods():
-    with DataWarehouse(VALID_XDMOD_URL) as dw:
+def dw_methods(request):
+    xdmod_host = VALID_XDMOD_URL
+    if hasattr(request, "param"):
+        xdmod_host = request.param
+    with DataWarehouse(xdmod_host) as dw:
         yield __get_dw_methods(dw)
 
 
@@ -152,16 +163,7 @@ def test_KeyError(dw_methods, method, params, match):
 
 @pytest.mark.parametrize(
     'method',
-    [
-        'get_data',
-        'get_raw_data',
-        'describe_realms',
-        'describe_metrics',
-        'describe_dimensions',
-        'get_filter_values',
-        'describe_raw_realms',
-        'describe_raw_fields',
-    ],
+    list(METHOD_PARAMS.keys()),
 )
 def test_RuntimeError_outside_context(
     dw_methods_outside_runtime_context,
@@ -439,3 +441,12 @@ def test_case_insensitive(dw_methods, method, param, value1, value2):
     data1 = __run_method(dw_methods, method, {param: value1})
     data2 = __run_method(dw_methods, method, {param: value2})
     assert data1.equals(data2)
+
+
+@pytest.mark.parametrize(
+    'dw_methods,method',
+    [(VALID_XDMOD_URL + "/", method)for method in list(METHOD_PARAMS.keys())],
+    indirect=['dw_methods']
+)
+def test_trailing_slashes(dw_methods, method):
+    __run_method(dw_methods, method)

tests/unit/test_datawarehouse_unit.py

@@ -10,7 +10,11 @@
 
 @pytest.fixture(scope='module', autouse=True)
 def set_environ():
-    token = os.environ['XDMOD_API_TOKEN'] if 'XDMOD_API_TOKEN' in os.environ else ''
+    token = (
+        os.environ['XDMOD_API_TOKEN']
+        if 'XDMOD_API_TOKEN' in os.environ
+        else ''
+    )
     os.environ['XDMOD_API_TOKEN'] = INVALID_STR
     yield
     os.environ['XDMOD_API_TOKEN'] = token

xdmod_data/__init__.py

@@ -1 +1 @@
-from .__version__ import __version__
+from .__version__ import __version__

xdmod_data/_http_requester.py

@@ -1,5 +1,6 @@
 import json
 import os
+import re
 import requests
 from urllib.parse import urlencode
 import xdmod_data._validator as _validator
@@ -10,6 +11,7 @@ class _HttpRequester:
     def __init__(self, xdmod_host):
         self.__in_runtime_context = False
         _validator._assert_str('xdmod_host', xdmod_host)
+        xdmod_host = re.sub('/+$', '', xdmod_host)
         self.__xdmod_host = xdmod_host
         try:
             self.__api_token = os.environ['XDMOD_API_TOKEN']

xdmod_data/warehouse.py

@@ -441,3 +441,5 @@ def __describe_metrics_or_dimensions(self, realm, m_or_d):
             ('id', 'label', 'description'),
             'id',
         )
+
+