wagtail 6.2.1__py3-none-any.whl → 6.3rc1__py3-none-any.whl

wagtail/admin/tests/test_templatetags.py

@@ -11,7 +11,8 @@ from django.test.utils import override_settings
 from django.utils import timezone
 from freezegun import freeze_time
 
-from wagtail.admin.staticfiles import versioned_static
+from wagtail.admin.localization import get_js_translation_strings
+from wagtail.admin.staticfiles import VERSION_HASH, versioned_static
 from wagtail.admin.templatetags.wagtailadmin_tags import (
     avatar_url,
     i18n_enabled,
@@ -27,6 +28,7 @@ from wagtail.models import Locale, Page
 from wagtail.test.utils import WagtailTestUtils
 from wagtail.test.utils.template_tests import AdminTemplateTestUtils
 from wagtail.users.models import UserProfile
+from wagtail.utils.deprecation import RemovedInWagtail70Warning
 
 
 class TestAvatarTemplateTag(WagtailTestUtils, TestCase):
@@ -99,9 +101,12 @@ class TestNotificationStaticTemplateTag(SimpleTestCase):
 
 
 class TestVersionedStatic(SimpleTestCase):
+    def test_version_hash(self):
+        self.assertEqual(len(VERSION_HASH), 8)
+
     def test_versioned_static(self):
         result = versioned_static("wagtailadmin/js/core.js")
-        self.assertRegex(result, r"^/static/wagtailadmin/js/core.js\?v=(\w+)$")
+        self.assertRegex(result, r"^/static/wagtailadmin/js/core.js\?v=(\w{8})$")
 
     @mock.patch("wagtail.admin.staticfiles.static")
     def test_versioned_static_version_string(self, mock_static):
@@ -375,7 +380,12 @@ class TestInternationalisationTags(TestCase):
             self.assertTrue(i18n_enabled())
 
     def test_locales(self):
-        locales_output = locales_tag()
+        with self.assertWarnsMessage(
+            RemovedInWagtail70Warning,
+            "The `locales` template tag will be removed in a future release.",
+        ):
+            locales_output = locales_tag()
+
         self.assertIsInstance(locales_output, str)
         self.assertEqual(
             json.loads(locales_output),
@@ -398,6 +408,20 @@ class TestInternationalisationTags(TestCase):
         with self.assertNumQueries(0):
             self.assertIsNone(locale_label_from_id(self.locale_ids[-1] + 100), None)
 
+    def test_js_translation_strings(self):
+        template = """
+            {% load wagtailadmin_tags %}
+            {% js_translation_strings %}
+        """
+
+        expected = json.dumps(get_js_translation_strings())
+
+        with self.assertWarnsMessage(
+            RemovedInWagtail70Warning,
+            "The `js_translation_strings` template tag will be removed in a future release.",
+        ):
+            self.assertHTMLEqual(expected, Template(template).render(Context()))
+
 
 class ComponentTest(SimpleTestCase):
     def test_render_block_component(self):

wagtail/admin/tests/test_upgrade_notification.py

@@ -1,12 +1,17 @@
 from django.test import RequestFactory, TestCase, override_settings
 
+from wagtail import __version__
 from wagtail.admin.views.home import UpgradeNotificationPanel
 from wagtail.test.utils import WagtailTestUtils
+from wagtail.users.models import UserProfile
 
 
 class TestUpgradeNotificationPanel(WagtailTestUtils, TestCase):
-    DATA_ATTRIBUTE_UPGRADE_CHECK = "data-w-upgrade"
-    DATA_ATTRIBUTE_UPGRADE_CHECK_LTS = "data-w-upgrade-lts-only"
+    ATTR_UPGRADE_CHECK_LTS = "data-w-upgrade-lts-only-value"
+    ATTR_CURRENT_VERSION = "data-w-upgrade-current-version-value"
+    ATTR_DISMISSIBLE_ID = "data-w-dismissible-id-value"
+    ATTR_LAST_DISMISSED_VALUE = "data-w-dismissible-value-param"
+    DISMISSIBLE_ID = "last_upgrade_check"
 
     @classmethod
     def setUpTestData(cls):
@@ -59,9 +64,21 @@ class TestUpgradeNotificationPanel(WagtailTestUtils, TestCase):
         parent_context = {"request": self.request}
 
         result = self.panel.render_html(parent_context)
-
-        self.assertIn(self.DATA_ATTRIBUTE_UPGRADE_CHECK, result)
-        self.assertNotIn(self.DATA_ATTRIBUTE_UPGRADE_CHECK_LTS, result)
+        soup = self.get_soup(result)
+        controller = soup.select_one("[data-controller]")
+        self.assertIsNotNone(controller)
+        self.assertEqual(
+            set(controller["data-controller"].split()),
+            {"w-upgrade", "w-dismissible"},
+        )
+        self.assertFalse(controller.get(self.ATTR_UPGRADE_CHECK_LTS))
+        self.assertEqual(
+            controller.get(self.ATTR_DISMISSIBLE_ID),
+            self.DISMISSIBLE_ID,
+        )
+        toggle = soup.select_one("[data-action='w-dismissible#toggle']")
+        self.assertIsNotNone(toggle)
+        self.assertIsNone(toggle.get(self.ATTR_LAST_DISMISSED_VALUE))
 
     @override_settings(WAGTAIL_ENABLE_UPDATE_CHECK=False)
     def test_render_html_setting_false(self):
@@ -72,22 +89,58 @@ class TestUpgradeNotificationPanel(WagtailTestUtils, TestCase):
 
         self.assertEqual(result, "")
 
-    @override_settings(WAGTAIL_ENABLE_UPDATE_CHECK="LTS")
-    def test_render_html_setting_LTS(self):
+    def test_render_html_setting_lts(self):
         self.request.user = self.superuser
         parent_context = {"request": self.request}
-
-        result = self.panel.render_html(parent_context)
-
-        self.assertIn(self.DATA_ATTRIBUTE_UPGRADE_CHECK, result)
-        self.assertIn(self.DATA_ATTRIBUTE_UPGRADE_CHECK_LTS, result)
-
-    @override_settings(WAGTAIL_ENABLE_UPDATE_CHECK="lts")
-    def test_render_html_setting_lts(self):
+        setting_values = ["lts", "LTS"]
+        for value in setting_values:
+            with self.subTest(setting=value):
+                with override_settings(WAGTAIL_ENABLE_UPDATE_CHECK=value):
+                    result = self.panel.render_html(parent_context)
+
+                soup = self.get_soup(result)
+                controller = soup.select_one("[data-controller]")
+                self.assertIsNotNone(controller)
+                self.assertEqual(
+                    set(controller["data-controller"].split()),
+                    {"w-upgrade", "w-dismissible"},
+                )
+                self.assertEqual(
+                    controller.get(self.ATTR_UPGRADE_CHECK_LTS),
+                    "true",
+                )
+                self.assertEqual(
+                    controller.get(self.ATTR_DISMISSIBLE_ID),
+                    self.DISMISSIBLE_ID,
+                )
+                toggle = soup.select_one("[data-action='w-dismissible#toggle']")
+                self.assertIsNotNone(toggle)
+                self.assertIsNone(toggle.get(self.ATTR_LAST_DISMISSED_VALUE))
+
+    def test_render_html_dismissed_version(self):
+        profile = UserProfile.get_for_user(self.superuser)
+        profile.dismissibles.update({self.DISMISSIBLE_ID: "6.2.2"})
+        profile.save()
         self.request.user = self.superuser
         parent_context = {"request": self.request}
 
         result = self.panel.render_html(parent_context)
-
-        self.assertIn(self.DATA_ATTRIBUTE_UPGRADE_CHECK, result)
-        self.assertIn(self.DATA_ATTRIBUTE_UPGRADE_CHECK_LTS, result)
+        soup = self.get_soup(result)
+
+        controller = soup.select_one("[data-controller='w-upgrade w-dismissible']")
+        self.assertIsNotNone(controller)
+
+        self.assertEqual(
+            controller.get(self.ATTR_DISMISSIBLE_ID),
+            self.DISMISSIBLE_ID,
+        )
+        self.assertEqual(
+            controller.get(self.ATTR_CURRENT_VERSION),
+            __version__,
+        )
+        toggle = soup.select_one("[data-action='w-dismissible#toggle']")
+        self.assertIsNotNone(toggle)
+        self.assertEqual(
+            toggle.get(self.ATTR_LAST_DISMISSED_VALUE),
+            "6.2.2",
+        )

wagtail/admin/tests/test_views.py

@@ -81,7 +81,7 @@ class TestLoginView(WagtailTestUtils, TestCase):
         response = self.client.get(reverse("wagtailadmin_login"))
         self.assertContains(
             response,
-            '<html lang="de" dir="ltr" class="w-theme-system w-density-default">',
+            '<html lang="de" dir="ltr" class="w-theme-system w-density-default w-contrast-system">',
         )
 
     @override_settings(LANGUAGE_CODE="he")
@@ -89,7 +89,7 @@ class TestLoginView(WagtailTestUtils, TestCase):
         response = self.client.get(reverse("wagtailadmin_login"))
         self.assertContains(
             response,
-            '<html lang="he" dir="rtl" class="w-theme-system w-density-default">',
+            '<html lang="he" dir="rtl" class="w-theme-system w-density-default w-contrast-system">',
         )
 
     @override_settings(

wagtail/admin/tests/test_views_generic.py

@@ -24,6 +24,19 @@ class TestGenericIndexView(WagtailTestUtils, TestCase):
         self.assertEqual(h1.text.strip(), "Model with string type primary keys")
 
 
+class TestGenericIndexViewWithoutModel(WagtailTestUtils, TestCase):
+    fixtures = ["test.json"]
+
+    def get(self, params={}):
+        return self.client.get(reverse("testapp_generic_index_without_model"), params)
+
+    def test_non_integer_primary_key(self):
+        response = self.get()
+        self.assertEqual(response.status_code, 200)
+        response_object_count = response.context_data["object_list"].count()
+        self.assertEqual(response_object_count, 3)
+
+
 class TestGenericEditView(WagtailTestUtils, TestCase):
     fixtures = ["test.json"]
 

wagtail/admin/tests/test_widgets.py

@@ -119,7 +119,7 @@ class TestAdminPageChooserWidget(TestCase):
         )
 
         html = widget.render("test", self.child_page, {"id": "test-id"})
-        self.assertIn("Choose a page (Simple Page)", html)
+        self.assertIn("Choose a page (Simple page)", html)
 
     def test_render_with_target_model_as_single_instance(self):
         widget = widgets.AdminPageChooser(target_models=SimplePage)
@@ -131,7 +131,7 @@ class TestAdminPageChooserWidget(TestCase):
         )
 
         html = widget.render("test", self.child_page, {"id": "test-id"})
-        self.assertIn("Choose a page (Simple Page)", html)
+        self.assertIn("Choose a page (Simple page)", html)
 
     def test_render_with_target_model_as_single_string(self):
         widget = widgets.AdminPageChooser(target_models="tests.SimplePage")
@@ -143,7 +143,7 @@ class TestAdminPageChooserWidget(TestCase):
         )
 
         html = widget.render("test", self.child_page, {"id": "test-id"})
