Skip to content

Commit eb273d1

Browse files
authored
fix(feature_flags): make test conditions deterministic (#2059)
1 parent e536a39 commit eb273d1

File tree

2 files changed

+16
-16
lines changed

2 files changed

+16
-16
lines changed

docs/utilities/feature_flags.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -827,10 +827,10 @@ def test_flags_condition_match(mocker):
827827
expected_value = True
828828
mocked_app_config_schema = {
829829
"my_feature": {
830-
"default": expected_value,
830+
"default": False,
831831
"rules": {
832832
"tenant id equals 12345": {
833-
"when_match": True,
833+
"when_match": expected_value,
834834
"conditions": [
835835
{
836836
"action": RuleAction.EQUALS.value,

tests/functional/feature_flags/test_feature_flags.py

+14-14
Original file line numberDiff line numberDiff line change
@@ -808,10 +808,10 @@ def test_flags_not_equal_match(mocker, config):
808808
expected_value = True
809809
mocked_app_config_schema = {
810810
"my_feature": {
811-
"default": expected_value,
811+
"default": False,
812812
"rules": {
813813
"tenant id not equals 345345435": {
814-
"when_match": True,
814+
"when_match": expected_value,
815815
"conditions": [
816816
{
817817
"action": RuleAction.NOT_EQUALS.value,
@@ -889,10 +889,10 @@ def test_flags_less_than_match(mocker, config):
889889
expected_value = True
890890
mocked_app_config_schema = {
891891
"my_feature": {
892-
"default": expected_value,
892+
"default": False,
893893
"rules": {
894894
"Date less than 2021.10.31": {
895-
"when_match": True,
895+
"when_match": expected_value,
896896
"conditions": [
897897
{
898898
"action": RuleAction.KEY_LESS_THAN_VALUE.value,
@@ -946,10 +946,10 @@ def test_flags_less_than_or_equal_match_1(mocker, config):
946946
expected_value = True
947947
mocked_app_config_schema = {
948948
"my_feature": {
949-
"default": expected_value,
949+
"default": False,
950950
"rules": {
951951
"Date less than or equal 2021.10.31": {
952-
"when_match": True,
952+
"when_match": expected_value,
953953
"conditions": [
954954
{
955955
"action": RuleAction.KEY_LESS_THAN_OR_EQUAL_VALUE.value,
@@ -974,10 +974,10 @@ def test_flags_less_than_or_equal_match_2(mocker, config):
974974
expected_value = True
975975
mocked_app_config_schema = {
976976
"my_feature": {
977-
"default": expected_value,
977+
"default": False,
978978
"rules": {
979979
"Date less than or equal 2021.10.31": {
980-
"when_match": True,
980+
"when_match": expected_value,
981981
"conditions": [
982982
{
983983
"action": RuleAction.KEY_LESS_THAN_OR_EQUAL_VALUE.value,
@@ -1059,10 +1059,10 @@ def test_flags_greater_than_match(mocker, config):
10591059
expected_value = True
10601060
mocked_app_config_schema = {
10611061
"my_feature": {
1062-
"default": expected_value,
1062+
"default": False,
10631063
"rules": {
10641064
"Date greater than 2021.10.31": {
1065-
"when_match": True,
1065+
"when_match": expected_value,
10661066
"conditions": [
10671067
{
10681068
"action": RuleAction.KEY_GREATER_THAN_VALUE.value,
@@ -1116,10 +1116,10 @@ def test_flags_greater_than_or_equal_match_1(mocker, config):
11161116
expected_value = True
11171117
mocked_app_config_schema = {
11181118
"my_feature": {
1119-
"default": expected_value,
1119+
"default": False,
11201120
"rules": {
11211121
"Date greater than or equal 2021.10.31": {
1122-
"when_match": True,
1122+
"when_match": expected_value,
11231123
"conditions": [
11241124
{
11251125
"action": RuleAction.KEY_GREATER_THAN_OR_EQUAL_VALUE.value,
@@ -1144,10 +1144,10 @@ def test_flags_greater_than_or_equal_match_2(mocker, config):
11441144
expected_value = True
11451145
mocked_app_config_schema = {
11461146
"my_feature": {
1147-
"default": expected_value,
1147+
"default": False,
11481148
"rules": {
11491149
"Date greater than or equal 2021.10.31": {
1150-
"when_match": True,
1150+
"when_match": expected_value,
11511151
"conditions": [
11521152
{
11531153
"action": RuleAction.KEY_GREATER_THAN_OR_EQUAL_VALUE.value,

0 commit comments

Comments
 (0)