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

Add decorator deleting cache for template folder client update method

Browse Source
This commit is contained in:
Pea Tyczynska
2018-11-13 11:43:59 +00:00
parent 0a4b6dbb33
commit c27b97380d
2 changed files with 19 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
app/notify_client/template_folder_api_client.py
View File

@@ -43,6 +43,7 @@ class TemplateFolderAPIClient(NotifyAdminAPIClient):
template_ids,
))
@cache.delete('service-{service_id}-template-folders')
def update_template_folder(self, service_id, template_folder_id, name):
self.post(
'/service/{}/template-folder/{}'.format(service_id, template_folder_id),

18
tests/app/notify_client/test_template_folder_client.py
View File

@@ -103,3 +103,21 @@ def test_move_templates_and_folders_to_root(mocker, api_user_active):
'templates': ['a', 'b', 'c'],
},
)
def test_update_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/{}'.format(some_service_id, template_folder_id)
data = {'name': 'foo'}
client = TemplateFolderAPIClient()
mock_post = mocker.patch('app.notify_client.template_folder_api_client.TemplateFolderAPIClient.post')
client.update_template_folder(some_service_id, template_folder_id, name='foo')
mock_post.assert_called_once_with(expected_url, data)
mock_redis_delete.assert_called_once_with('service-{}-template-folders'.format(some_service_id))