darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-01 23:55:58 -05:00
Code Issues Packages Projects Releases Wiki Activity

Fix for performance platform updates.

Browse Source 
Changed the query to get the performance platform stats from ft_notification_status. But the date used for the query needed to be a date, not datetime so the equality worked.
This commit is contained in:
Rebecca Law
2019-04-01 12:03:57 +01:00
parent 0561fe5e11
commit 1456aa7789
3 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/celery/nightly_tasks.py
View File

@@ -157,7 +157,7 @@ def send_daily_performance_platform_stats():
def send_total_sent_notifications_to_performance_platform(day): def send_total_sent_notifications_to_performance_platform(day):
count_dict = total_sent_notifications.get_total_sent_notifications_for_day(day) count_dict = total_sent_notifications.get_total_sent_notifications_for_day(day.date())
email_sent_count = count_dict.get('email').get('count') email_sent_count = count_dict.get('email').get('count')
sms_sent_count = count_dict.get('sms').get('count') sms_sent_count = count_dict.get('sms').get('count')
letter_sent_count = count_dict.get('letter').get('count') letter_sent_count = count_dict.get('letter').get('count')

2
tests/app/billing/test_billing.py
View File

@@ -3,6 +3,7 @@ from datetime import datetime, timedelta
import json import json
import pytest import pytest
from freezegun import freeze_time
from app.models import FactBilling from app.models import FactBilling
from app.dao.date_util import get_current_financial_year_start_year, get_month_start_and_end_date_in_utc from app.dao.date_util import get_current_financial_year_start_year, get_month_start_and_end_date_in_utc
@@ -156,6 +157,7 @@ def test_update_free_sms_fragment_limit_data(client, sample_service):
assert annual_billing.free_sms_fragment_limit == 9999 assert annual_billing.free_sms_fragment_limit == 9999
@freeze_time('2018-04-21 14:00')
def test_get_yearly_usage_by_monthly_from_ft_billing_populates_deltas(client, notify_db_session): def test_get_yearly_usage_by_monthly_from_ft_billing_populates_deltas(client, notify_db_session):
service = create_service() service = create_service()
sms_template = create_template(service=service, template_type="sms") sms_template = create_template(service=service, template_type="sms")

7
tests/app/celery/test_nightly_tasks.py
View File

@@ -261,7 +261,7 @@ def test_send_total_sent_notifications_to_performance_platform_calls_with_correc
perf_mock = mocker.patch( perf_mock = mocker.patch(
'app.celery.nightly_tasks.total_sent_notifications.send_total_notifications_sent_for_day_stats') # noqa 'app.celery.nightly_tasks.total_sent_notifications.send_total_notifications_sent_for_day_stats') # noqa
today = datetime.utcnow().date() today = datetime.utcnow()
create_ft_notification_status(bst_date=today, notification_type='sms', service=sample_template.service, create_ft_notification_status(bst_date=today, notification_type='sms', service=sample_template.service,
template=sample_template) template=sample_template)
create_ft_notification_status(bst_date=today, notification_type='email', service=sample_email_template.service, create_ft_notification_status(bst_date=today, notification_type='email', service=sample_email_template.service,
@@ -283,8 +283,9 @@ def test_send_total_sent_notifications_to_performance_platform_calls_with_correc
send_total_sent_notifications_to_performance_platform(yesterday) send_total_sent_notifications_to_performance_platform(yesterday)
perf_mock.assert_has_calls([ perf_mock.assert_has_calls([
call(yesterday, 'sms', 2), call(yesterday.date(), 'sms', 2),
call(yesterday, 'email', 3) call(yesterday.date(), 'email', 3),
call(yesterday.date(), 'letter', 0)
]) ])