Skip to content

Commit 088f4af

Browse files
update: add None parameter to impression events in decision tests
1 parent 0e25622 commit 088f4af

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

tests/test_user_context.py

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -401,7 +401,8 @@ def test_decide_feature_rollout(self):
401401
'rollout',
402402
expected.enabled,
403403
'test_user',
404-
user_attributes
404+
user_attributes,
405+
None
405406
)
406407

407408
# assert notification count
@@ -564,7 +565,8 @@ def test_decide_feature_null_variation(self):
564565
'rollout',
565566
expected.enabled,
566567
'test_user',
567-
{'browser': 'chrome'}
568+
{'browser': 'chrome'},
569+
None
568570
)
569571

570572
def test_decide_feature_null_variation__send_flag_decision_false(self):
@@ -841,7 +843,8 @@ def test_decide__option__exclude_variables(self):
841843
'feature-test',
842844
expected.enabled,
843845
'test_user',
844-
{'browser': 'chrome'}
846+
{'browser': 'chrome'},
847+
None
845848
)
846849

847850
def test_decide__option__include_reasons__feature_test(self):
@@ -953,7 +956,8 @@ def test_decide__option__enabled_flags_only(self):
953956
'rollout',
954957
expected.enabled,
955958
'test_user',
956-
user_attributes
959+
user_attributes,
960+
None
957961
)
958962

959963
def test_decide__default_options__with__options(self):
@@ -1512,7 +1516,8 @@ def test_should_return_valid_decision_after_setting_and_removing_forced_decision
15121516
'feature-test',
15131517
expected.enabled,
15141518
'test_user',
1515-
{}
1519+
{},
1520+
None
15161521
)
15171522

15181523
self.assertTrue('User "test_user" is in variation "control" of experiment test_experiment.'

0 commit comments

Comments
 (0)