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

Merge pull request #1949 from alphagov/fix-inbound-search

Browse Source 
Fix bug with inbound message page
This commit is contained in:
Rebecca Law
2018-03-12 13:32:41 +00:00
committed by GitHub
parent a429e799e1 26866f441e
commit 3a604ff7d8
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/main/views/conversation.py
View File

@@ -102,7 +102,9 @@ def get_user_number(service_id, notification_id):
def get_sms_thread(service_id, user_number):
for notification in sorted((
notification_api_client.get_notifications_for_service(service_id, to=user_number)['notifications'] +
notification_api_client.get_notifications_for_service(service_id,
to=user_number,
template_type='sms')['notifications'] +
service_api_client.get_inbound_sms(service_id, user_number=user_number)
), key=lambda notification: notification['created_at']):

4
tests/app/main/views/test_conversation.py
View File

@@ -79,7 +79,7 @@ def test_view_conversation(
mock_get_inbound_sms
):
mock_get_notifications(
mock = mock_get_notifications(
mocker,
api_user_active,
template_content='Hello ((name))',
@@ -158,6 +158,8 @@ def test_view_conversation(
normalize_spaces(statuses[index].text),
) == expected
mock.assert_called_once_with(SERVICE_ONE_ID, to='07123 456789', template_type='sms')
def test_view_conversation_updates(
logged_in_client,