add HG returns control_1 + exposure, even if child returns None + bump reddit-decider

Author: mrlevitas

requirements.txt

@@ -1,7 +1,7 @@
 -r requirements-transitive.txt
 baseplate==2.0.0a1
 black==21.4b2
-reddit-decider==1.2.15
+reddit-decider==1.3.0
 flake8==3.9.1
 mypy==0.790
 pyramid==2.0   # required for `from baseplate.frameworks.pyramid import BaseplateRequest` which calls `import pyramid.events`

setup.py

@@ -19,7 +19,7 @@
     install_requires=[
         "baseplate>=2.0.0a1,<3.0",
         "reddit-edgecontext>=1.0.0a3,<2.0",
-        "reddit-decider~=1.2.23",
+        "reddit-decider~=1.3.0",
         "typing_extensions>=3.10.0.0,<5.0",
     ],
     package_data={"reddit_experiments": ["py.typed"]},

tests/decider_tests.py

@@ -1263,6 +1263,46 @@ def test_get_experiment(self):
             self.assertEqual(experiment.stop_ts, cfg["stop_ts"])
             self.assertEqual(experiment.owner, cfg["owner"])
 
+    def test_get_variant_without_expose_with_HG_as_control_1_and_child_returns_none_does_expose(self):
+        self.exp_base_config["exp_1"].update({"parent_hg_name": "hg"})
+        # force child "exp_1" to return `None`
+        self.exp_base_config["exp_1"]["experiment"]["variants"] = [
+            {"name": "control_1", "size": 0.0, "range_end": 0.0, "range_start": 0.0},
+        ]
+
+        self.exp_base_config.update(self.parent_hg_config)
+        # force "hg" to bucket "control_1"
+        self.exp_base_config["hg"]["experiment"]["variants"] = [
+          {
+            "name": "control_1",
+            "size": 1.00,
+            "range_end": 1.0,
+            "range_start": 0
+          },
+          {
+            "name": "holdout",
+            "size": 0.00,
+            "range_end": 0.0,
+            "range_start": 0.00
+          }
+        ]
+
+        with create_temp_config_file(self.exp_base_config) as f:
+            decider = self.setup_decider(f.name, self.dc)
+
+            self.assertEqual(self.event_logger.log.call_count, 0)
+            variant = decider.get_variant_without_expose("exp_1")
+
+            assert variant == None
+
+            # exposure from control_1 of "hg"
+            self.assertEqual(self.event_logger.log.call_count, 1)
+            event_fields = self.event_logger.log.call_args[1]
+
+            # `variant == None` for child but event will fire with `variant == "control_1"` for analysis
+            self.assert_exposure_event_fields(
+                experiment_name="hg", variant="control_1", event_fields=event_fields
+            )
 
 class TestDeciderGetDynamicConfig(unittest.TestCase):
     def setUp(self):