darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-02-05 02:42:26 -05:00
Code Issues Packages Projects Releases Wiki Activity

Restore template content when deleting template

Browse Source 
When the template content was renamed in
9ee8610da0 I missed doing the same change for the
delete template route.

This commit does the same fix, so that template content is still visible when
you’re about to delete a template (so you can make sure it’s the right one).
This commit is contained in:
Chris Hill-Scott
2016-02-08 12:30:01 +00:00
parent d57e54874d
commit de6ed99006
2 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
app/main/views/templates.py
View File

@@ -92,6 +92,7 @@ def delete_service_template(service_id, template_id):
else: else:
raise e raise e
template['template_content'] = template['content']
form = TemplateForm(**template) form = TemplateForm(**template)
if request.method == 'POST': if request.method == 'POST':

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

@@ -95,8 +95,11 @@ def test_should_show_delete_template_page(app_,
service_id=service_id, service_id=service_id,
template_id=template_id)) template_id=template_id))
content = response.get_data(as_text=True)
assert response.status_code == 200 assert response.status_code == 200
assert 'Are you sure' in response.get_data(as_text=True) assert 'Are you sure' in content
assert 'Two week reminder' in content
assert 'Your vehicle tax is about to expire' in content
mock_get_service_template.assert_called_with( mock_get_service_template.assert_called_with(
service_id, template_id) service_id, template_id)