Skip to content

Commit 4dfe87d

Browse files
More test related fixes
1 parent 52e4fea commit 4dfe87d

File tree

2 files changed

+14
-7
lines changed

2 files changed

+14
-7
lines changed

tests/integration_tests/charts/api_tests.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1035,6 +1035,7 @@ def test_get_chart(self):
10351035
"id": 1,
10361036
"first_name": "admin",
10371037
"last_name": "user",
1038+
"email": "admin@fab.org",
10381039
}
10391040
],
10401041
"params": None,

tests/integration_tests/reports/api_tests.py

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -301,12 +301,18 @@ def test_get_report_schedule(self):
301301
for key in expected_result:
302302
assert data["result"][key] == expected_result[key]
303303
# needed because order may vary
304-
assert {"first_name": "admin", "id": 1, "last_name": "user"} in data["result"][
305-
"owners"
306-
]
307-
assert {"first_name": "alpha", "id": 5, "last_name": "user"} in data["result"][
308-
"owners"
309-
]
304+
assert {
305+
"email": "admin@fab.org",
306+
"first_name": "admin",
307+
"id": 1,
308+
"last_name": "user",
309+
} in data["result"]["owners"]
310+
assert {
311+
"email": "alpha@fab.org",
312+
"first_name": "alpha",
313+
"id": 5,
314+
"last_name": "user",
315+
} in data["result"]["owners"]
310316
assert len(data["result"]["owners"]) == 2
311317

312318
def test_info_report_schedule(self):
@@ -382,7 +388,7 @@ def test_get_list_report_schedule(self):
382388
assert expected_fields == data_keys
383389

384390
# Assert nested fields
385-
expected_owners_fields = ["first_name", "id", "last_name"]
391+
expected_owners_fields = ["email", "first_name", "id", "last_name"]
386392
data_keys = sorted(list(data["result"][0]["owners"][0].keys())) # noqa: C414
387393
assert expected_owners_fields == data_keys
388394

0 commit comments

Comments
 (0)