darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-08-15 15:28:02 -04:00
Code Issues Packages Projects Releases Wiki Activity

Don’t store the underlying dict

Browse Source 
This will give us smaller objects to cache, and forces us to be explicit
about which properties we’re using.
This commit is contained in:
Chris Hill-Scott
2020-06-15 16:08:11 +01:00
parent 145d671e86
commit 415083cc3e
3 changed files with 24 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
tests/app/notifications/test_process_notification.py
View File

@@ -29,7 +29,7 @@ from tests.app.db import create_service, create_template, create_api_key
def test_create_content_for_notification_passes(sample_email_template):
template = get_template_model(sample_email_template.id, sample_email_template.service_id)
content = create_content_for_notification(template, None)
assert str(content) == template._dict['content'] + '\n'
assert str(content) == template.content + '\n'
def test_create_content_for_notification_with_placeholders_passes(sample_template_with_placeholders):
@@ -37,7 +37,7 @@ def test_create_content_for_notification_with_placeholders_passes(sample_templat
sample_template_with_placeholders.id, sample_template_with_placeholders.service_id
)
content = create_content_for_notification(template, {'name': 'Bobby'})
assert content.content == template._dict['content']
assert content.content == template.content
assert 'Bobby' in str(content)