Fix tests

Author: esrakartalOpt

optimizely/decision_service.py

@@ -714,8 +714,8 @@ def get_decision_for_flag(
             reasons.extend(holdout_decision['reasons'])
 
             decision = holdout_decision['decision']
-            # Check if user was bucketed into holdout (has a decision)
-            if decision is None:
+            # Check if user was bucketed into holdout (has a variation)
+            if decision is None or decision.variation is None:
                 continue
 
             message = (
@@ -905,9 +905,9 @@ def get_variation_for_holdout(
             self.logger.info(message)
             decide_reasons.append(message)
 
-            # Create Decision for holdout - experiment is None, source is HOLDOUT
+            # Create Decision for holdout - pass holdout dict as experiment, source is HOLDOUT
             holdout_decision: Decision = Decision(
-                experiment=None,
+                experiment=holdout,
                 variation=variation,
                 source=enums.DecisionSources.HOLDOUT,
                 cmab_uuid=None

tests/test_user_context.py

@@ -873,7 +873,7 @@ def test_decide__option__include_reasons__feature_rollout(self):
             'Audiences for rule 1 collectively evaluated to TRUE.',
             'User "test_user" meets audience conditions for targeting rule 1.',
             'User "test_user" bucketed into a targeting rule 1.',
-            'The user "test_user" is bucketed into a rollout for feature flag "test_feature_in_rollout".'
+            "The user 'test_user' is bucketed into a rollout for feature flag 'test_feature_in_rollout'."
         ]
 
         self.assertEqual(expected_reasons, actual.reasons)