From 58fe9d5b5621e66804de8ab39efa774abe3fd91a Mon Sep 17 00:00:00 2001 From: Pea Tyczynska Date: Tue, 21 Dec 2021 16:05:29 +0000 Subject: [PATCH] Revert "Remove free text allowance remaining column from organisation report" --- app/main/views/organisations.py | 3 +-- tests/app/main/views/organisations/test_organisations.py | 6 +++--- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/app/main/views/organisations.py b/app/main/views/organisations.py index 5e4f2bea9..5e6aef6bb 100644 --- a/app/main/views/organisations.py +++ b/app/main/views/organisations.py @@ -188,8 +188,7 @@ def download_organisation_usage_report(org_id): ('service_id', 'Service ID'), ('service_name', 'Service Name'), ('emails_sent', 'Emails sent'), - # TODO: reinstate this once the API is returning correct data - # ('sms_remainder', 'Free text message allowance remaining'), + ('sms_remainder', 'Free text message allowance remaining'), ]) monetary_column_names = OrderedDict([ diff --git a/tests/app/main/views/organisations/test_organisations.py b/tests/app/main/views/organisations/test_organisations.py index 561155800..3f50d5684 100644 --- a/tests/app/main/views/organisations/test_organisations.py +++ b/tests/app/main/views/organisations/test_organisations.py @@ -720,10 +720,10 @@ def test_download_organisation_usage_report( ) assert csv_report.string == ( - "Service ID,Service Name,Emails sent," + "Service ID,Service Name,Emails sent,Free text message allowance remaining," "Spent on text messages (£),Spent on letters (£)" - "\r\n596364a0-858e-42c8-9062-a8fe822260eb,Service 1,13000,1.93,30.50" - "\r\n147ad62a-2951-4fa1-9ca0-093cd1a52c52,Service 1,23000,3.94,60.50\r\n" + "\r\n596364a0-858e-42c8-9062-a8fe822260eb,Service 1,13000,,1.93,30.50" + "\r\n147ad62a-2951-4fa1-9ca0-093cd1a52c52,Service 1,23000,,3.94,60.50\r\n" )