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

update utils to 23.1.0 - to show alphanumeric inbound messages on the frontend

Browse Source
This commit is contained in:
Leo Hemsted
2017-11-23 15:08:11 +00:00
parent 8c915174a3
commit 36b95a740e
2 changed files with 16 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
requirements.txt
View File

@@ -21,4 +21,4 @@ notifications-python-client==4.6.0
awscli>=1.11,<1.12
awscli-cwlogs>=1.4,<1.5
git+https://github.com/alphagov/notifications-utils.git@23.0.1#egg=notifications-utils==23.0.1
git+https://github.com/alphagov/notifications-utils.git@23.1.0#egg=notifications-utils==23.1.0

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

@@ -286,3 +286,18 @@ def test_conversation_reply_redirects_with_phone_number_from_notification(
('.sms-message-wrapper', 'service one: Template <em>content</em> with & entity'),
]:
assert normalize_spaces(page.select_one(element).text) == expected_text
def test_get_user_phone_number_when_not_a_standard_phone_number(mocker):
mocker.patch(
'app.main.views.conversation.service_api_client.get_inbound_sms_by_id',
return_value={
'user_number': 'ALPHANUM3R1C',
'notify_number': '07900000002'
}
)
mocker.patch(
'app.main.views.conversation.notification_api_client.get_notification',
side_effect=HTTPError,
)
assert get_user_number('service', 'notification') == 'ALPHANUM3R1C'