Skip to content

Commit 67c1b23

Browse files
authored
Merge pull request #119 from reddit/remove-owner-from-tests
Remove owner from tests
2 parents 914f4a5 + 746a735 commit 67c1b23

File tree

6 files changed

+109
-103
lines changed

6 files changed

+109
-103
lines changed

tests/decider_tests.py

+15-15
Original file line numberDiff line numberDiff line change
@@ -379,7 +379,7 @@ def setUp(self):
379379
"emit_event": True,
380380
"start_ts": 37173982,
381381
"stop_ts": 2147483648,
382-
"owner": "test_owner",
382+
"owner": "",
383383
"experiment": {
384384
"variants": [
385385
{"range_start": 0.0, "range_end": 0.2, "name": "control_1"},
@@ -416,7 +416,7 @@ def setUp(self):
416416
"stop_ts": 9668199193,
417417
"id": 2,
418418
"name": "hg",
419-
"owner": "test",
419+
"owner": "",
420420
"value": "range_variant",
421421
}
422422
}
@@ -426,7 +426,7 @@ def setUp(self):
426426
"enabled": True,
427427
"version": "4",
428428
"type": "range_variant",
429-
"owner": "test",
429+
"owner": "",
430430
"emit_event": True,
431431
"experiment": {
432432
"variants": [
@@ -446,7 +446,7 @@ def setUp(self):
446446
"enabled": True,
447447
"version": "5",
448448
"type": "range_variant",
449-
"owner": "test",
449+
"owner": "",
450450
"emit_event": True,
451451
"experiment": {
452452
"variants": [
@@ -544,7 +544,7 @@ def test_none_returned_on_get_variant_call_with_bad_id(self):
544544
"test": {
545545
"id": "1",
546546
"name": "test",
547-
"owner": "test_owner",
547+
"owner": "",
548548
"type": "r2",
549549
"version": "1",
550550
"start_ts": 0,
@@ -583,7 +583,7 @@ def test_none_returned_on_get_variant_call_with_no_experiment_data(self):
583583
"test": {
584584
"id": 1,
585585
"name": "test",
586-
"owner": "test_owner",
586+
"owner": "",
587587
"type": "dynamic_config",
588588
"version": "1",
589589
"start_ts": 0,
@@ -604,7 +604,7 @@ def test_get_variant_calls_with_partial_data(self):
604604
"test": {
605605
"id": 1,
606606
"name": "test",
607-
"owner": "test_owner",
607+
"owner": "",
608608
"type": "dynamic_config",
609609
"version": "1",
610610
"start_ts": 0,
@@ -1517,7 +1517,7 @@ def test_range_variant_emit_event_override(self):
15171517
"version": "1",
15181518
"start_ts": 1522306800,
15191519
"stop_ts": 32533405261,
1520-
"owner": "[email protected]",
1520+
"owner": "",
15211521
"type": "feature_rollout",
15221522
"emit_event": False,
15231523
"experiment": {
@@ -1537,7 +1537,7 @@ def test_range_variant_emit_event_override(self):
15371537
"version": "1",
15381538
"start_ts": 1522306800,
15391539
"stop_ts": 32533405261,
1540-
"owner": "[email protected]",
1540+
"owner": "",
15411541
"type": "range_variant",
15421542
"emit_event": False,
15431543
"measured": True,
@@ -1659,7 +1659,7 @@ def test_get_variant_for_okta_groups(self):
16591659
"type": "dynamic_config",
16601660
"version": "1",
16611661
"enabled": False,
1662-
"owner": "test",
1662+
"owner": "",
16631663
"name": "$override_group",
16641664
"value_type": "Map",
16651665
"experiment": {"experiment_version": 1},
@@ -1701,7 +1701,7 @@ def setUp(self):
17011701
"type": "dynamic_config",
17021702
"start_ts": 37173982,
17031703
"stop_ts": 2147483648,
1704-
"owner": "test_owner",
1704+
"owner": "",
17051705
"experiment": {
17061706
"experiment_version": 1,
17071707
},
@@ -1855,7 +1855,7 @@ def test_get_all_values(self):
18551855
"type": "dynamic_config",
18561856
"version": "2",
18571857
"enabled": True,
1858-
"owner": "test",
1858+
"owner": "",
18591859
"name": "dc_missing_value_type",
18601860
"experiment": {"experiment_version": 2},
18611861
}
@@ -1866,7 +1866,7 @@ def test_get_all_values(self):
18661866
"id": 6299,
18671867
"name": "genexp_0",
18681868
"enabled": True,
1869-
"owner": "test",
1869+
"owner": "",
18701870
"version": "5",
18711871
"emit_event": True,
18721872
"type": "range_variant",
@@ -1890,7 +1890,7 @@ def test_get_all_values(self):
18901890
"id": 3248,
18911891
"name": "exp_0",
18921892
"enabled": True,
1893-
"owner": "test",
1893+
"owner": "",
18941894
"version": "2",
18951895
"type": "range_variant",
18961896
"emit_event": True,
@@ -1914,7 +1914,7 @@ def test_get_all_values(self):
19141914
"id": 3246,
19151915
"name": "exp_1",
19161916
"enabled": True,
1917-
"owner": "test",
1917+
"owner": "",
19181918
"version": "2",
19191919
"type": "range_variant",
19201920
"emit_event": True,

0 commit comments

Comments
 (0)