updated test_helper.py to add a fake for the dynamic API version request

Updated fake() to reflect the use of a url

Author: smubbs

shopify/api_version.py

@@ -2,7 +2,6 @@
 import json
 from six.moves.urllib import request
 
-
 class InvalidVersionError(Exception):
     pass
 
@@ -28,7 +27,7 @@ def define_version(cls, version):
     def define_known_versions(cls):
         req = request.urlopen("https://app.shopify.com/services/apis.json")
         data = json.loads(req.read().decode("utf-8"))
-        for api in data['apis']:    
+        for api in data['apis']:
             if api['handle'] == 'admin':
                 for release in api['versions']:
                     if release['handle'] == 'unstable':

test/fixtures/api_version.json

@@ -0,0 +1 @@
+{"apis":[{"handle":"admin","versions":[{"handle":"2019-04","latest_supported":false,"display_name":"2019-04 (Unsupported)","supported":true},{"handle":"2019-07","latest_supported":false,"display_name":"2019-07 (Unsupported)","supported":true},{"handle":"2019-10","latest_supported":false,"display_name":"2019-10 (Unsupported)","supported":true},{"handle":"2020-01","latest_supported":false,"display_name":"2020-01","supported":true},{"handle":"2020-04","latest_supported":false,"display_name":"2020-04","supported":true},{"handle":"2020-07","latest_supported":false,"display_name":"2020-07","supported":true},{"handle":"2020-10","latest_supported":true,"display_name":"2020-10 (Latest)","supported":true},{"handle":"2021-01","latest_supported":false,"display_name":"2021-01 (Release candidate)","supported":false},{"handle":"unstable","latest_supported":false,"display_name":"unstable","supported":false}]},{"handle":"storefront","versions":[{"handle":"2019-07","latest_supported":false,"display_name":"2019-07 (Unsupported)","supported":true},{"handle":"2019-10","latest_supported":false,"display_name":"2019-10 (Unsupported)","supported":true},{"handle":"2020-01","latest_supported":false,"display_name":"2020-01","supported":true},{"handle":"2020-04","latest_supported":false,"display_name":"2020-04","supported":true},{"handle":"2020-07","latest_supported":false,"display_name":"2020-07","supported":true},{"handle":"2020-10","latest_supported":true,"display_name":"2020-10 (Latest)","supported":true},{"handle":"2021-01","latest_supported":false,"display_name":"2021-01 (Release candidate)","supported":false},{"handle":"unstable","latest_supported":false,"display_name":"unstable","supported":false}]}]}

test/test_helper.py

@@ -20,6 +20,14 @@ def setUp(self):
         self.http = http_fake.TestHandler
         self.http.set_response(Exception('Bad request'))
         self.http.site = 'https://this-is-my-test-show.myshopify.com'
+        self.fake('apis',
+                  url='https://app.shopify.com/services/apis.json',
+                  method='GET',
+                  code=200,
+                  response_headers={'Content-type': 'application/json'},
+                  body=self.load_fixture('api_version'),
+                  has_user_agent=False
+                  )
 
     def load_fixture(self, name, format='json'):
         with open(os.path.dirname(__file__)+'/fixtures/%s.%s' % (name, format), 'rb') as f:
@@ -35,13 +43,10 @@ def fake(self, endpoint, **kwargs):
             extension = ""
         else:
             extension = ".%s" % (kwargs.pop('extension', 'json'))
-
-        url = "https://this-is-my-test-show.myshopify.com%s/%s%s" % (prefix, endpoint, extension)
-        try:
-           url = kwargs['url']
-        except KeyError:
-           pass
-
+        if kwargs.get('url'):
+            url = kwargs.get('url')
+        else:
+            url = "https://this-is-my-test-show.myshopify.com%s/%s%s" % (prefix, endpoint, extension)
         headers = {}
         if kwargs.pop('has_user_agent', True):
             userAgent = 'ShopifyPythonAPI/%s Python/%s' % (shopify.VERSION, sys.version.split(' ', 1)[0])