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

Update test with multiple placeholders added

Browse Source 
It wasn’t possible to test for this before because the order of
placeholders was non-deterministic. The order has been made
deterministic in https://github.com/alphagov/notifications-utils/pull/674
This commit is contained in:
Chris Hill-Scott
2019-11-20 13:52:00 +00:00
parent 534be751dd
commit bf385486cf
4 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -1487,11 +1487,11 @@ def test_should_403_when_create_template_with_process_type_of_priority_for_non_p
),
(
"hello ((name))",
"hello ((full name))",
"hello ((first name)) ((middle name)) ((last name))",
[
'You removed ((name))',
'You added ((full name))',
'Before you send any messages, make sure your API calls include full name.',
'You added ((first name)) ((middle name)) and ((last name))',
'Before you send any messages, make sure your API calls include first name, middle name and last name.',
]
),
])