diff --git a/app/notify_client/service_api_client.py b/app/notify_client/service_api_client.py index 4c918fa66..76c09c9ed 100644 --- a/app/notify_client/service_api_client.py +++ b/app/notify_client/service_api_client.py @@ -9,6 +9,7 @@ class ServiceAPIClient(NotifyAdminAPIClient): def __init__(self): super().__init__("a" * 73, "b") + @cache.delete('user', key_from_args=[4]) def create_service( self, service_name, diff --git a/tests/app/notify_client/test_service_api_client.py b/tests/app/notify_client/test_service_api_client.py index 39fc82665..354d08f16 100644 --- a/tests/app/notify_client/test_service_api_client.py +++ b/tests/app/notify_client/test_service_api_client.py @@ -58,12 +58,12 @@ def test_client_creates_service_with_correct_data( mocker.patch('app.notify_client.current_user', id='123') client.create_service( - service_name='My first service', - organisation_type='central_government', - message_limit=1, - restricted=True, - user_id=fake_uuid, - email_from='test@example.com', + 'My first service', + 'central_government', + 1, + True, + fake_uuid, + 'test@example.com', ) mock_post.assert_called_once_with( '/service', diff --git a/tests/app/notify_client/test_user_client.py b/tests/app/notify_client/test_user_client.py index 3eb47d458..dc90413f7 100644 --- a/tests/app/notify_client/test_user_client.py +++ b/tests/app/notify_client/test_user_client.py @@ -248,6 +248,7 @@ def test_returns_value_from_cache( (user_api_client, 'set_user_permissions', [user_id, SERVICE_ONE_ID, []], {}), (user_api_client, 'activate_user', [api_user_pending(fake_uuid())], {}), (service_api_client, 'remove_user_from_service', [SERVICE_ONE_ID, user_id], {}), + (service_api_client, 'create_service', ['', '', 0, False, user_id, fake_uuid()], {}), (invite_api_client, 'accept_invite', [SERVICE_ONE_ID, user_id], {}), ]) def test_deletes_user_cache(