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

Add validation-failed and virus-scan-failed in the failed counts for stats

Browse Source
This commit is contained in:
Rebecca Law
2018-10-29 16:57:06 +00:00
parent f0c61c8f29
commit 80d02c434b
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/service/statistics.py
View File

@@ -83,7 +83,9 @@ def _update_statuses_from_row(update_dict, row):
update_dict['requested'] += row.count update_dict['requested'] += row.count
if row.status in ('delivered', 'sent'): if row.status in ('delivered', 'sent'):
update_dict['delivered'] += row.count update_dict['delivered'] += row.count
elif row.status in ('failed', 'technical-failure', 'temporary-failure', 'permanent-failure'): elif row.status in (
'failed', 'technical-failure', 'temporary-failure',
'permanent-failure', 'validation-failed', 'virus-scan-failed'):
update_dict['failed'] += row.count update_dict['failed'] += row.count

4
tests/app/service/test_statistics.py
View File

@@ -35,7 +35,9 @@ NewStatsRow = collections.namedtuple('row', ('notification_type', 'status', 'key
StatsRow('email', 'technical-failure', 1), StatsRow('email', 'technical-failure', 1),
StatsRow('email', 'temporary-failure', 1), StatsRow('email', 'temporary-failure', 1),
StatsRow('email', 'permanent-failure', 1), StatsRow('email', 'permanent-failure', 1),
], [4, 0, 4], [0, 0, 0], [0, 0, 0]), StatsRow('email', 'validation-failed', 1),
StatsRow('email', 'virus-scan-failed', 1),
], [6, 0, 6], [0, 0, 0], [0, 0, 0]),
'convert_sent_to_delivered': ([ 'convert_sent_to_delivered': ([
StatsRow('sms', 'sending', 1), StatsRow('sms', 'sending', 1),
StatsRow('sms', 'delivered', 1), StatsRow('sms', 'delivered', 1),