darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-05-30 19:10:42 -04:00
Code Issues Packages Projects Releases Wiki Activity

Explain technical failure on the notification page

Browse Source
This commit is contained in:
Chris Hill-Scott
2019-01-10 17:25:23 +00:00
parent 989875294b
commit c1158d7d70
2 changed files with 24 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
app/templates/views/notifications/notification.html
View File

@@ -45,6 +45,11 @@
<p class="notification-status-cancelled">
Validation failed – content is outside the printable area
</p>
{% elif notification_status == 'technical-failure' %}
<p class="notification-status-cancelled">
Technical failure – Notify will resend once the team have
fixed the problem
</p>
{% else %}
<p>
{{ letter_print_day }}

25
tests/app/main/views/test_notifications.py
View File

@@ -191,13 +191,26 @@ def test_notification_page_shows_page_for_letter_notification(
assert mock_page_count.call_args_list[0][1]['values'] == {'name': 'Jo'}
@pytest.mark.parametrize('notification_status, expected_message', [
('permanent-failure', 'Cancelled 1 January at 1:02am'),
('cancelled', 'Cancelled 1 January at 1:02am'),
('validation-failed', 'Validation failed – content is outside the printable area'),
])
@pytest.mark.parametrize('notification_status, expected_message', (
(
'permanent-failure',
'Cancelled 1 January at 1:02am',
),
(
'cancelled',
'Cancelled 1 January at 1:02am',
),
(
'validation-failed',
'Validation failed – content is outside the printable area',
),
(
'technical-failure',
'Technical failure – Notify will resend once the team have fixed the problem',
),
))
@freeze_time("2016-01-01 01:01")
def test_notification_page_shows_cancelled_letter(
def test_notification_page_shows_cancelled_or_failed_letter(
client_request,
mocker,
fake_uuid,