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

fix bug with copying template from folder in other service

Browse Source
This commit is contained in:
Leo Hemsted
2019-04-03 13:59:44 +01:00
parent a85ee2a8e9
commit 3e3c11f3a0
3 changed files with 43 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
tests/app/main/views/test_templates.py
View File

@@ -1189,6 +1189,42 @@ def test_load_edit_template_with_copy_of_template(
)
def test_copy_template_loads_template_from_within_subfolder(
client_request,
active_user_with_permission_to_two_services,
mock_get_service_templates,
mock_get_non_empty_organisations_and_services_for_user,
mocker
):
template = template_json(
SERVICE_TWO_ID,
TEMPLATE_ONE_ID,
name='foo',
folder=PARENT_FOLDER_ID
)
mock_get_service_template = mocker.patch(
'app.service_api_client.get_service_template',
return_value={'data': template}
)
mock_get_template_folder = mocker.patch(
'app.template_folder_api_client.get_template_folder',
return_value=_folder('Parent folder', PARENT_FOLDER_ID),
)
client_request.login(active_user_with_permission_to_two_services)
page = client_request.get(
'main.copy_template',
service_id=SERVICE_ONE_ID,
template_id=TEMPLATE_ONE_ID,
from_service=SERVICE_TWO_ID,
)
assert page.select_one('input')['value'] == 'foo (copy)'
mock_get_service_template.assert_called_once_with(SERVICE_TWO_ID, TEMPLATE_ONE_ID)
mock_get_template_folder.assert_called_once_with(SERVICE_TWO_ID, PARENT_FOLDER_ID)
def test_cant_copy_template_from_non_member_service(
client_request,
mock_get_service_email_template,