Configure CDSHandler as library

According to https://www.kbairak.net/programming/python/2020/09/16/global-singleton-vs-instance-for-libraries.html

Author: Konstantinos Bairaktaris

tests/native/core/test_cds.py

@@ -23,11 +23,9 @@ def _lang_lists_equal(self, list_1, list_2):
     @patch('transifex.native.cds.logger')
     def test_fetch_languages(self, patched_logger):
         cds_host = 'https://some.host'
-        cds_handler = CDSHandler(
-            ['el', 'en'],
-            'some_token',
-            host=cds_host
-        )
+        cds_handler = CDSHandler(configured_languages=['el', 'en'],
+                                 token='some_token',
+                                 host=cds_host)
 
         # correct response
         responses.add(
@@ -140,11 +138,9 @@ def test_fetch_languages(self, patched_logger):
     @patch('transifex.native.cds.logger')
     def test_fetch_translations(self, patched_logger):
         cds_host = 'https://some.host'
-        cds_handler = CDSHandler(
-            ['el', 'en', 'fr'],
-            'some_token',
-            host=cds_host
-        )
+        cds_handler = CDSHandler(configured_languages=['el', 'en', 'fr'],
+                                 token='some_token',
+                                 host=cds_host)
 
         # add response for languages
         responses.add(
@@ -286,11 +282,9 @@ def test_fetch_translations(self, patched_logger):
     def test_fetch_translations_etags_management(self, patched_logger):
 
         cds_host = 'https://some.host'
-        cds_handler = CDSHandler(
-            ['el', 'en'],
-            'some_token',
-            host=cds_host
-        )
+        cds_handler = CDSHandler(configured_languages=['el', 'en'],
+                                 token='some_token',
+                                 host=cds_host)
 
         # add response for languages
         responses.add(
@@ -352,23 +346,19 @@ def test_fetch_translations_etags_management(self, patched_logger):
         assert cds_handler.etags.get('el') == 'some_unique_tag_is_here'
 
     def test_push_source_strings_no_secret(self):
-        cds_handler = CDSHandler(
-            ['el', 'en'],
-            'some_token',
-        )
+        cds_handler = CDSHandler(configured_languages=['el', 'en'],
+                                 token='some_token')
         with pytest.raises(Exception):
             cds_handler.push_source_strings([], False)
 
     @responses.activate
     @patch('transifex.native.cds.logger')
     def test_push_source_strings(self, patched_logger):
         cds_host = 'https://some.host'
-        cds_handler = CDSHandler(
-            ['el', 'en'],
-            'some_token',
-            secret='some_secret',
-            host=cds_host
-        )
+        cds_handler = CDSHandler(configured_languages=['el', 'en'],
+                                 token='some_token',
+                                 secret='some_secret',
+                                 host=cds_host)
 
         # test push no correct
         responses.add(
@@ -426,12 +416,10 @@ def test_push_source_strings(self, patched_logger):
 
     def test_get_headers(self):
         cds_host = 'https://some.host'
-        cds_handler = CDSHandler(
-            ['el', 'en'],
-            'some_token',
-            secret='some_secret',
-            host=cds_host
-        )
+        cds_handler = CDSHandler(configured_languages=['el', 'en'],
+                                 token='some_token',
+                                 secret='some_secret',
+                                 host=cds_host)
         assert cds_handler._get_headers() == {
             'Authorization': 'Bearer some_token',
             'Accept-Encoding': 'gzip',
@@ -456,11 +444,9 @@ def test_get_headers(self):
     @responses.activate
     def test_retry_fetch_languages(self):
         cds_host = 'https://some.host'
-        cds_handler = CDSHandler(
-            ['el', 'en'],
-            'some_token',
-            host=cds_host,
-        )
+        cds_handler = CDSHandler(configured_languages=['el', 'en'],
+                                 token='some_token',
+                                 host=cds_host)
         responses.add(responses.GET, cds_host + '/languages', status=202)
         responses.add(responses.GET, cds_host + '/languages', status=202)
         responses.add(responses.GET, cds_host + '/languages',
@@ -477,11 +463,9 @@ def test_retry_fetch_languages(self):
     @responses.activate
     def test_retry_fetch_translations(self):
         cds_host = 'https://some.host'
-        cds_handler = CDSHandler(
-            ['el', 'en'],
-            'some_token',
-            host=cds_host,
-        )
+        cds_handler = CDSHandler(configured_languages=['el', 'en'],
+                                 token='some_token',
+                                 host=cds_host)
         responses.add(responses.GET, cds_host + '/content/el', status=202)
         responses.add(responses.GET, cds_host + '/content/el', status=202)
         responses.add(responses.GET,

transifex/native/cds.py

@@ -47,21 +47,33 @@ def get(self, key):
 class CDSHandler(object):
     """Handles communication with the Content Delivery Service."""
 
-    def __init__(self, configured_languages, token, secret=None,
-                 host=TRANSIFEX_CDS_HOST):
+    def __init__(self, **kwargs):
         """Constructor.
 
         :param list configured_languages: a list of language codes for the
             configured languages in the application
         :param str token: the API token to use for connecting to the CDS
         :param str host: the host of the Content Delivery Service
         """
-        self.configured_language_codes = configured_languages
-        self.token = token
-        self.secret = secret
-        self.host = host or TRANSIFEX_CDS_HOST
+        self.configured_language_codes = None
+        self.token = None
+        self.secret = None
+        self.host = TRANSIFEX_CDS_HOST
         self.etags = EtagStore()
 
+        self.setup(**kwargs)
+
+    def setup(self, configured_languages=None, token=None, secret=None,
+              host=None):
+        if configured_languages is not None:
+            self.configured_language_codes = configured_languages
+        if token is not None:
+            self.token = token
+        if secret is not None:
+            self.secret = secret
+        if host is not None:
+            self.host = host
+
     def fetch_languages(self):
         """Fetch the languages defined in the CDS for the specific project.
 

transifex/native/core.py

@@ -49,9 +49,9 @@ def init(
         self._cache = MemoryCache()
         self._missing_policy = missing_policy or SourceStringPolicy()
         self._error_policy = error_policy or SourceStringErrorPolicy()
-        self._cds_handler = CDSHandler(
-            self._languages, token, secret=secret, host=cds_host
-        )
+        self._cds_handler = CDSHandler(configured_languages=self._languages,
+                                       token=token, secret=secret,
+                                       host=cds_host)
 
     def translate(
         self, source_string, language_code, is_source=False,