fix(auth-utility): handle runtime exception in base64 encoding utility (#56)

This commit handles the exception when the provided props are an array of None items. When any item is None, the `delimiter.join` throws an exception because it expects the str instances.

Author: sufyankhanrao

apimatic_core/utilities/auth_helper.py

@@ -7,7 +7,7 @@ class AuthHelper:
 
     @staticmethod
     def get_base64_encoded_value(*props, delimiter=':'):
-        if props:
+        if props.__len__() > 0 and not any(prop is None for prop in props):
             joined = delimiter.join(props)
             encoded = base64.b64encode(str.encode(joined)).decode('iso-8859-1')
             return encoded

setup.py

@@ -12,7 +12,7 @@
 
 setup(
     name='apimatic-core',
-    version='0.2.8',
+    version='0.2.9',
     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/utility_tests/test_auth_helper.py

@@ -8,6 +8,15 @@ def test_base64_encoded_none_value(self):
         expected_base64_encoded_value = None
         assert actual_base64_encoded_value == expected_base64_encoded_value
 
+    def test_base64_encoded_provided_none_values(self):
+        actual_base64_encoded_value = AuthHelper.get_base64_encoded_value(None, None)
+        expected_base64_encoded_value = None
+        assert actual_base64_encoded_value == expected_base64_encoded_value
+
+        actual_base64_encoded_value = AuthHelper.get_base64_encoded_value('test_username', None)
+        expected_base64_encoded_value = None
+        assert actual_base64_encoded_value == expected_base64_encoded_value
+
     def test_base64_encoded_value(self):
         actual_base64_encoded_value = AuthHelper.get_base64_encoded_value('test_username', 'test_password')
         expected_base64_encoded_value = 'dGVzdF91c2VybmFtZTp0ZXN0X3Bhc3N3b3Jk'