From 029daf18202ef6d8c28628521c50d7c9619b534c Mon Sep 17 00:00:00 2001 From: Rebecca Law Date: Tue, 22 Aug 2017 16:43:22 +0100 Subject: [PATCH] Added an assert that the set cache method is not called if the cache values are empty --- tests/app/template_statistics/test_rest.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/app/template_statistics/test_rest.py b/tests/app/template_statistics/test_rest.py index a13993768..f71bea3af 100644 --- a/tests/app/template_statistics/test_rest.py +++ b/tests/app/template_statistics/test_rest.py @@ -182,8 +182,9 @@ def set_up_notifications(notify_db, notify_db_session): @freeze_time('2016-08-18') -def test_returns_empty_list_if_no_templates_used(client, sample_service): +def test_returns_empty_list_if_no_templates_used(client, sample_service, mocker): auth_header = create_authorization_header() + mock_redis = mocker.patch('app.redis_store.set_hash_and_expire') response = client.get( '/service/{}/template-statistics'.format(sample_service.id), @@ -193,6 +194,7 @@ def test_returns_empty_list_if_no_templates_used(client, sample_service): assert response.status_code == 200 json_resp = json.loads(response.get_data(as_text=True)) assert len(json_resp['data']) == 0 + mock_redis.assert_not_called() def test_get_template_statistics_by_id_returns_last_notification(