From 4805ccdeed20b54438778def19a4b6bcdf895564 Mon Sep 17 00:00:00 2001 From: Chris Hill-Scott Date: Tue, 7 Feb 2017 12:47:50 +0000 Subject: [PATCH] Check email and letter count in monthly stats test MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Before we were only checking counts for SMS, because we’d only created SMS notifications. This commit also checks the counts for email and letter, which should be 0. But they’re not. So this commit is adding the test case which demonstrates the bug. --- tests/app/dao/test_services_dao.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/app/dao/test_services_dao.py b/tests/app/dao/test_services_dao.py index fde32621f..af72342d0 100644 --- a/tests/app/dao/test_services_dao.py +++ b/tests/app/dao/test_services_dao.py @@ -545,6 +545,8 @@ def test_fetch_monthly_historical_stats_separates_weeks(notify_db, notify_db_ses ('2017-03', 'created', 2), ): assert result[date]['sms'][status] == count + assert result[date]['email'][status] == count + assert result[date]['letter'][status] == count assert result.keys() == { '2016-04', '2016-05', '2016-06',