Merge pull request #86 from reddit/add_emit_event_field_to_ExperimentConfig

Add `emit_event` field to `get_experiment()` return `ExperimentConfig`

Author: mrlevitas

reddit_decider/__init__.py

@@ -49,6 +49,7 @@ class ExperimentConfig:
     start_ts: int
     stop_ts: int
     owner: str
+    emit_event: Optional[bool] = None
 
 
 class DeciderContext:
@@ -890,6 +891,7 @@ def get_experiment(self, experiment_name: str) -> Optional[ExperimentConfig]:
             start_ts=exp_dict.get("variant_set", {}).get("start_ts"),
             stop_ts=exp_dict.get("variant_set", {}).get("stop_ts"),
             owner=exp_dict.get("owner"),
+            emit_event=bool(exp_dict.get("emit_event")),
         )
 
 

tests/decider_tests.py

@@ -1313,6 +1313,7 @@ def test_get_experiment(self):
             self.assertEqual(experiment.start_ts, cfg["start_ts"])
             self.assertEqual(experiment.stop_ts, cfg["stop_ts"])
             self.assertEqual(experiment.owner, cfg["owner"])
+            self.assertEqual(experiment.emit_event, True)
 
     def test_get_variant_without_expose_with_HG_as_control_1_and_child_returns_none_does_expose(
         self,