darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-23 17:01:35 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1734 from alphagov/count-letters-performance-platform

Browse Source 
Send count of sent letters to performance platform
This commit is contained in:
Chris Hill-Scott
2018-03-05 14:51:47 +00:00
committed by GitHub
parent d4db959a55 7e1aa03371
commit 2d45070086
3 changed files with 16 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
app/celery/scheduled_tasks.py
View File

@@ -221,6 +221,7 @@ def send_total_sent_notifications_to_performance_platform():
count_dict = total_sent_notifications.get_total_sent_notifications_yesterday()
email_sent_count = count_dict.get('email').get('count')
sms_sent_count = count_dict.get('sms').get('count')
letter_sent_count = count_dict.get('letter').get('count')
start_date = count_dict.get('start_date')
current_app.logger.info(
@@ -240,6 +241,12 @@ def send_total_sent_notifications_to_performance_platform():
email_sent_count
)
total_sent_notifications.send_total_notifications_sent_for_day_stats(
start_date,
'letter',
letter_sent_count
)
@notify_celery.task(name='switch-current-sms-provider-on-slow-delivery')
@statsd(namespace="tasks")

6
app/performance_platform/total_sent_notifications.py
View File

@@ -27,6 +27,7 @@ def get_total_sent_notifications_yesterday():
email_count = get_total_sent_notifications_in_date_range(start_date, end_date, 'email')
sms_count = get_total_sent_notifications_in_date_range(start_date, end_date, 'sms')
letter_count = get_total_sent_notifications_in_date_range(start_date, end_date, 'letter')
return {
"start_date": start_date,
@@ -35,5 +36,8 @@ def get_total_sent_notifications_yesterday():
},
"sms": {
"count": sms_count
}
},
"letter": {
"count": letter_count
},
}

4
tests/app/performance_platform/test_total_sent_notifications.py
View File

@@ -56,6 +56,7 @@ def test_get_total_sent_notifications_yesterday_returns_expected_totals_dict(
# Create some notifications for the day before
yesterday = datetime(2016, 1, 10, 15, 30, 0, 0)
with freeze_time(yesterday):
notification_history(notification_type='letter')
notification_history(notification_type='sms')
notification_history(notification_type='sms')
notification_history(notification_type='email')
@@ -71,5 +72,8 @@ def test_get_total_sent_notifications_yesterday_returns_expected_totals_dict(
},
"sms": {
"count": 2
},
"letter": {
"count": 1
}
}