change super to call directly init functions, add setter for parent property

Author: Martin Kudlej

threescale_api/auth.py

@@ -37,7 +37,7 @@ class UserKeyAuth(BaseClientAuth):
     """Provides user_key authentication for api client calls"""
 
     def __init__(self, app, location=None):
-        super(UserKeyAuth, self).__init__(app, location)
+        BaseClientAuth.__init__(self, app, location)
         self.credentials = {
             self.app.service.proxy.list()["auth_user_key"]: self.app["user_key"]
         }
@@ -46,19 +46,19 @@ def __call__(self, request):
         if self.location == "authorization":
             auth = requests.auth.HTTPBasicAuth(next(iter(self.credentials.values())), "")
             return auth(request)
-        return super().__call__(request)
+        return BaseClientAuth.__call__(self, request)
 
 
 class AppIdKeyAuth(BaseClientAuth):
     """Provides app_id/app_key pair based authentication for api client calls"""
 
     def __init__(self, app, location=None):
-        super(AppIdKeyAuth, self).__init__(app, location)
+        BaseClientAuth.__init__(self, app, location)
         proxy = self.app.service.proxy.list()
         self.credentials = {
             proxy["auth_app_id"]: self.app["application_id"],
             proxy["auth_app_key"]: self.app.keys.list()["keys"][0]["key"]["value"]
         }
 
     def __call__(self, request):
-        return super().__call__(request)
+        return BaseClientAuth.__call__(self, request)

threescale_api/defaults.py

@@ -129,6 +129,7 @@ def fetch(self, entity_id: int = None, **kwargs) -> dict:
         Returns(dict): Resource dict from the 3scale
         """
         log.debug(self._log_message("[FETCH] Fetch ", entity_id=entity_id, args=kwargs))
+        print(self._log_message("[FETCH] Fetch ", entity_id=entity_id, args=kwargs))
         url = self._entity_url(entity_id=entity_id)
         response = self.rest.get(url=url, **kwargs)
         return utils.extract_response(response=response, entity=self._entity_name)
@@ -230,6 +231,7 @@ def _list(self, **kwargs) -> List['DefaultResource']:
         return instance
 
     def _entity_url(self, entity_id=None) -> str:
+        print(str(self.url), str(entity_id), str(self.threescale_client.admin_api_url))
         if not entity_id:
             return self.url
         return self.url + '/' + str(entity_id)
@@ -261,7 +263,7 @@ def __make_instance(self, extracted: dict, klass):
 
 class DefaultResource(collections.abc.MutableMapping):
     def __init__(self, client: DefaultClient = None, entity_id: int = None, entity_name: str = None,
-                 entity: dict = None):
+                 entity: dict = None, **kwargs):
         """Create instance of the resource
         Args:
             client: Client instance of the resource
@@ -281,6 +283,10 @@ def threescale_client(self) -> 'ThreeScaleClient':
     @property
     def parent(self) -> 'DefaultResource':
         return self.client.parent
+    
+    @parent.setter
+    def parent(self, parent):
+        self.client.parent = parent
 
     @parent.setter
     def parent(self, parent):
@@ -412,7 +418,7 @@ def get_default(self, **kwargs) -> Optional['DefaultResource']:
 
 class DefaultPlanResource(DefaultResource):
     def __init__(self, entity_name='system_name', **kwargs):
-        super().__init__(entity_name=entity_name, **kwargs)
+        DefaultResource.__init__(self, entity_name=entity_name, **kwargs)
 
     def set_default(self, **kwargs) -> 'DefaultStateResource':
         """Set the plan default
@@ -458,7 +464,7 @@ def set_state(self, state: str, **kwargs) -> 'DefaultStateResource':
 
 class DefaultUserResource(DefaultStateResource):
     def __init__(self, entity_name='username', **kwargs):
-        super().__init__(entity_name=entity_name, **kwargs)
+        DefaultStateResource.__init__(self, entity_name=entity_name, **kwargs)
 
     def suspend(self, **kwargs) -> 'DefaultUserResource':
         """Suspends the user

threescale_api/errors.py

@@ -1,7 +1,7 @@
 class ThreeScaleApiError(Exception):
     def __init__(self, message, *args):
         self.message = message
-        super(ThreeScaleApiError, self).__init__(message, *args)
+        Exception.__init__(self, message, *args)
 
 
 class ApiClientError(ThreeScaleApiError):
@@ -13,4 +13,4 @@ def __init__(self, code, reason, body, message: str = None):
         msg = f"Response({self.code} {reason}): {body}"
         if message:
             msg += f"; {message}"
-        super(ApiClientError, self).__init__(msg)
+        ThreeScaleApiError.__init__(self, msg)