-        self.assertIn("Choose a page (Simple Page)", html)
+        self.assertIn("Choose a page (Simple page)", html)
 
     def test_render_with_multiple_target_models(self):
         target_models = [SimplePage, "tests.eventpage"]

wagtail/admin/tests/test_workflows.py

@@ -43,6 +43,7 @@ from wagtail.signals import page_published, published
 from wagtail.test.testapp.models import (
     FullFeaturedSnippet,
     ModeratedModel,
+    MultiPreviewModesPage,
     SimplePage,
     SimpleTask,
 )
@@ -162,6 +163,33 @@ class TestWorkflowsIndexView(AdminTemplateTestUtils, WagtailTestUtils, TestCase)
         self.assertNotContains(response, "There are no enabled workflows.")
         self.assertContains(response, "test_workflow")
 
+    def test_multiple_snippets_assigned_to_workflow(self):
+        Workflow.objects.create(name="Nocontenttypes")
+        multi_ct_workflow = Workflow.objects.create(name="Multicontenttypes")
+        for model in [FullFeaturedSnippet, ModeratedModel]:
+            WorkflowContentType.objects.create(
+                workflow=multi_ct_workflow,
+                content_type=ContentType.objects.get_for_model(model),
+            )
+
+        response = self.get()
+        self.assertEqual(response.status_code, 200)
+        soup = self.get_soup(response.content)
+        cells = [
+            text
+            for td in soup.select("td")
+            if (text := td.get_text(separator=" | ", strip=True))
+        ]
+        self.assertEqual(
+            cells,
+            [
+                "Multicontenttypes",
+                "0 pages | 2 snippet types",
+                "Nocontenttypes",
+                "0 pages | 0 snippet types",
+            ],
+        )
+
     def test_num_queries(self):
         self.create_workflows()
         self.get()
