diff --git a/app/formatters.py b/app/formatters.py index c0dda4738..11ae113d9 100644 --- a/app/formatters.py +++ b/app/formatters.py @@ -285,7 +285,7 @@ def format_notification_status_as_url(status, notification_type): return { 'email': url(_anchor='email-statuses'), - 'sms': url(_anchor='sms-statuses') + 'sms': url(_anchor='text-message-statuses') }.get(notification_type) diff --git a/tests/app/main/test_formatters.py b/tests/app/main/test_formatters.py index 3739c942b..58f0b1b56 100644 --- a/tests/app/main/test_formatters.py +++ b/tests/app/main/test_formatters.py @@ -22,9 +22,9 @@ from app.formatters import ( ('temporary-failure', 'email', partial(url_for, 'main.message_status', _anchor='email-statuses')), ('permanent-failure', 'email', partial(url_for, 'main.message_status', _anchor='email-statuses')), ('technical-failure', 'email', partial(url_for, 'main.message_status', _anchor='email-statuses')), - ('temporary-failure', 'sms', partial(url_for, 'main.message_status', _anchor='sms-statuses')), - ('permanent-failure', 'sms', partial(url_for, 'main.message_status', _anchor='sms-statuses')), - ('technical-failure', 'sms', partial(url_for, 'main.message_status', _anchor='sms-statuses')), + ('temporary-failure', 'sms', partial(url_for, 'main.message_status', _anchor='text-message-statuses')), + ('permanent-failure', 'sms', partial(url_for, 'main.message_status', _anchor='text-message-statuses')), + ('technical-failure', 'sms', partial(url_for, 'main.message_status', _anchor='text-message-statuses')), # Letter statuses are never linked ('technical-failure', 'letter', lambda: None), ('cancelled', 'letter', lambda: None), diff --git a/tests/app/main/views/test_index.py b/tests/app/main/views/test_index.py index 78deab351..e5b4eb842 100644 --- a/tests/app/main/views/test_index.py +++ b/tests/app/main/views/test_index.py @@ -188,12 +188,12 @@ def test_message_status_page_contains_message_status_ids(client_request): page = client_request.get('main.message_status') assert page.find(id='email-statuses') - assert page.find(id='sms-statuses') + assert page.find(id='text-message-statuses') def test_message_status_page_contains_link_to_support(client_request): page = client_request.get('main.message_status') - sms_status_table = page.find(id='sms-statuses').findNext('tbody') + sms_status_table = page.find(id='text-message-statuses').findNext('tbody') temp_fail_details_cell = sms_status_table.select_one('tr:nth-child(4) > td:nth-child(2)') assert temp_fail_details_cell.find('a').attrs['href'] == url_for('main.support')