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

Changed error format on template subject duplication error

Browse Source
This commit is contained in:
Martyn Inglis
2016-02-22 14:32:41 +00:00
parent b3b3cca65f
commit dbe914f401
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/template/rest.py
View File

@@ -44,7 +44,7 @@ def create_template(service_id):
message = "Failed to create template"
if "templates_subject_key" in str(ex):
message = 'Duplicate template subject'
return jsonify(result="error", message=message), 400
return jsonify(result="error", message=[{'subject': message}]), 400
return jsonify(result="error", message=message), 500
return jsonify(data=template_schema.dump(new_template).data), 201

2
tests/app/template/test_rest.py
View File

@@ -193,7 +193,7 @@ def test_must_have_a_uniqe_subject_on_an_email_template(notify_api, sample_servi
assert response.status_code == 400
json_resp = json.loads(response.get_data(as_text=True))
assert json_resp['result'] == 'error'
assert json_resp['message'] == 'Duplicate template subject'
assert json_resp['message'][0]['subject'] == 'Duplicate template subject'
def test_should_be_able_to_update_a_template(notify_api, sample_service):