ensure stats returned for lifespan of service

even if they've never sent a notification for realsies

tests/app/service/test_rest.py

@@ -1121,3 +1121,29 @@ def test_get_detailed_service(notify_db, notify_db_session, notify_api, sample_s
     assert 'statistics' in service.keys()
     assert set(service['statistics'].keys()) == set(['sms', 'email'])
     assert service['statistics']['sms'] == stats
+
+
+@freeze_time('2016-07-28')
+def test_get_weekly_notification_stats(notify_api, sample_notification):
+    with notify_api.test_request_context(), notify_api.test_client() as client:
+        resp = client.get(
+            '/service/{}/notifications/weekly'.format(sample_notification.service_id),
+            headers=[create_authorization_header()]
+        )
+
+    assert resp.status_code == 200
+    data = json.loads(resp.get_data(as_text=True))['data']
+    assert data == {
+        '2016-07-25': {
+            'sms': {
+                'requested': 1,
+                'delivered': 0,
+                'failed': 0
+            },
+            'email': {
+                'requested': 0,
+                'delivered': 0,
+                'failed': 0
+            }
+        }
+    }

tests/app/service/test_statistics.py

@@ -69,6 +69,10 @@ def test_create_zeroed_stats_dicts():
     }
 
 
+def _stats(requested, delivered, failed):
+    return {'requested': requested, 'delivered': delivered, 'failed': failed}
+
+
 @freeze_time('2016-07-28T12:00:00')
 @pytest.mark.parametrize('created_at, statistics, expected_results', [
     # with no stats and just today, return this week's stats
@@ -78,6 +82,13 @@ def test_create_zeroed_stats_dicts():
             'email': _stats(0, 0, 0)
         }
     }),
+    # with a random created time, still create the dict for midnight
+    (datetime(2016, 7, 28, 12, 13, 14), [], {
+        datetime(2016, 7, 25, 0, 0, 0): {
+            'sms': _stats(0, 0, 0),
+            'email': _stats(0, 0, 0)
+        }
+    }),
     # with no stats but a service
     (datetime(2016, 7, 14), [], {
         datetime(2016, 7, 11): {
@@ -111,7 +122,7 @@ def test_create_zeroed_stats_dicts():
     (datetime(2016, 7, 21), [
         WeeklyStatsRow('sms', 'created', datetime(2016, 7, 18), 1),
         WeeklyStatsRow('sms', 'delivered', datetime(2016, 7, 18), 1),
-        WeeklyStatsRow('sms', 'created', datetime(2016, 7, 18), 1),
+        WeeklyStatsRow('sms', 'created', datetime(2016, 7, 25), 1),
     ], {
         datetime(2016, 7, 18): {
             'sms': _stats(2, 1, 0),
@@ -125,7 +136,3 @@ def test_create_zeroed_stats_dicts():
 ])
 def test_format_weekly_notification_stats(statistics, created_at, expected_results):
     assert format_weekly_notification_stats(statistics, created_at) == expected_results
-
-
-def _stats(requested, delivered, failed):
-    return {'requested': requested, 'delivered': delivered, 'failed': failed}