darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-03 09:51:11 -05:00
Code Issues Packages Projects Releases Wiki Activity

Check email and letter count in monthly stats test

Browse Source 
Before we were only checking counts for SMS, because we’d only created
SMS notifications.

This commit also checks the counts for email and letter, which should be
0. But they’re not. So this commit is adding the test case which
demonstrates the bug.
This commit is contained in:
Chris Hill-Scott
2017-02-07 12:47:50 +00:00
parent dd2e754730
commit 4805ccdeed
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tests/app/dao/test_services_dao.py
View File

@@ -545,6 +545,8 @@ def test_fetch_monthly_historical_stats_separates_weeks(notify_db, notify_db_ses
('2017-03', 'created', 2), ('2017-03', 'created', 2),
): ):
assert result[date]['sms'][status] == count assert result[date]['sms'][status] == count
assert result[date]['email'][status] == count
assert result[date]['letter'][status] == count
assert result.keys() == { assert result.keys() == {
'2016-04', '2016-05', '2016-06', '2016-04', '2016-05', '2016-06',