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

time_left considers flexible data retention periods

Browse Source 
Also update tests
This commit is contained in:
Pea Tyczynska
2018-11-26 15:15:06 +00:00
parent c9908befc2
commit 9d8e602ac8
5 changed files with 21 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tests/app/main/views/test_activity.py
View File

@@ -453,7 +453,7 @@ def test_should_show_notifications_for_a_service_with_next_previous(
"job_created_at, expected_message", [
("2016-01-10 11:09:00.000000+00:00", "Data available for 7 days"),
("2016-01-04 11:09:00.000000+00:00", "Data available for 1 day"),
("2016-01-03 11:09:00.000000+00:00", "Data available for 11 hours"),
("2016-01-03 11:09:00.000000+00:00", "Data available for 12 hours"),
("2016-01-02 23:59:59.000000+00:00", "Data no longer available")
]
)

11
tests/app/main/views/test_jobs.py
View File

@@ -181,6 +181,7 @@ def test_should_show_page_for_one_job(
mock_get_job,
mocker,
mock_get_notifications,
mock_get_service_data_retention_by_notification_type,
fake_uuid,
status_argument,
expected_api_call,
@@ -227,6 +228,7 @@ def test_get_jobs_should_tell_user_if_more_than_one_page(
mock_get_job,
mock_get_service_template,
mock_get_notifications_with_previous_next,
mock_get_service_data_retention_by_notification_type,
):
response = logged_in_client.get(url_for(
'main.view_job',
@@ -248,6 +250,7 @@ def test_should_show_job_in_progress(
mock_get_job_in_progress,
mocker,
mock_get_notifications,
mock_get_service_data_retention_by_notification_type,
fake_uuid,
):
@@ -267,6 +270,7 @@ def test_should_show_letter_job(
client_request,
mock_get_service_letter_template,
mock_get_job,
mock_get_service_data_retention_by_notification_type,
fake_uuid,
active_user_with_permissions,
mocker,
@@ -322,6 +326,7 @@ def test_should_show_letter_job_with_banner_after_sending(
mock_get_service_letter_template,
mock_get_job,
mock_get_notifications,
mock_get_service_data_retention_by_notification_type,
fake_uuid,
):
@@ -344,6 +349,7 @@ def test_should_show_scheduled_job(
active_user_with_permissions,
mock_get_service_template,
mock_get_scheduled_job,
mock_get_service_data_retention_by_notification_type,
mocker,
mock_get_notifications,
fake_uuid,
@@ -411,6 +417,7 @@ def test_should_show_updates_for_one_job_as_json(
mock_get_notifications,
mock_get_service_template,
mock_get_job,
mock_get_service_data_retention_by_notification_type,
mocker,
fake_uuid,
):
@@ -433,7 +440,7 @@ def test_should_show_updates_for_one_job_as_json(
"job_created_at, expected_message", [
("2016-01-10 11:09:00.000000+00:00", "Data available for 7 days"),
("2016-01-04 11:09:00.000000+00:00", "Data available for 1 day"),
("2016-01-03 11:09:00.000000+00:00", "Data available for 11 hours"),
("2016-01-03 11:09:00.000000+00:00", "Data available for 12 hours"),
("2016-01-02 23:59:59.000000+00:00", "Data no longer available")
]
)
@@ -447,6 +454,7 @@ def test_should_show_letter_job_with_first_class_if_notifications_are_first_clas
client_request,
mock_get_service_letter_template,
mock_get_job,
mock_get_service_data_retention_by_notification_type,
fake_uuid,
active_user_with_permissions,
mocker,
@@ -474,6 +482,7 @@ def test_should_show_letter_job_with_first_class_if_no_notifications(
mock_get_job,
fake_uuid,
mock_get_notifications_with_no_notifications,
mock_get_service_data_retention_by_notification_type,
mocker
):
mocker.patch('app.main.views.jobs.current_service', postage='first')

1
tests/app/main/views/test_send.py
View File

@@ -2001,6 +2001,7 @@ def test_create_job_should_call_api(
mock_get_job,
mock_get_notifications,
mock_get_service_template,
mock_get_service_data_retention_by_notification_type,
mocker,
fake_uuid,
when