From a282850a0c9a271e6f69acce0e10d6ea73fa6e9d Mon Sep 17 00:00:00 2001 From: Chris Hill-Scott Date: Thu, 13 Jul 2017 08:56:33 +0100 Subject: [PATCH] Allow get template fixture to accept version The other template fixtures do, because the real API call does, it was omitted from the letter equivalent. --- tests/conftest.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/conftest.py b/tests/conftest.py index c224ac513..d823bd954 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -374,7 +374,7 @@ def mock_get_service_email_template_without_placeholders(mocker): @pytest.fixture(scope='function') def mock_get_service_letter_template(mocker, content=None, subject=None): - def _create(service_id, template_id): + def _get(service_id, template_id, version=None): template = template_json( service_id, template_id, @@ -386,7 +386,8 @@ def mock_get_service_letter_template(mocker, content=None, subject=None): return {'data': template} return mocker.patch( - 'app.service_api_client.get_service_template', side_effect=_create) + 'app.service_api_client.get_service_template', side_effect=_get + ) @pytest.fixture(scope='function')