@@ -290,7 +318,15 @@ class TestWorkflowsIndexView(AdminTemplateTestUtils, WagtailTestUtils, TestCase)
         self.assertContains(response, url + "?p=3")
 
         response = self.get({"p": 4})
-        self.assertEqual(response.status_code, 404)
+        # Check response
+        self.assertEqual(response.status_code, 200)
+        self.assertTemplateUsed(response, "wagtailadmin/workflows/index.html")
+
+        # Check that we got the last page
+        self.assertEqual(
+            response.context["page_obj"].number,
+            response.context["paginator"].num_pages,
+        )
 
 
 class TestWorkflowPermissions(WagtailTestUtils, TestCase):
@@ -981,7 +1017,7 @@ class TestTaskIndexView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
         self.assertEqual(response.status_code, 200)
         self.assertTemplateUsed(response, "wagtailadmin/workflows/task_index.html")
         self.assertBreadcrumbsItemsRendered(
-            [{"url": "", "label": "Tasks"}],
+            [{"url": "", "label": "Workflow tasks"}],
             response.content,
         )
 
@@ -1199,7 +1235,15 @@ class TestTaskIndexView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
         self.assertContains(response, url + "?p=3")
 
         response = self.get({"p": 4})
-        self.assertEqual(response.status_code, 404)
+        # Check response
+        self.assertEqual(response.status_code, 200)
+        self.assertTemplateUsed(response, "wagtailadmin/workflows/task_index.html")
+
+        # Check that we got the last page
+        self.assertEqual(
+            response.context["page_obj"].number,
+            response.context["paginator"].num_pages,
+        )
 
     def test_num_queries(self):
         workflows = [Workflow.objects.create(name=f"workflow_{i}") for i in range(7)]
