darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-09-09 15:19:49 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #3418 from alphagov/fix-letter-branding-preview

Browse Source 
Fix letter branding preview
This commit is contained in:
Chris Hill-Scott
2020-04-21 11:35:45 +01:00
committed by GitHub
parent 788edc46f2 e56c2d28b0
commit 52e966337d
2 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/main/views/templates.py
View File

@@ -248,7 +248,8 @@ def letter_branding_preview_image(filename):
'The point of using Lorem Ipsum is that it has a more-or-less ' 'The point of using Lorem Ipsum is that it has a more-or-less '
'normal distribution of letters, as opposed to using ‘Content ' 'normal distribution of letters, as opposed to using ‘Content '
'here, content here’, making it look like readable English.' 'here, content here’, making it look like readable English.'
) ),
'template_type': 'letter',
} }
filename = None if filename == 'no-branding' else filename filename = None if filename == 'no-branding' else filename

9
tests/app/main/views/test_templates.py
View File

@@ -845,7 +845,14 @@ def test_letter_branding_preview_image(
url_for('no_cookie.letter_branding_preview_image', filename=original_filename) url_for('no_cookie.letter_branding_preview_image', filename=original_filename)
) )
mocked_preview.assert_called_with(ANY, new_filename) mocked_preview.assert_called_with(
{
'subject': 'An example letter',
'content': ANY,
'template_type': 'letter',
},
new_filename,
)
assert resp.get_data(as_text=True) == 'foo' assert resp.get_data(as_text=True) == 'foo'