improve way to set initializers [#27]

Author: dnl-blkv

bunq/__init__.py

@@ -1,7 +1,7 @@
 from bunq.sdk.json import converter
 
 
-def create_converter_initializer():
+def initialize_converter():
     """
     :rtype: None
     """
@@ -15,40 +15,28 @@ def create_converter_initializer():
     from bunq.sdk.json import converter
     from bunq.sdk.model.generated import object_
 
-    is_initialized = False
-
-    if not is_initialized:
-        converter.register_adapter(core.Installation,
-                                   adapters.InstallationAdapter)
-        converter.register_adapter(
-            core.SessionServer,
-            adapters.SessionServerAdapter
-        )
-        converter.register_adapter(
-            context.InstallationContext,
-            adapters.InstallationContextAdapter
-        )
-        converter.register_adapter(
-            context.ApiEnvironmentType,
-            adapters.ApiEnvironmentTypeAdapter
-        )
-        converter.register_adapter(float, adapters.FloatAdapter)
-        converter.register_adapter(
-            object_.Geolocation,
-            adapters.GeolocationAdapter
-        )
-        converter.register_adapter(
-            object_.MonetaryAccountReference,
-            adapters.MonetaryAccountReferenceAdapter
-        )
-        converter.register_adapter(object_.ShareDetail,
-                                   adapters.ShareDetailAdapter)
-        converter.register_adapter(datetime.datetime,
-                                   adapters.DateTimeAdapter)
-        converter.register_adapter(client.Pagination,
-                                   adapters.PaginationAdapter)
-        is_initialized = True
-
-    yield is_initialized
-
-converter.set_initializer(create_converter_initializer())
+    converter.register_adapter(core.Installation, adapters.InstallationAdapter)
+    converter.register_adapter(
+        core.SessionServer,
+        adapters.SessionServerAdapter
+    )
+    converter.register_adapter(
+        context.InstallationContext,
+        adapters.InstallationContextAdapter
+    )
+    converter.register_adapter(
+        context.ApiEnvironmentType,
+        adapters.ApiEnvironmentTypeAdapter
+    )
+    converter.register_adapter(float, adapters.FloatAdapter)
+    converter.register_adapter(object_.Geolocation, adapters.GeolocationAdapter)
+    converter.register_adapter(
+        object_.MonetaryAccountReference,
+        adapters.MonetaryAccountReferenceAdapter
+    )
+    converter.register_adapter(object_.ShareDetail, adapters.ShareDetailAdapter)
+    converter.register_adapter(datetime.datetime, adapters.DateTimeAdapter)
+    converter.register_adapter(client.Pagination, adapters.PaginationAdapter)
+
+
+converter.set_initializer_function(initialize_converter)

bunq/sdk/json/converter.py

@@ -52,7 +52,7 @@ class JsonAdapter(object):
     @classmethod
     def set_initializer(cls, initializer):
         """
-        :type initializer: function
+        :type initializer: Generator[bool, None, None]
         """
 
         cls._initializer = initializer
@@ -603,12 +603,26 @@ def types(self):
         return self._types
 
 
-def set_initializer(initializer):
+def set_initializer_function(initializer_function):
     """
-    :type initializer: Generator[bool, None, None]
+    :type initializer_function: callable
     """
 
-    JsonAdapter.set_initializer(initializer)
+    JsonAdapter.set_initializer(create_initializer(initializer_function))
+
+
+def create_initializer(initializer_function):
+    """
+    :type initializer_function: callable
+    """
+
+    is_initialized = False
+
+    if not is_initialized:
+        initializer_function()
+        is_initialized = True
+
+    yield is_initialized
 
 
 def register_adapter(target_class, adapter):