Merge pull request #912 from alphagov/fix-dashboard-page

Fix dashboard page
2026-05-05 16:38:59 -04:00 · 2016-08-30 11:00:29 +01:00
parent 69df08e0d0 fc3af8b56e
commit 1651c72b3d
2 changed files with 187 additions and 4 deletions
--- a/tests/app/notify_client/test_job_client.py
+++ b/tests/app/notify_client/test_job_client.py
@@ -157,3 +157,178 @@ def test_client_parses_job_stats(mocker):
    assert result['data']['notifications_sent'] == 50
    assert result['data']['notification_count'] == 80
    assert result['data']['notifications_failed'] == 40
+
+
+def test_client_parses_empty_job_stats(mocker):
+    mocker.patch('app.notify_client.current_user', id='1')
+
+    service_id = 'service_id'
+    job_id = 'job_id'
+    expected_data = {'data': {
+        'status': 'finished',
+        'template_version': 3,
+        'id': job_id,
+        'updated_at': '2016-08-24T08:29:28.332972+00:00',
+        'service': service_id,
+        'processing_finished': '2016-08-24T08:11:48.676365+00:00',
+        'statistics': [],
+        'original_file_name': 'test-notify-email.csv',
+        'created_by': {
+            'name': 'test-user@digital.cabinet-office.gov.uk',
+            'id': '3571f2ae-7a39-4fb4-9ad7-8453f5257072'
+        },
+        'created_at': '2016-08-24T08:09:56.371073+00:00',
+        'template': 'c0309261-9c9e-4530-8fed-5f67b02260d2',
+        'notification_count': 80,
+        'processing_started': '2016-08-24T08:09:57.661246+00:00'
+    }}
+
+    expected_url = '/service/{}/job/{}'.format(service_id, job_id)
+
+    client = JobApiClient()
+    mock_get = mocker.patch('app.notify_client.job_api_client.JobApiClient.get', return_value=expected_data)
+
+    result = client.get_job(service_id, job_id)
+
+    mock_get.assert_called_once_with(url=expected_url, params={})
+    assert result['data']['notifications_sent'] == 0
+    assert result['data']['notification_count'] == 80
+    assert result['data']['notifications_failed'] == 0
+
+
+def test_client_parses_job_stats_for_service(mocker):
+    mocker.patch('app.notify_client.current_user', id='1')
+
+    service_id = 'service_id'
+    job_1_id = 'job_id_1'
+    job_2_id = 'job_id_2'
+    expected_data = {'data': [{
+        'status': 'finished',
+        'template_version': 3,
+        'id': job_1_id,
+        'updated_at': '2016-08-24T08:29:28.332972+00:00',
+        'service': service_id,
+        'processing_finished': '2016-08-24T08:11:48.676365+00:00',
+        'statistics': [
+            {'status': 'failed', 'count': 10},
+            {'status': 'technical-failure', 'count': 10},
+            {'status': 'temporary-failure', 'count': 10},
+            {'status': 'permanent-failure', 'count': 10},
+            {'status': 'created', 'count': 10},
+            {'status': 'sending', 'count': 10},
+            {'status': 'pending', 'count': 10},
+            {'status': 'delivered', 'count': 10}
+        ],
+        'original_file_name': 'test-notify-email.csv',
+        'created_by': {
+            'name': 'test-user@digital.cabinet-office.gov.uk',
+            'id': '3571f2ae-7a39-4fb4-9ad7-8453f5257072'
+        },
+        'created_at': '2016-08-24T08:09:56.371073+00:00',
+        'template': 'c0309261-9c9e-4530-8fed-5f67b02260d2',
+        'notification_count': 80,
+        'processing_started': '2016-08-24T08:09:57.661246+00:00'
+    }, {
+        'status': 'finished',
+        'template_version': 3,
+        'id': job_2_id,
+        'updated_at': '2016-08-24T08:29:28.332972+00:00',
+        'service': service_id,
+        'processing_finished': '2016-08-24T08:11:48.676365+00:00',
+        'statistics': [
+            {'status': 'failed', 'count': 5},
+            {'status': 'technical-failure', 'count': 5},
+            {'status': 'temporary-failure', 'count': 5},
+            {'status': 'permanent-failure', 'count': 5},
+            {'status': 'created', 'count': 5},
+            {'status': 'sending', 'count': 5},
+            {'status': 'pending', 'count': 5},
+            {'status': 'delivered', 'count': 5}
+        ],
+        'original_file_name': 'test-notify-email.csv',
+        'created_by': {
+            'name': 'test-user@digital.cabinet-office.gov.uk',
+            'id': '3571f2ae-7a39-4fb4-9ad7-8453f5257072'
+        },
+        'created_at': '2016-08-24T08:09:56.371073+00:00',
+        'template': 'c0309261-9c9e-4530-8fed-5f67b02260d2',
+        'notification_count': 40,
+        'processing_started': '2016-08-24T08:09:57.661246+00:00'
+    }]}
+
+    expected_url = '/service/{}/job'.format(service_id)
+
+    client = JobApiClient()
+    mock_get = mocker.patch('app.notify_client.job_api_client.JobApiClient.get', return_value=expected_data)
+
+    result = client.get_job(service_id)
+
+    mock_get.assert_called_once_with(url=expected_url, params={})
+    assert result['data'][0]['id'] == job_1_id
+    assert result['data'][0]['notifications_sent'] == 50
+    assert result['data'][0]['notification_count'] == 80
+    assert result['data'][0]['notifications_failed'] == 40
+    assert result['data'][1]['id'] == job_2_id
+    assert result['data'][1]['notifications_sent'] == 25
+    assert result['data'][1]['notification_count'] == 40
+    assert result['data'][1]['notifications_failed'] == 20
+
+
+def test_client_parses_empty_job_stats_for_service(mocker):
+    mocker.patch('app.notify_client.current_user', id='1')
+
+    service_id = 'service_id'
+    job_1_id = 'job_id_1'
+    job_2_id = 'job_id_2'
+    expected_data = {'data': [{
+        'status': 'finished',
+        'template_version': 3,
+        'id': job_1_id,
+        'updated_at': '2016-08-24T08:29:28.332972+00:00',
+        'service': service_id,
+        'processing_finished': '2016-08-24T08:11:48.676365+00:00',
+        'statistics': [],
+        'original_file_name': 'test-notify-email.csv',
+        'created_by': {
+            'name': 'test-user@digital.cabinet-office.gov.uk',
+            'id': '3571f2ae-7a39-4fb4-9ad7-8453f5257072'
+        },
+        'created_at': '2016-08-24T08:09:56.371073+00:00',
+        'template': 'c0309261-9c9e-4530-8fed-5f67b02260d2',
+        'notification_count': 80,
+        'processing_started': '2016-08-24T08:09:57.661246+00:00'
+    }, {
+        'status': 'finished',
+        'template_version': 3,
+        'id': job_2_id,
+        'updated_at': '2016-08-24T08:29:28.332972+00:00',
+        'service': service_id,
+        'processing_finished': '2016-08-24T08:11:48.676365+00:00',
+        'statistics': [],
+        'original_file_name': 'test-notify-email.csv',
+        'created_by': {
+            'name': 'test-user@digital.cabinet-office.gov.uk',
+            'id': '3571f2ae-7a39-4fb4-9ad7-8453f5257072'
+        },
+        'created_at': '2016-08-24T08:09:56.371073+00:00',
+        'template': 'c0309261-9c9e-4530-8fed-5f67b02260d2',
+        'notification_count': 40,
+        'processing_started': '2016-08-24T08:09:57.661246+00:00'
+    }]}
+
+    expected_url = '/service/{}/job'.format(service_id)
+
+    client = JobApiClient()
+    mock_get = mocker.patch('app.notify_client.job_api_client.JobApiClient.get', return_value=expected_data)
+
+    result = client.get_job(service_id)
+
+    mock_get.assert_called_once_with(url=expected_url, params={})
+    assert result['data'][0]['id'] == job_1_id
+    assert result['data'][0]['notifications_sent'] == 0
+    assert result['data'][0]['notification_count'] == 80
+    assert result['data'][0]['notifications_failed'] == 0
+    assert result['data'][1]['id'] == job_2_id
+    assert result['data'][1]['notifications_sent'] == 0
+    assert result['data'][1]['notification_count'] == 40
+    assert result['data'][1]['notifications_failed'] == 0