8000 feat: add datafile accessor by pthompson127 · Pull Request #275 · optimizely/python-sdk · GitHub
[go: up one dir, main page]
Skip to content

feat: add datafile accessor #275

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 13 commits into from
Jul 1, 2020
Merged

feat: add datafile accessor #275

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
e1643b8
feat: add datafile accessor
pthompson127 Jun 18, 2020
35039e5
Merge branch 'master' of https://github.com/optimizely/python-sdk int…
pthompson127 Jun 18, 2020
e250eac
refactor: fetch datafile from project_config in OptimizelyConfigServi…
pthompson127 Jun 22, 2020
a1a5de0
test: fix existing test to accommodate new config format due to dataf…
pthompson127 Jun 22, 2020
11bcb83
test: replace deprecated assert functions to eliminate warnings
pthompson127 Jun 22, 2020
2d6e858
test: add tests for datafile accessor methods
pthompson127 Jun 23, 2020
e9ef483
style: fix indent and spacing of test comments
pthompson127 Jun 23, 2020
6e2038d
style: reorder datafile accessor method to more appropriate location
pthompson127 Jun 23, 2020
7c0d615
fix: revert back to Regexp b/c python2.7 and pypy were failing
pthompson127 Jun 23, 2020
d96ec4d
docs: modify comment and incorporate changes
pthompson127 Jun 24, 2020
ad1cfae
docs: update method comment
pthompson127 Jun 26, 2020
b90ddad
Merge branch 'master' into peter/get-datafile
pthompson127 Jun 29, 2020
f12dd62
Merge branch 'master' into peter/get-datafile
pthompson127 Jul 1, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 12 additions & 2 deletions optimizely/optimizely_config.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,19 @@


class OptimizelyConfig(object):
def __init__(self, revision, experiments_map, features_map):
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

def get_datafile(self):
""" Get the datafile associated with OptimizelyConfig.

Returns:
A JSON string representation of the environment's datafile.
"""
return self.datafile


class OptimizelyExperiment(object):
Expand Down Expand Up @@ -68,6 +77,7 @@ def __init__(self, project_config):
self.is_valid = False
return

self._datafile = project_config.to_datafile()
self.experiments = project_config.experiments
self.feature_flags = project_config.feature_flags
self.groups = project_config.groups
Expand All @@ -88,7 +98,7 @@ def get_config(self):
experiments_key_map, experiments_id_map = self._get_experiments_maps()
features_map = self._get_features_map(experiments_id_map)

return OptimizelyConfig(self.revision, experiments_key_map, features_map)
return OptimizelyConfig(self.revision, experiments_key_map, features_map, self._datafile)

def _create_lookup_maps(self):
""" Creates lookup maps to avoid redundant iteration of config objects. """
Expand Down
Loading
0