Merge pull request #34 from HumanCellAtlas/jmackey-api-url-fix

[EASY] Allow Python bindings to redirect to separate url - hotfix

hca/added_command.py

@@ -284,7 +284,8 @@ def run(cls, args):
         query_route = [split_endpoint]
         query_route.extend(cls._get_ordered_path_args(args))
 
-        base_url = args.get('api_url', cls._get_base_url())
+        kwargs = args.get('kwargs', {})
+        base_url = kwargs.get('api_url', cls._get_base_url())
         url = base_url + "/" + "/".join(query_route)
 
         query_payload, body_payload, header_payload = cls._build_non_body_payloads(args)

test/test.py

@@ -12,6 +12,7 @@
 import pprint
 from six.moves import reload_module
 
+import requests
 import six
 
 pkg_root = os.path.abspath(os.path.join(os.path.dirname(__file__), '..'))
@@ -361,6 +362,14 @@ def test_python_bindings(self):
         resp = api.put_bundles(bundle_uuid, files=files, creator_uid=1, replica="aws")
         self.assertTrue(resp.ok)
 
+    def test_python_api_url(self):
+        kwargs = {'uuid': "fake_uuid",
+                  'replica': "aws",
+                  'api_url': "https://thisisafakeurljslfjlshsfs.com"}
+        self.assertRaises(requests.exceptions.ConnectionError,
+                          api.get_files,
+                          **kwargs)
+
     def test_python_subscriptions(self):
         query = {'bool': {}}
         resp = api.put_subscriptions(query=query, callback_url="www.example.com", replica="aws")