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

Make tests work with new template list HTML

Browse Source 
Adds the extra text added to each checkbox label.
It's a copy of the text of the link in the same
list item which does add a lot of duplication to
the test data.

This reformats a lot of the test data, stacking it
to separate out the duplicate items.
This commit is contained in:
Tom Byers
2020-06-09 16:44:05 +01:00
parent ee03753187
commit 64cc9f2520
2 changed files with 158 additions and 91 deletions
Download Patch File Download Diff File Expand all files Collapse all files

247
tests/app/main/views/test_template_folders.py
View File

@@ -54,29 +54,43 @@ def _folder(name, folder_id=None, parent=None, users_with_permission=None):
{}, {},
['Email', 'Text message', 'Letter'], ['Email', 'Text message', 'Letter'],
[ [
'folder_one 2 folders', 'folder_one folder_one 2 folders',
'folder_one folder_one_one 1 template, 1 folder', ('folder_one folder_one_one '
'folder_one folder_one_one folder_one_one_one 1 template', 'folder_one folder_one_one '
'folder_one folder_one_one folder_one_one_one sms_template_nested Text message template', '1 template, 1 folder'),
'folder_one folder_one_one letter_template_nested Letter template', ('folder_one folder_one_one folder_one_one_one '
'folder_one folder_one_two Empty', 'folder_one folder_one_one folder_one_one_one '
'folder_two Empty', '1 template'),
'sms_template_one Text message template', ('folder_one folder_one_one folder_one_one_one sms_template_nested '
'sms_template_two Text message template', 'folder_one folder_one_one folder_one_one_one sms_template_nested '
'email_template_one Email template', 'Text message template'),
'email_template_two Email template', ('folder_one folder_one_one letter_template_nested '
'letter_template_one Letter template', 'folder_one folder_one_one letter_template_nested '
'letter_template_two Letter template', 'Letter template'),
('folder_one folder_one_two '
'folder_one folder_one_two '
'Empty'),
'folder_two folder_two Empty',
('sms_template_one '
'sms_template_one '
'Text message template'),
('sms_template_two '
'sms_template_two '
'Text message template'),
'email_template_one email_template_one Email template',
'email_template_two email_template_two Email template',
'letter_template_one letter_template_one Letter template',
'letter_template_two letter_template_two Letter template',
], ],
[ [
'folder_one 2 folders', 'folder_one folder_one 2 folders',
'folder_two Empty', 'folder_two folder_two Empty',
'sms_template_one Text message template', 'sms_template_one sms_template_one Text message template',
'sms_template_two Text message template', 'sms_template_two sms_template_two Text message template',
'email_template_one Email template', 'email_template_one email_template_one Email template',
'email_template_two Email template', 'email_template_two email_template_two Email template',
'letter_template_one Letter template', 'letter_template_one letter_template_one Letter template',
'letter_template_two Letter template', 'letter_template_two letter_template_two Letter template',
], ],
[ [
'folder_one', 'folder_one',
@@ -102,29 +116,39 @@ def _folder(name, folder_id=None, parent=None, users_with_permission=None):
{'template_type': 'all'}, {'template_type': 'all'},
['Email', 'Text message', 'Letter'], ['Email', 'Text message', 'Letter'],
[ [
'folder_one 2 folders', 'folder_one folder_one 2 folders',
'folder_one folder_one_one 1 template, 1 folder', ('folder_one folder_one_one '
'folder_one folder_one_one folder_one_one_one 1 template', 'folder_one folder_one_one '
'folder_one folder_one_one folder_one_one_one sms_template_nested Text message template', '1 template, 1 folder'),
'folder_one folder_one_one letter_template_nested Letter template', ('folder_one folder_one_one folder_one_one_one '
'folder_one folder_one_two Empty', 'folder_one folder_one_one folder_one_one_one '
'folder_two Empty', '1 template'),
'sms_template_one Text message template', ('folder_one folder_one_one folder_one_one_one sms_template_nested '
'sms_template_two Text message template', 'folder_one folder_one_one folder_one_one_one sms_template_nested '
'email_template_one Email template', 'Text message template'),
'email_template_two Email template', ('folder_one folder_one_one letter_template_nested '
'letter_template_one Letter template', 'folder_one folder_one_one letter_template_nested '
'letter_template_two Letter template', 'Letter template'),
('folder_one folder_one_two '
'folder_one folder_one_two '
'Empty'),
'folder_two folder_two Empty',
'sms_template_one sms_template_one Text message template',
'sms_template_two sms_template_two Text message template',
'email_template_one email_template_one Email template',
'email_template_two email_template_two Email template',
'letter_template_one letter_template_one Letter template',
'letter_template_two letter_template_two Letter template',
], ],
[ [
'folder_one 2 folders', 'folder_one folder_one 2 folders',
'folder_two Empty', 'folder_two folder_two Empty',
'sms_template_one Text message template', 'sms_template_one sms_template_one Text message template',
'sms_template_two Text message template', 'sms_template_two sms_template_two Text message template',
'email_template_one Email template', 'email_template_one email_template_one Email template',
'email_template_two Email template', 'email_template_two email_template_two Email template',
'letter_template_one Letter template', 'letter_template_one letter_template_one Letter template',
'letter_template_two Letter template', 'letter_template_two letter_template_two Letter template',
], ],
[ [
'folder_one', 'folder_one',
@@ -150,17 +174,23 @@ def _folder(name, folder_id=None, parent=None, users_with_permission=None):
{'template_type': 'sms'}, {'template_type': 'sms'},
['All', 'Email', 'Letter'], ['All', 'Email', 'Letter'],
[ [
'folder_one 1 folder', 'folder_one folder_one 1 folder',
'folder_one folder_one_one 1 folder', ('folder_one folder_one_one '
'folder_one folder_one_one folder_one_one_one 1 template', 'folder_one folder_one_one '
'folder_one folder_one_one folder_one_one_one sms_template_nested Text message template', '1 folder'),
'sms_template_one Text message template', ('folder_one folder_one_one folder_one_one_one '
'sms_template_two Text message template', 'folder_one folder_one_one folder_one_one_one '
'1 template'),
('folder_one folder_one_one folder_one_one_one sms_template_nested '
'folder_one folder_one_one folder_one_one_one sms_template_nested '
'Text message template'),
'sms_template_one sms_template_one Text message template',
'sms_template_two sms_template_two Text message template',
], ],
[ [
'folder_one 1 folder', 'folder_one folder_one 1 folder',
'sms_template_one Text message template', 'sms_template_one sms_template_one Text message template',
'sms_template_two Text message template', 'sms_template_two sms_template_two Text message template',
], ],
[ [
'folder_one', 'folder_one',
@@ -179,15 +209,21 @@ def _folder(name, folder_id=None, parent=None, users_with_permission=None):
{'template_folder_id': PARENT_FOLDER_ID}, {'template_folder_id': PARENT_FOLDER_ID},
['Email', 'Text message', 'Letter'], ['Email', 'Text message', 'Letter'],
[ [
'folder_one_one 1 template, 1 folder', 'folder_one_one folder_one_one 1 template, 1 folder',
'folder_one_one folder_one_one_one 1 template', ('folder_one_one folder_one_one_one '
'folder_one_one folder_one_one_one sms_template_nested Text message template', 'folder_one_one folder_one_one_one '
'folder_one_one letter_template_nested Letter template', '1 template'),
'folder_one_two Empty', ('folder_one_one folder_one_one_one sms_template_nested '
'folder_one_one folder_one_one_one sms_template_nested '
'Text message template'),
('folder_one_one letter_template_nested '
'folder_one_one letter_template_nested '
'Letter template'),
'folder_one_two folder_one_two Empty',
], ],
[ [
'folder_one_one 1 template, 1 folder', 'folder_one_one folder_one_one 1 template, 1 folder',
'folder_one_two Empty', 'folder_one_two folder_one_two Empty',
], ],
[ [
'folder_one_one', 'folder_one_one',
@@ -205,12 +241,16 @@ def _folder(name, folder_id=None, parent=None, users_with_permission=None):
{'template_type': 'sms', 'template_folder_id': PARENT_FOLDER_ID}, {'template_type': 'sms', 'template_folder_id': PARENT_FOLDER_ID},
['All', 'Email', 'Letter'], ['All', 'Email', 'Letter'],
[ [
'folder_one_one 1 folder', 'folder_one_one folder_one_one 1 folder',
'folder_one_one folder_one_one_one 1 template', ('folder_one_one folder_one_one_one '
'folder_one_one folder_one_one_one sms_template_nested Text message template', 'folder_one_one folder_one_one_one '
'1 template'),
('folder_one_one folder_one_one_one sms_template_nested '
'folder_one_one folder_one_one_one sms_template_nested '
'Text message template'),
], ],
[ [
'folder_one_one 1 folder', 'folder_one_one folder_one_one 1 folder',
], ],
[ [
'folder_one_one', 'folder_one_one',
@@ -240,13 +280,15 @@ def _folder(name, folder_id=None, parent=None, users_with_permission=None):
{'template_folder_id': CHILD_FOLDER_ID}, {'template_folder_id': CHILD_FOLDER_ID},
['Email', 'Text message', 'Letter'], ['Email', 'Text message', 'Letter'],
[ [
'folder_one_one_one 1 template', 'folder_one_one_one folder_one_one_one 1 template',
'folder_one_one_one sms_template_nested Text message template', ('folder_one_one_one sms_template_nested '
'letter_template_nested Letter template', 'folder_one_one_one sms_template_nested '
'Text message template'),
'letter_template_nested letter_template_nested Letter template',
], ],
[ [
'folder_one_one_one 1 template', 'folder_one_one_one folder_one_one_one 1 template',
'letter_template_nested Letter template', 'letter_template_nested letter_template_nested Letter template',
], ],
[ [
'folder_one_one_one', 'folder_one_one_one',
@@ -266,10 +308,10 @@ def _folder(name, folder_id=None, parent=None, users_with_permission=None):
{'template_folder_id': GRANDCHILD_FOLDER_ID}, {'template_folder_id': GRANDCHILD_FOLDER_ID},
['Email', 'Text message', 'Letter'], ['Email', 'Text message', 'Letter'],
[ [
'sms_template_nested Text message template', 'sms_template_nested sms_template_nested Text message template',
], ],
[ [
'sms_template_nested Text message template', 'sms_template_nested sms_template_nested Text message template',
], ],
[ [
'sms_template_nested', 'sms_template_nested',
@@ -1462,45 +1504,70 @@ def test_show_custom_error_message(
( (
{}, {},
[ [
['folder_A', '1 template, 2 folders'], ['folder_A', 'folder_A', '1 template, 2 folders'],
['folder_E folder_F folder_G', '1 template'], ['folder_E folder_F folder_G',
['email_template_root', 'Email template'], 'folder_E', 'folder_F', 'folder_G',
'1 template'],
['email_template_root', 'email_template_root', 'Email template'],
], ],
[ [
['folder_A', '1 template, 2 folders'], ['folder_A', 'folder_A', '1 template, 2 folders'],
['folder_A', 'folder_C', '1 template'], ['folder_A folder_C',
['folder_A', 'folder_C', 'sms_template_C', 'Text message template'], 'folder_A', 'folder_C',
['folder_A', 'folder_D', 'Empty'], '1 template'],
['folder_A', 'sms_template_A', 'Text message template'], ['folder_A folder_C sms_template_C',
['folder_E folder_F folder_G', '1 template'], 'folder_A', 'folder_C', 'sms_template_C',
['folder_E folder_F folder_G', 'email_template_G', 'Email template'], 'Text message template'],
['email_template_root', 'Email template'], ['folder_A folder_D',
'folder_A', 'folder_D',
'Empty'],
['folder_A sms_template_A',
'folder_A', 'sms_template_A',
'Text message template'],
['folder_E folder_F folder_G',
'folder_E', 'folder_F', 'folder_G',
'1 template'],
['folder_E folder_F folder_G email_template_G',
'folder_E', 'folder_F', 'folder_G', 'email_template_G',
'Email template'],
['email_template_root', 'email_template_root', 'Email template'],
], ],
None, None,
), ),
( (
{'template_type': 'email'}, {'template_type': 'email'},
[ [
['folder_E folder_F folder_G', '1 template'], ['folder_E folder_F folder_G',
['email_template_root', 'Email template'], 'folder_E', 'folder_F', 'folder_G',
'1 template'],
['email_template_root', 'email_template_root', 'Email template'],
], ],
[ [
['folder_E folder_F folder_G', '1 template'], ['folder_E folder_F folder_G',
['folder_E folder_F folder_G', 'email_template_G', 'Email template'], 'folder_E', 'folder_F', 'folder_G',
['email_template_root', 'Email template'], '1 template'],
['folder_E folder_F folder_G email_template_G',
'folder_E', 'folder_F', 'folder_G', 'email_template_G',
'Email template'],
['email_template_root', 'email_template_root', 'Email template'],
], ],
None, None,
), ),
( (
{'template_type': 'sms'}, {'template_type': 'sms'},
[ [
['folder_A', '1 template, 1 folder'], ['folder_A', 'folder_A', '1 template, 1 folder'],
], ],
[ [
['folder_A', '1 template, 1 folder'], ['folder_A', 'folder_A', '1 template, 1 folder'],
['folder_A', 'folder_C', '1 template'], ['folder_A folder_C',
['folder_A', 'folder_C', 'sms_template_C', 'Text message template'], 'folder_A', 'folder_C',
['folder_A', 'sms_template_A', 'Text message template'], '1 template'],
['folder_A folder_C sms_template_C',
'folder_A', 'folder_C', 'sms_template_C',
'Text message template'],
['folder_A sms_template_A', 'folder_A', 'sms_template_A',
'Text message template'],
], ],
None, None,
), ),

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

@@ -279,7 +279,7 @@ def test_should_show_live_search_if_service_has_lots_of_folders(
) )
count_of_templates_and_folders = len(page.select('#template-list .govuk-label')) count_of_templates_and_folders = len(page.select('#template-list .govuk-label'))
count_of_folders = len(page.select('.template-list-folder:first-child')) count_of_folders = len(page.select('.template-list-folder:first-of-type'))
count_of_templates = count_of_templates_and_folders - count_of_folders count_of_templates = count_of_templates_and_folders - count_of_folders
assert len(page.select('.live-search')) == 1 assert len(page.select('.live-search')) == 1