fix: change datafile accessor feature to return a string representation of datafile (#283)

* fix: change datafile accessor methods to return a JSON representation of the datafile

* fix: change datafile accessor methods to return string instead of byte array of the datafile

* test: update optimizely config datafile var name in test

* refactor: add string encoding

* Setting as unicode

Co-authored-by: aliabbasrizvi <[email protected]>

Author: pthompson127

optimizely/optimizely_config.py

@@ -21,15 +21,15 @@ def __init__(self, revision, experiments_map, features_map, datafile=None):
         self.revision = revision
         self.experiments_map = experiments_map
         self.features_map = features_map
-        self.datafile = datafile
+        self._datafile = datafile
 
     def get_datafile(self):
         """ Get the datafile associated with OptimizelyConfig.
 
         Returns:
             A JSON string representation of the environment's datafile.
         """
-        return self.datafile
+        return self._datafile
 
 
 class OptimizelyExperiment(object):

optimizely/project_config.py

@@ -40,7 +40,7 @@ def __init__(self, datafile, logger, error_handler):
         """
 
         config = json.loads(datafile)
-        self._datafile = datafile
+        self._datafile = u'{}'.format(datafile)
         self.logger = logger
         self.error_handler = error_handler
         self.version = config.get('version')

tests/test_optimizely_config.py

@@ -456,7 +456,7 @@ def setUp(self):
                 }
             },
             'revision': '1',
-            'datafile': json.dumps(self.config_dict_with_features)
+            '_datafile': json.dumps(self.config_dict_with_features)
         }
 
         self.actual_config = self.opt_config_service.get_config()