From b5c78dc55293de467db245270109262cb973d703 Mon Sep 17 00:00:00 2001 From: Pea Tyczynska Date: Tue, 13 Nov 2018 16:20:18 +0000 Subject: [PATCH] Add delete_folder method to template folder api client --- app/notify_client/template_folder_api_client.py | 4 ++++ .../test_template_folder_client.py | 17 +++++++++++++++++ 2 files changed, 21 insertions(+) diff --git a/app/notify_client/template_folder_api_client.py b/app/notify_client/template_folder_api_client.py index 0a8952e9e..d6fad053b 100644 --- a/app/notify_client/template_folder_api_client.py +++ b/app/notify_client/template_folder_api_client.py @@ -50,5 +50,9 @@ class TemplateFolderAPIClient(NotifyAdminAPIClient): {"name": name} ) + @cache.delete('service-{service_id}-template-folders') + def delete_template_folder(self, service_id, template_folder_id): + self.post('/service/{}/template-folder/{}/delete'.format(service_id, template_folder_id)) + template_folder_api_client = TemplateFolderAPIClient() diff --git a/tests/app/notify_client/test_template_folder_client.py b/tests/app/notify_client/test_template_folder_client.py index 7a6a121c5..27422ea86 100644 --- a/tests/app/notify_client/test_template_folder_client.py +++ b/tests/app/notify_client/test_template_folder_client.py @@ -121,3 +121,20 @@ def test_update_template_folder_calls_correct_api_endpoint(mocker, api_user_acti mock_post.assert_called_once_with(expected_url, data) mock_redis_delete.assert_called_once_with('service-{}-template-folders'.format(some_service_id)) + + +def test_delete_template_folder_calls_correct_api_endpoint(mocker, api_user_active): + mock_redis_delete = mocker.patch('app.notify_client.RedisClient.delete') + + some_service_id = uuid.uuid4() + template_folder_id = uuid.uuid4() + expected_url = '/service/{}/template-folder/{}/delete'.format(some_service_id, template_folder_id) + + client = TemplateFolderAPIClient() + + mock_post = mocker.patch('app.notify_client.template_folder_api_client.TemplateFolderAPIClient.post') + + client.delete_template_folder(some_service_id, template_folder_id) + + mock_post.assert_called_once_with(expected_url) + mock_redis_delete.assert_called_once_with('service-{}-template-folders'.format(some_service_id))