darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-08-01 12:18:50 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2779 from alphagov/change-error-code

Browse Source 
Change error code for duplicate reply-to address to 409 meaning conflict
This commit is contained in:
Pea M. Tyczynska
2020-03-31 14:48:56 +01:00
committed by GitHub
parent 2a528f03b8 0250bee1a0
commit 401c8e41d6
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
tests/app/service/test_rest.py
View File

@@ -2622,7 +2622,7 @@ def test_verify_reply_to_email_address_doesnt_allow_duplicates(admin_request, no
'service.verify_reply_to_email_address',
service_id=service.id,
_data=data,
_expected_status=400
_expected_status=409
)
assert response["message"] == "Your service already uses ‘reply-here@example.gov.uk’ as an email reply-to address."
@@ -2651,7 +2651,7 @@ def test_add_service_reply_to_email_address_doesnt_allow_duplicates(
'service.add_service_reply_to_email_address',
service_id=service.id,
_data=data,
_expected_status=400
_expected_status=409
)
assert response["message"] == "Your service already uses ‘reply-here@example.gov.uk’ as an email reply-to address."