Renamed 'filename' parameter in Config initializer back to 'file'.

Author: JaviCerveraIngram

connect/config.py

@@ -12,41 +12,42 @@
 class Config(object):
     _instance = None  # Global instance
 
+    # noinspection PyShadowingBuiltins
     def __init__(
             self,
             api_url=None,
             api_key=None,
             products=None,
-            filename=None
+            file=None
     ):
         """
         initialization config for public api
         :param api_url: Public api url
         :param api_key: Service user ApiKey
         :param products (optional): Id products
-        :param filename: Config file path
+        :param file: Config file path
         """
         # Check arguments
-        if not filename and not any([api_key, api_url]):
+        if not file and not any([api_key, api_url]):
             raise ValueError('Filename or api_key and api_url are expected'
                              'in Config initialization')
         if products and not isinstance(products, (str, list)):
             raise TypeError('Products can be string or string list. Found type '
                             + type(products).__name__)
 
         # Load config from file name
-        if filename:
-            if not os.path.exists(filename):
-                raise IOError('No filename `{}` on directory'.format(filename))
+        if file:
+            if not os.path.exists(file):
+                raise IOError('No file `{}` on directory'.format(file))
 
-            with open(filename) as config_file:
+            with open(file) as config_file:
                 configs = config_file.read()
 
             try:
                 configs = json.loads(configs)
             except Exception as ex:
-                raise TypeError('Invalid config filename `{}`\n'
-                                'ERROR: {}'.format(filename, str(ex)))
+                raise TypeError('Invalid config file `{}`\n'
+                                'ERROR: {}'.format(file, str(ex)))
 
             (api_url, api_key, products) = (configs.get('apiEndpoint', ''),
                                             configs.get('apiKey', ''),
@@ -71,7 +72,7 @@ def __init__(
     @classmethod
     def get_instance(cls):
         if not cls._instance:
-            cls._instance = Config(filename='config.json')
+            cls._instance = Config(file='config.json')
         return cls._instance
 
     @property

example/example.py

@@ -15,7 +15,7 @@
 logger.setLevel("DEBUG")
 
 # If we remove this line, it is done implicitly
-Config(filename='config.json')
+Config(file='config.json')
 
 
 class ExampleRequestProcessor(FulfillmentAutomation):

tests/test_config.py

@@ -46,11 +46,11 @@ def test_global_config_immutable_properties():
 
 def test_init_config_with_non_existing_file():
     with pytest.raises(IOError):
-        Config(filename='non_existing_config.json')
+        Config(file='non_existing_config.json')
 
 
 def test_init_config_with_file():
-    _assert_config(Config(filename='config.json'))
+    _assert_config(Config(file='config.json'))
 
 
 def test_init_config_with_arguments():
@@ -72,7 +72,7 @@ def test_init_config_with_invalid_arguments():
 
 # noinspection PyPropertyAccess
 def test_config_immutable_properties():
-    config = Config(filename='config.json')
+    config = Config(file='config.json')
     with pytest.raises(AttributeError):
         config.api_key = conf_dict.get('apiKey')
         config.api_url = conf_dict.get('apiEndpoint')