@@ -1877,7 +1921,7 @@ class TestSubmitPageToWorkflow(BasePageWorkflowTests):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save " disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         # submit for approval
@@ -1893,7 +1937,7 @@ class TestSubmitPageToWorkflow(BasePageWorkflowTests):
         self.assertNotContains(response, "Request changes")
         self.assertContains(
             response,
-            '<button type="submit" class="button action-save " disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         # After submit, as a moderator, should only see save, approve, and reject buttons
@@ -1907,7 +1951,7 @@ class TestSubmitPageToWorkflow(BasePageWorkflowTests):
         self.assertContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save " disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         self.reject()
@@ -1923,7 +1967,7 @@ class TestSubmitPageToWorkflow(BasePageWorkflowTests):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save " disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         # After cancel, as a submitter, should only see save and submit buttons
@@ -1936,7 +1980,7 @@ class TestSubmitPageToWorkflow(BasePageWorkflowTests):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save " disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
     def test_workflow_action_menu_items_when_reverting(self):
@@ -1956,7 +2000,7 @@ class TestSubmitPageToWorkflow(BasePageWorkflowTests):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save warning" disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         # submit for approval
@@ -1972,7 +2016,7 @@ class TestSubmitPageToWorkflow(BasePageWorkflowTests):
         self.assertNotContains(response, "Request changes")
         self.assertContains(
             response,
-            '<button type="submit" class="button action-save warning" disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         # After submit, as a moderator, should only see save button
@@ -1986,7 +2030,7 @@ class TestSubmitPageToWorkflow(BasePageWorkflowTests):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save warning" disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         self.reject()
@@ -2002,7 +2046,7 @@ class TestSubmitPageToWorkflow(BasePageWorkflowTests):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save warning" disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         # After cancel, as a submitter, should only see save button
@@ -2016,7 +2060,7 @@ class TestSubmitPageToWorkflow(BasePageWorkflowTests):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save warning" disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
     @override_settings(WAGTAILADMIN_BASE_URL="http://admin.example.com")
@@ -2163,7 +2207,7 @@ class TestSubmitPageToWorkflow(BasePageWorkflowTests):
 
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save " disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
         self.assertNotContains(
             response,
@@ -2228,7 +2272,7 @@ class TestSubmitSnippetToWorkflowNotLockable(TestSubmitSnippetToWorkflow):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save warning" disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         # submit for approval
@@ -2245,7 +2289,7 @@ class TestSubmitSnippetToWorkflowNotLockable(TestSubmitSnippetToWorkflow):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save warning" disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         # After submit, as a moderator, should only see save, approve, and reject buttons
@@ -2259,7 +2303,7 @@ class TestSubmitSnippetToWorkflowNotLockable(TestSubmitSnippetToWorkflow):
         self.assertContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save warning" disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         self.reject()
@@ -2275,7 +2319,7 @@ class TestSubmitSnippetToWorkflowNotLockable(TestSubmitSnippetToWorkflow):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save warning" disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
     def test_workflow_action_menu_items_when_reverting(self):
@@ -2295,7 +2339,7 @@ class TestSubmitSnippetToWorkflowNotLockable(TestSubmitSnippetToWorkflow):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save warning" disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         # submit for approval
@@ -2312,7 +2356,7 @@ class TestSubmitSnippetToWorkflowNotLockable(TestSubmitSnippetToWorkflow):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save warning" disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         # After submit, as a moderator, should only see save button
@@ -2326,7 +2370,7 @@ class TestSubmitSnippetToWorkflowNotLockable(TestSubmitSnippetToWorkflow):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save warning" disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
         self.reject()
@@ -2342,7 +2386,7 @@ class TestSubmitSnippetToWorkflowNotLockable(TestSubmitSnippetToWorkflow):
         self.assertNotContains(response, "Request changes")
         self.assertNotContains(
             response,
-            '<button type="submit" class="button action-save warning" disabled>',
+            '<button type="submit" class="button action-save" disabled>',
         )
 
     def test_submit_for_approval_changes_lock_status(self):
@@ -4117,12 +4161,15 @@ class TestTaskChooserChosenView(WagtailTestUtils, TestCase):
 
 class TestWorkflowUsageView(WagtailTestUtils, TestCase):
     def setUp(self):
-        self.login()
+        self.user = self.login()
         self.workflow = Workflow.objects.get()
 
         self.root_page = Page.objects.get(depth=1)
         self.home_page = Page.objects.get(depth=2)
 
+        self.child_page_with_default_workflow = self.home_page.add_child(
+            instance=SimplePage(title="A page", content="I'm a page")
+        )
         self.child_page_with_another_workflow = self.home_page.add_child(
             instance=SimplePage(title="Another page", content="I'm another page")
         )
@@ -4137,11 +4184,109 @@ class TestWorkflowUsageView(WagtailTestUtils, TestCase):
         )
 
         self.assertEqual(response.status_code, 200)
+        self.assertTemplateUsed(response, "wagtailadmin/pages/listing.html")
+
+        object_set = {page.id for page in response.context["object_list"]}
+        # Should not contain child_page_with_another_workflow. It should also
+        # not contain the root page, as it's irrelevant (you'll never be able to
+        # edit and submit it to the workflow)
+        self.assertEqual(
+            object_set,
+            {self.home_page.id, self.child_page_with_default_workflow.id},
+        )
+
+    def test_with_no_permission(self):
+        group = Group.objects.create(name="test group")
+        self.user.is_superuser = False
+        self.user.save()
+        self.user.groups.add(group)
+        group.permissions.add(
+            Permission.objects.get(
+                content_type__app_label="wagtailadmin", codename="access_admin"
+            )
+        )
+        # No GroupPagePermission created
+
+        response = self.client.get(
+            reverse("wagtailadmin_workflows:usage", args=[self.workflow.id])
+        )
+        self.assertEqual(response.status_code, 302)
+        self.assertRedirects(response, reverse("wagtailadmin_home"))
+
+        # Only a page permission is created, but no workflow permission, not enough
+        permission = GroupPagePermission.objects.create(
+            group=group,
+            page=Page.objects.first(),
+            permission_type="change",
+        )
+
+        response = self.client.get(
+            reverse("wagtailadmin_workflows:usage", args=[self.workflow.id])
+        )
+        self.assertEqual(response.status_code, 302)
+        self.assertRedirects(response, reverse("wagtailadmin_home"))
+
+        # Delete page permission and add workflow permission, also not enough
+        permission.delete()
+        group.permissions.add(
+            Permission.objects.get(
+                content_type__app_label="wagtailcore", codename="change_workflow"
+            )
+        )
+
+        response = self.client.get(
+            reverse("wagtailadmin_workflows:usage", args=[self.workflow.id])
+        )
+        self.assertEqual(response.status_code, 302)
+        self.assertRedirects(response, reverse("wagtailadmin_home"))
+
+    def test_with_minimal_permissions(self):
+        group = Group.objects.create(name="test group")
+        self.user.is_superuser = False
+        self.user.save()
+        self.user.groups.add(group)
+        group.permissions.add(
+            Permission.objects.get(
+                content_type__app_label="wagtailadmin", codename="access_admin"
+            ),
+            Permission.objects.get(
+                content_type__app_label="wagtailcore", codename="change_workflow"
+            ),
+        )
+        GroupPagePermission.objects.create(
+            group=group,
+            page=Page.objects.first(),
+            permission_type="change",
+        )
+
+        # With a workflow permission and a page permission, the user should be
+        # able to access the view
+        response = self.client.get(
+            reverse("wagtailadmin_workflows:usage", args=[self.workflow.id])
+        )
+
+        self.assertEqual(response.status_code, 200)
+
+    def test_get_search_and_filtered_results(self):
+        page_1 = SimplePage(title="Hello wagtail", content="test")
+        page_2 = SimplePage(title="Hello django", content="test")
+        self.home_page.add_child(instance=page_1)
+        self.home_page.add_child(instance=page_2)
+        self.home_page.add_child(instance=MultiPreviewModesPage(title="Hello python"))
+
+        response = self.client.get(
+            reverse("wagtailadmin_workflows:usage_results", args=[self.workflow.id]),
+            {
+                "content_type": ContentType.objects.get_for_model(SimplePage).id,
+                "q": "hello",
+            },
+        )
+
+        self.assertEqual(response.status_code, 200)
+        self.assertTemplateUsed(response, "wagtailadmin/generic/listing_results.html")
 
-        object_set = {page.id for page in response.context["used_by"].object_list}
-        self.assertIn(self.root_page.id, object_set)
-        self.assertIn(self.home_page.id, object_set)
-        self.assertNotIn(self.child_page_with_another_workflow.id, object_set)
+        object_set = {page.id for page in response.context["object_list"]}
+        self.assertEqual(object_set, {page_1.id, page_2.id})
 
 
 @freeze_time("2020-06-01 12:00:00")

wagtail/admin/tests/tests.py

@@ -29,7 +29,7 @@ class TestHome(WagtailTestUtils, TestCase):
     def test_simple(self):
         response = self.client.get(reverse("wagtailadmin_home"))
         self.assertEqual(response.status_code, 200)
-        self.assertContains(response, "Welcome to the Test Site Wagtail CMS")
+        self.assertContains(response, "Test Site")
 
     def test_admin_menu(self):
         response = self.client.get(reverse("wagtailadmin_home"))