darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-03-05 01:42:45 -05: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 67c27fff8c
commit 2b49e5eb7c
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'],
[
'folder_one 2 folders',
'folder_one folder_one_one 1 template, 1 folder',
'folder_one folder_one_one folder_one_one_one 1 template',
'folder_one folder_one_one folder_one_one_one sms_template_nested Text message template',
'folder_one folder_one_one letter_template_nested Letter template',
'folder_one folder_one_two Empty',
'folder_two Empty',
'sms_template_one Text message template',
'sms_template_two Text message template',
'email_template_one Email template',
'email_template_two Email template',
'letter_template_one Letter template',
'letter_template_two Letter template',
'folder_one folder_one 2 folders',
('folder_one folder_one_one '
'folder_one folder_one_one '
'1 template, 1 folder'),
('folder_one folder_one_one folder_one_one_one '
'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'),
('folder_one folder_one_one letter_template_nested '
'folder_one folder_one_one letter_template_nested '
'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_two Empty',
'sms_template_one Text message template',
'sms_template_two Text message template',
'email_template_one Email template',
'email_template_two Email template',
'letter_template_one Letter template',
'letter_template_two Letter template',
'folder_one folder_one 2 folders',
'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',
@@ -102,29 +116,39 @@ def _folder(name, folder_id=None, parent=None, users_with_permission=None):
{'template_type': 'all'},
['Email', 'Text message', 'Letter'],
[
'folder_one 2 folders',
'folder_one folder_one_one 1 template, 1 folder',
'folder_one folder_one_one folder_one_one_one 1 template',
'folder_one folder_one_one folder_one_one_one sms_template_nested Text message template',
'folder_one folder_one_one letter_template_nested Letter template',
'folder_one folder_one_two Empty',
'folder_two Empty',
'sms_template_one Text message template',
'sms_template_two Text message template',
'email_template_one Email template',
'email_template_two Email template',
'letter_template_one Letter template',
'letter_template_two Letter template',
'folder_one folder_one 2 folders',
('folder_one folder_one_one '
'folder_one folder_one_one '
'1 template, 1 folder'),
('folder_one folder_one_one folder_one_one_one '
'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'),
('folder_one folder_one_one letter_template_nested '
'folder_one folder_one_one letter_template_nested '
'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_two Empty',
'sms_template_one Text message template',
'sms_template_two Text message template',
'email_template_one Email template',
'email_template_two Email template',
'letter_template_one Letter template',
'letter_template_two Letter template',
'folder_one folder_one 2 folders',
'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',
@@ -150,17 +174,23 @@ def _folder(name, folder_id=None, parent=None, users_with_permission=None):
{'template_type': 'sms'},
['All', 'Email', 'Letter'],
[
'folder_one 1 folder',
'folder_one folder_one_one 1 folder',
'folder_one folder_one_one folder_one_one_one 1 template',
'folder_one folder_one_one folder_one_one_one sms_template_nested Text message template',
'sms_template_one Text message template',
'sms_template_two Text message template',
'folder_one folder_one 1 folder',
('folder_one folder_one_one '
'folder_one folder_one_one '
'1 folder'),
('folder_one folder_one_one folder_one_one_one '
'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',
'sms_template_one Text message template',
'sms_template_two Text message template',
'folder_one folder_one 1 folder',
'sms_template_one sms_template_one Text message template',
'sms_template_two sms_template_two Text message template',
],
[
'folder_one',
@@ -179,15 +209,21 @@ def _folder(name, folder_id=None, parent=None, users_with_permission=None):
{'template_folder_id': PARENT_FOLDER_ID},
['Email', 'Text message', 'Letter'],
[
'folder_one_one 1 template, 1 folder',
'folder_one_one folder_one_one_one 1 template',
'folder_one_one folder_one_one_one sms_template_nested Text message template',
'folder_one_one letter_template_nested Letter template',
'folder_one_two Empty',
'folder_one_one folder_one_one 1 template, 1 folder',
('folder_one_one folder_one_one_one '
'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 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_two Empty',
'folder_one_one folder_one_one 1 template, 1 folder',
'folder_one_two folder_one_two Empty',
],
[
'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},
['All', 'Email', 'Letter'],
[
'folder_one_one 1 folder',
'folder_one_one folder_one_one_one 1 template',
'folder_one_one folder_one_one_one sms_template_nested Text message template',
'folder_one_one folder_one_one 1 folder',
('folder_one_one folder_one_one_one '
'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',
@@ -240,13 +280,15 @@ def _folder(name, folder_id=None, parent=None, users_with_permission=None):
{'template_folder_id': CHILD_FOLDER_ID},
['Email', 'Text message', 'Letter'],
[
'folder_one_one_one 1 template',
'folder_one_one_one sms_template_nested Text message template',
'letter_template_nested Letter template',
'folder_one_one_one folder_one_one_one 1 template',
('folder_one_one_one sms_template_nested '
'folder_one_one_one sms_template_nested '
'Text message template'),
'letter_template_nested letter_template_nested Letter template',
],
[
'folder_one_one_one 1 template',
'letter_template_nested Letter template',
'folder_one_one_one folder_one_one_one 1 template',
'letter_template_nested letter_template_nested Letter template',
],
[
'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},
['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',
@@ -1462,45 +1504,70 @@ def test_show_custom_error_message(
(
{},
[
['folder_A', '1 template, 2 folders'],
['folder_E folder_F folder_G', '1 template'],
['email_template_root', 'Email template'],
['folder_A', 'folder_A', '1 template, 2 folders'],
['folder_E folder_F folder_G',
'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_C', '1 template'],
['folder_A', 'folder_C', 'sms_template_C', 'Text message template'],
['folder_A', 'folder_D', 'Empty'],
['folder_A', 'sms_template_A', 'Text message template'],
['folder_E folder_F folder_G', '1 template'],
['folder_E folder_F folder_G', 'email_template_G', 'Email template'],
['email_template_root', 'Email template'],
['folder_A', 'folder_A', '1 template, 2 folders'],
['folder_A folder_C',
'folder_A', 'folder_C',
'1 template'],
['folder_A folder_C sms_template_C',
'folder_A', 'folder_C', 'sms_template_C',
'Text message 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,
),
(
{'template_type': 'email'},
[
['folder_E folder_F folder_G', '1 template'],
['email_template_root', 'Email template'],
['folder_E folder_F folder_G',
'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', 'email_template_G', 'Email template'],
['email_template_root', 'Email 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,
),
(
{'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_C', '1 template'],
['folder_A', 'folder_C', 'sms_template_C', 'Text message template'],
['folder_A', 'sms_template_A', 'Text message template'],
['folder_A', 'folder_A', '1 template, 1 folder'],
['folder_A folder_C',
'folder_A', 'folder_C',
'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,
),

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

@@ -258,7 +258,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_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
assert len(page.select('.live-search')) == 1