darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-24 16:24:08 -04:00
Code Issues Packages Projects Releases Wiki Activity

update free sms fragment limit to go via billing client instead of service

Browse Source
This commit is contained in:
Leo Hemsted
2017-12-04 16:03:11 +00:00
parent 109832b090
commit eb3d4acb45
6 changed files with 9 additions and 25 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
tests/app/main/views/test_service_settings.py
View File

@@ -1596,7 +1596,6 @@ def test_should_show_page_to_set_sms_allowance(
])
def test_should_set_sms_allowance(
logged_in_platform_admin_client,
mock_update_service,
given_allowance,
expected_api_argument,
mock_get_free_sms_fragment_limit,
@@ -1615,15 +1614,10 @@ def test_should_set_sms_allowance(
assert response.status_code == 302
assert response.location == url_for('main.service_settings', service_id=SERVICE_ONE_ID, _external=True)
mock_update_service.assert_called_once_with(
mock_create_or_update_free_sms_fragment_limit.assert_called_with(
SERVICE_ONE_ID,
free_sms_fragment_limit=expected_api_argument,
expected_api_argument
)
# Not assert until using the annual_billing end points
# mock_create_or_update_free_sms_fragment_limit.assert_called_with(
# SERVICE_ONE_ID,
# expected_api_argument
# )
def test_switch_service_enable_letters(