From 2b49e5eb7c4e12a16c4da85a9e9ce58933e9b2b4 Mon Sep 17 00:00:00 2001 From: Tom Byers Date: Tue, 9 Jun 2020 16:44:05 +0100 Subject: [PATCH] Make tests work with new template list HTML 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. --- tests/app/main/views/test_template_folders.py | 247 +++++++++++------- tests/app/main/views/test_templates.py | 2 +- 2 files changed, 158 insertions(+), 91 deletions(-) diff --git a/tests/app/main/views/test_template_folders.py b/tests/app/main/views/test_template_folders.py index 91da73454..b70a3cf79 100644 --- a/tests/app/main/views/test_template_folders.py +++ b/tests/app/main/views/test_template_folders.py @@ -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, ), diff --git a/tests/app/main/views/test_templates.py b/tests/app/main/views/test_templates.py index 5ecc1ed70..c9679f293 100644 --- a/tests/app/main/views/test_templates.py +++ b/tests/app/main/views/test_templates.py @@ -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