darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-25 08:44:23 -04:00
Code Issues Packages Projects Releases Wiki Activity

Allow get template fixture to accept version

Browse Source 
The other template fixtures do, because the real API call does, it was
omitted from the letter equivalent.
This commit is contained in:
Chris Hill-Scott
2017-07-13 08:56:33 +01:00
parent 9a81489066
commit a282850a0c
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
tests/conftest.py
View File

@@ -374,7 +374,7 @@ def mock_get_service_email_template_without_placeholders(mocker):
@pytest.fixture(scope='function') @pytest.fixture(scope='function')
def mock_get_service_letter_template(mocker, content=None, subject=None): 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( template = template_json(
service_id, service_id,
template_id, template_id,
@@ -386,7 +386,8 @@ def mock_get_service_letter_template(mocker, content=None, subject=None):
return {'data': template} return {'data': template}
return mocker.patch( 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') @pytest.fixture(scope='function')