darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-09-10 10:03:21 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fix test that fails 1 in 60 times

Browse Source 
This test looks for how many times the string `50` appears in the text
of the `<main>` element of the page. The `<main>` element also contains
some times, for example 1:23pm.

This means that when the time reaches 1:50pm, 2:50pm, etc the number of
times the string `50` appears in the page changes. Which causes the test
assertions to fail.
This commit is contained in:
Chris Hill-Scott
2020-01-14 14:06:04 +00:00
parent 4552587829
commit 0b6744854b
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tests/app/main/views/test_dashboard.py
View File

@@ -552,8 +552,11 @@ def test_should_not_show_recent_templates_on_dashboard_if_only_one_template_used
# count appears as total, but not per template # count appears as total, but not per template
expected_count = stats[0]['count'] expected_count = stats[0]['count']
assert expected_count == 50 assert expected_count == 50
assert main.count(str(expected_count)) == 1 assert normalize_spaces(
assert '50 text messages sent' in normalize_spaces(main) page.select_one('#total-sms .big-number').text
) == (
'{} text messages sent'.format(expected_count)
)
@freeze_time("2016-07-01 12:00") # 4 months into 2016 financial year @freeze_time("2016-07-01 12:00") # 4 months into 2016 financial year