darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-06-25 18:01:57 -04:00
Code Issues Packages Projects Releases Wiki Activity

make sure service api client doesnt expect free sms fragment

Browse Source
This commit is contained in:
Leo Hemsted
2017-12-04 16:48:13 +00:00
parent b9b3d7e9f3 c366e2e44b
commit 2a28278cb8
2 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/notify_client/service_api_client.py
View File

@@ -20,7 +20,6 @@ class ServiceAPIClient(NotifyAdminAPIClient):
self,
service_name,
organisation_type,
free_sms_fragment_limit,
message_limit,
restricted,
user_id,
@@ -32,7 +31,6 @@ class ServiceAPIClient(NotifyAdminAPIClient):
data = {
"name": service_name,
"organisation_type": organisation_type,
"free_sms_fragment_limit": free_sms_fragment_limit,
"active": True,
"message_limit": message_limit,
"user_id": user_id,

2
tests/app/main/views/test_dashboard.py
View File

@@ -630,7 +630,7 @@ def test_usage_page_with_year_argument(
logged_in_client,
mock_get_usage,
mock_get_billable_units,
mock_get_free_sms_fragment_limit
mock_get_free_sms_fragment_limit,
):
assert logged_in_client.get(url_for('main.usage', service_id=SERVICE_ONE_ID, year=2000)).status_code == 200
mock_get_billable_units.assert_called_once_with(SERVICE_ONE_ID, 2000)