Skip to content

Commit 18da160

Browse files
Tomas-PytelTomas-Pytel
authored andcommitted
Update routes tests, reuse existing fixture
1 parent ef99849 commit 18da160

2 files changed

Lines changed: 13 additions & 24 deletions

File tree

tests/conftest.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,10 +18,10 @@ def ff_from_dict(featureflags):
1818
featureflags.load_conf_from_dict(
1919
{
2020
"dict_only": False,
21-
"file_1": True,
22-
"file_2": False,
23-
"file_3": True,
24-
"file_4": True,
21+
"feat_1": True,
22+
"feat_2": False,
23+
"feat_3": True,
24+
"feat_4": True,
2525
}
2626
)
2727
return FeatureFlags

tests/test_routes.py

Lines changed: 9 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -13,37 +13,26 @@ def test_openapi_json(featureflags):
1313
assert "/all" in response.text
1414

1515

16-
def test_create_ff(featureflags):
17-
featureflags.load_conf_from_url("https://pastebin.com/raw/4Ai3j2DC")
18-
assert featureflags.get_features() == {
19-
"web_only": False,
20-
"web_1": True,
21-
"web_2": False,
22-
"web_3": True,
23-
"web_4": False,
24-
}
25-
26-
27-
def test_routes_all():
16+
def test_routes_all(ff_from_dict):
2817
response = client.get("/ff/all")
2918
assert response.status_code == 200
3019
assert response.json() == {
31-
"web_only": False,
32-
"web_1": True,
33-
"web_2": False,
34-
"web_3": True,
35-
"web_4": False,
20+
"dict_only": False,
21+
"feat_1": True,
22+
"feat_2": False,
23+
"feat_3": True,
24+
"feat_4": True,
3625
}
3726

3827

3928
def test_routes_enable_ff():
40-
response = client.get("/ff/enable/web_only")
29+
response = client.get("/ff/enable/dict_only")
4130
assert response.status_code == 200
42-
assert response.json() == {"feature_flag": "web_only", "enabled": True}
31+
assert response.json() == {"feature_flag": "dict_only", "enabled": True}
4332

4433
response = client.get("/ff/all")
4534
assert response.status_code == 200
46-
assert response.json()["web_only"] is True
35+
assert response.json()["dict_only"] is True
4736

4837

4938
def test_routes_disable_ff():

0 commit comments

Comments
 (0)