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

Add comment clarifying timings in test case

Browse Source
This commit is contained in:
Chris Hill-Scott
2020-01-23 16:31:21 +00:00
parent 291734b0c4
commit 3b108e8c93
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
tests/app/main/views/test_jobs.py
View File

@@ -349,6 +349,12 @@ def test_should_show_job_without_notifications(
'No messages to show yet…' 'No messages to show yet…'
)), )),
# Created a while ago, started exactly 24h ago # Created a while ago, started exactly 24h ago
# ---
# It doesn’t matter that 24h (1 day) and 7 days (the service’s data
# retention) don’t match up. We’re testing the case of no
# notifications existing more than 1 day after the job started
# processing. In this case we assume it’s because the service’s
# data retention has kicked in.
(datetime(2020, 1, 1, 0, 0, 0), datetime(2020, 1, 9, 1, 0, 0), ( (datetime(2020, 1, 1, 0, 0, 0), datetime(2020, 1, 9, 1, 0, 0), (
'These messages have been deleted because they were sent more than 7 days ago' 'These messages have been deleted because they were sent more than 7 days ago'
)), )),