feat(multiple-auth): improvements in multiple auth implementation (#48)

This commit adds improvements in the implementation of multiple authentication schemes. The improvements include the addition of auth-validation exception type and early returning in case of OR authentication group.

closes #47

Author: sufyankhanrao

README.md

@@ -46,6 +46,13 @@ pip install apimatic-core
 |--------------------------------------------------------------|--------------------------------------------------------------------------------------|
 | [`LazyProperty`](apimatic_core/decorators/lazy_property.py)  | A decorator class for lazy instantiation                                             |
 
+## Exceptions
+| Name                                                                                 | Description                                                              |
+|--------------------------------------------------------------------------------------|--------------------------------------------------------------------------|
+| [`OneOfValidationException`](apimatic_core/exceptions/oneof_validation_exception.py) | An exception class for the failed validation of oneOf (union-type) cases |
+| [`AnyOfValidationException`](apimatic_core/exceptions/anyof_validation_exception.py) | An exception class for the failed validation of anyOf (union-type) cases |
+| [`AuthValidationException`](apimatic_core/exceptions/auth_validation_exception.py)   | An exception class for the failed validation of authentication schemes   |
+
 ## Factories
 | Name                                                                      | Description                                                                 |
 |---------------------------------------------------------------------------|-----------------------------------------------------------------------------|

apimatic_core/authentication/multiple/or_auth_group.py

@@ -16,10 +16,12 @@ def is_valid(self):
             return False
 
         for participant in self.mapped_group:
-            if participant.is_valid():
-                self._is_valid_group = True
-            else:
-                self.error_messages.append(participant.error_message)
+            self._is_valid_group = participant.is_valid()
+            # returning as we encounter the first participant as valid
+            if self._is_valid_group:
+                return self._is_valid_group
 
-        return self.is_valid_group
+            self.error_messages.append(participant.error_message)
+
+        return self._is_valid_group
 

apimatic_core/authentication/multiple/single_auth.py

@@ -12,7 +12,7 @@ def __init__(self, auth_participant):
         self._auth_participant = auth_participant
         self._mapped_auth = None
         self._error_message = None
-        self._is_valid = True
+        self._is_valid = False
 
     def with_auth_managers(self, auth_managers):
         if not auth_managers.get(self._auth_participant):
@@ -23,9 +23,9 @@ def with_auth_managers(self, auth_managers):
         return self
 
     def is_valid(self):
-        if not self._mapped_auth.is_valid():
+        self._is_valid = self._mapped_auth.is_valid()
+        if not self._is_valid:
             self._error_message = self._mapped_auth.error_message
-            self._is_valid = False
 
         return self._is_valid
 

apimatic_core/exceptions/__init__.py

@@ -1,4 +1,5 @@
 __all__ = [
     'oneof_validation_exception',
-    'anyof_validation_exception'
+    'anyof_validation_exception',
+    'auth_validation_exception'
 ]

apimatic_core/exceptions/anyof_validation_exception.py

@@ -1,5 +1,10 @@
+"""
+This is an exception class which will be raised when union type validation fails.
+"""
+
 
 class AnyOfValidationException(Exception):
+
     def __init__(self, message):
         self.message = message
         super().__init__(self.message)

apimatic_core/exceptions/auth_validation_exception.py

@@ -0,0 +1,10 @@
+"""
+This is an exception class which will be raised when auth validation fails.
+"""
+
+
+class AuthValidationException(Exception):
+
+    def __init__(self, message):
+        self.message = message
+        super().__init__(self.message)

apimatic_core/exceptions/oneof_validation_exception.py

@@ -1,5 +1,10 @@
+"""
+This is an exception class which will be raised when union type validation fails.
+"""
+
 
 class OneOfValidationException(Exception):
+
     def __init__(self, message):
         self.message = message
         super().__init__(self.message)

apimatic_core/request_builder.py

@@ -1,3 +1,4 @@
+from apimatic_core.exceptions.auth_validation_exception import AuthValidationException
 from apimatic_core.http.request.http_request import HttpRequest
 from apimatic_core.types.array_serialization_format import SerializationFormats
 from apimatic_core.utilities.api_helper import ApiHelper
@@ -229,4 +230,4 @@ def apply_auth(self, auth_managers, http_request):
             if self._auth.with_auth_managers(auth_managers).is_valid():
                 self._auth.apply(http_request)
             else:
-                raise PermissionError(self._auth.error_message)
+                raise AuthValidationException(self._auth.error_message)

setup.py

@@ -12,7 +12,7 @@
 
 setup(
     name='apimatic-core',
-    version='0.2.5',
+    version='0.2.6',
     description='A library that contains core logic and utilities for '
                 'consuming REST APIs using Python SDKs generated by APIMatic.',
     long_description=long_description,

tests/apimatic_core/api_logger_tests/test_api_logger.py

@@ -30,15 +30,15 @@ def test_end_to_end_success_case(self):
                              'Calling the on_before_request method of http_call_back for end-to-end-test.',
                              'Raw request for end-to-end-test is: {\'http_method\': \'POST\', \'query_url\': '
                              '\'http://localhost:3000/body/model\', \'headers\': {\'Content-Type\': '
-                             '\'application/json\', \'accept\': \'application/json\', \'Authorization\': '
+                             '\'application/json\', \'accept\': \'application/json\', \'Basic-Authorization\': '
                              '\'Basic dGVzdF91c2VybmFtZTp0ZXN0X3Bhc3N3b3Jk\'}, \'query_parameters\': None, '
                              '\'parameters\': \'{"Key": "Value"}\', \'files\': {}}',
                              'Raw response for end-to-end-test is: {\'status_code\': 200, \'reason_phrase\': None, '
                              '\'headers\': {\'Content-Type\': \'application/json\', \'accept\': \'application/json\', '
-                             '\'Authorization\': \'Basic dGVzdF91c2VybmFtZTp0ZXN0X3Bhc3N3b3Jk\'}, \'text\': '
+                             '\'Basic-Authorization\': \'Basic dGVzdF91c2VybmFtZTp0ZXN0X3Bhc3N3b3Jk\'}, \'text\': '
                              '\'{"Key": "Value"}\', \'request\': {\'http_method\': \'POST\', \'query_url\': '
                              '\'http://localhost:3000/body/model\', \'headers\': {\'Content-Type\': '
-                             '\'application/json\', \'accept\': \'application/json\', \'Authorization\': '
+                             '\'application/json\', \'accept\': \'application/json\', \'Basic-Authorization\': '
                              '\'Basic dGVzdF91c2VybmFtZTp0ZXN0X3Bhc3N3b3Jk\'}, \'query_parameters\': None, '
                              '\'parameters\': \'{"Key": "Value"}\', \'files\': {}}}',
                              'Calling on_after_response method of http_call_back for end-to-end-test.',