darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-08-23 07:46:06 -04:00
Code Issues Packages Projects Releases Wiki Activity

Added a missing test, letter search has not yet been implemented.

Browse Source
This commit is contained in:
Rebecca Law
2018-03-08 09:31:42 +00:00
committed by Chris Hill-Scott
parent e3a75d1b7d
commit 031a19d387
1 changed files with 12 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -1972,6 +1972,18 @@ def test_search_for_notification_by_to_field_return_multiple_matches(client, not
assert str(notification4.id) not in notification_ids
def test_search_for_notification_by_to_field_return_400_for_letter_type(
client, notify_db, notify_db_session, sample_service
):
response = client.get(
'/service/{}/notifications?to={}&template_type={}'.format(sample_service.id, 'A. Name', 'letter'),
headers=[create_authorization_header()]
)
response.status_code = 400
error_message= json.loads(response.get_data(as_text=True))
assert error_message['message'] == 'Only email and SMS can use search by recipient'
def test_update_service_calls_send_notification_as_service_becomes_live(notify_db, notify_db_session, client, mocker):
send_notification_mock = mocker.patch('app.service.rest.send_notification_to_service_users')