wagtail 7.2.1__py3-none-any.whl → 7.3rc1__py3-none-any.whl

wagtail/admin/tests/test_forms.py

@@ -3,7 +3,10 @@ from django.test import SimpleTestCase, TestCase
 
 from wagtail.admin.forms.auth import LoginForm, PasswordResetForm
 from wagtail.admin.forms.models import WagtailAdminModelForm
-from wagtail.test.testapp.models import Advert
+from wagtail.models import Page
+from wagtail.test.snippets.models import MultiSectionRichTextSnippet
+from wagtail.test.testapp.models import Advert, EventPage
+from wagtail.test.utils.form_data import inline_formset, nested_form_data, rich_text
 
 
 class CustomLoginForm(LoginForm):
@@ -66,3 +69,191 @@ class TestDeferRequiredFields(TestCase):
         form.defer_required_fields()
         form.restore_required_fields()
         self.assertFalse(form.is_valid())
+
+
+class SnippetForm(WagtailAdminModelForm):
+    class Meta:
+        model = MultiSectionRichTextSnippet
+        fields = []
+        formsets = {
+            "sections": {
+                "fields": ["body"],
+            }
+        }
+
+
+class TestGetFieldUpdatesForResave(TestCase):
+    def test_get_field_updates_for_resave_on_create(self):
+        form = SnippetForm(
+            nested_form_data(
+                {
+                    "sections": inline_formset(
+                        [
+                            {"id": "", "body": rich_text("<p>Section 1 body</p>")},
+                            {"id": "", "body": rich_text("<p>Section 2 body</p>")},
+                        ]
+                    )
+                }
+            )
+        )
+        self.assertTrue(form.is_valid())
+        snippet = form.save()
+        section_1 = snippet.sections.get(body__contains="Section 1 body")
+        section_2 = snippet.sections.get(body__contains="Section 2 body")
+        self.assertCountEqual(
+            form.get_field_updates_for_resave(),
+            [
+                ("sections-INITIAL_FORMS", "2"),
+                ("sections-0-id", str(section_1.id)),
+                ("sections-1-id", str(section_2.id)),
+            ],
+        )
+
+    def test_get_field_updates_for_resave_on_update(self):
+        snippet = MultiSectionRichTextSnippet()
+        section_1 = snippet.sections.create(body="<p>Initial body 1</p>")
+        snippet.save()
+
+        form = SnippetForm(
+            nested_form_data(
+                {
+                    "sections": inline_formset(
+                        [
+                            {
+                                "id": str(section_1.id),
+                                "body": rich_text("<p>Section 1 body</p>"),
+                            },
+                            {"id": "", "body": rich_text("<p>Section 2 body</p>")},
+                        ],
+                        initial=1,
+                    )
+                }
+            ),
+            instance=snippet,
+        )
+        self.assertTrue(form.is_valid())
+        form.save()
+        section_2 = snippet.sections.get(body__contains="Section 2 body")
+        self.assertCountEqual(
+            form.get_field_updates_for_resave(),
+            [
+                ("sections-INITIAL_FORMS", "2"),
+                ("sections-1-id", str(section_2.id)),
+            ],
+        )
+
+    def test_get_field_updates_for_resave_on_update_when_deleting_child(self):
+        snippet = MultiSectionRichTextSnippet()
+        section_1 = snippet.sections.create(body="<p>Initial body 1</p>")
+        section_2 = snippet.sections.create(body="<p>Initial body 2</p>")
+        snippet.save()
+
+        form = SnippetForm(
+            nested_form_data(
+                {
+                    "sections": inline_formset(
+                        [
+                            {
+                                "id": str(section_1.id),
+                                "body": rich_text("<p>Section 1 body</p>"),
+                                "DELETE": "on",
+                            },
+                            {
+                                "id": str(section_2.id),
+                                "body": rich_text("<p>Section 2 body</p>"),
+                            },
+                            {
+                                # created and immediately deleted - should not appear in updates
+                                "id": "",
+                                "body": rich_text("<p>Section 3 body</p>"),
+                                "DELETE": "on",
+                            },
+                        ],
+                        initial=2,
+                    )
+                }
+            ),
+            instance=snippet,
+        )
+        self.assertTrue(form.is_valid())
+        form.save()
+        self.assertCountEqual(
+            form.get_field_updates_for_resave(),
+            [
+                ("sections-INITIAL_FORMS", "3"),
+                ("sections-0-id", ""),
+            ],
+        )
+
+    def test_get_field_updates_for_resave_with_nested_inline_panel(self):
+        event_page = EventPage(
+            title="Test event",
+            date_from="2024-01-01",
+            audience="public",
+            location="Test location",
+            cost="Free",
+        )
+        speaker = event_page.speakers.create(first_name="First", last_name="Last")
+        award_1 = speaker.awards.create(name="Award 1")
+        root_page = Page.objects.filter(depth=2).first()
+        root_page.add_child(instance=event_page)
+
+        form = event_page.get_edit_handler().get_form_class()(
+            nested_form_data(
+                {
+                    "title": "Test event",
+                    "date_from": "2024-01-01",
+                    "audience": "public",
+                    "location": "Test location",
+                    "cost": "Free",
+                    "slug": "test-event",
+                    "speakers": inline_formset(
+                        [
+                            {
+                                "id": str(speaker.id),
+                                "first_name": "First",
+                                "last_name": "Last",
+                                "awards": inline_formset(
+                                    [
+                                        {"id": str(award_1.id), "name": "Award 1"},
+                                        {"id": "", "name": "Award 2"},
+                                    ],
+                                    initial=1,
+                                ),
+                            },
+                            {
+                                "id": "",
+                                "first_name": "New",
+                                "last_name": "Speaker",
+                                "awards": inline_formset(
+                                    [
+                                        {"id": "", "name": "New Award"},
+                                    ]
+                                ),
+                            },
+                        ],
+                        initial=1,
+                    ),
+                    "carousel_items": inline_formset([]),
+                    "related_links": inline_formset([]),
+                    "head_counts": inline_formset([]),
+                },
+            ),
+            instance=event_page,
+        )
+        self.assertTrue(form.is_valid())
+        form.save()
+        award_2 = speaker.awards.get(name="Award 2")
+        speaker_2 = event_page.speakers.get(first_name="New", last_name="Speaker")
+        award_3 = speaker_2.awards.get(name="New Award")
+        self.assertCountEqual(
+            form.get_field_updates_for_resave(),
+            [
+                ("speakers-INITIAL_FORMS", "2"),
+                ("speakers-0-awards-INITIAL_FORMS", "2"),
+                ("speakers-0-awards-1-id", str(award_2.id)),
+                ("speakers-1-id", str(speaker_2.id)),
+                ("speakers-1-awards-INITIAL_FORMS", "1"),
+                ("speakers-1-awards-0-id", str(award_3.id)),
+            ],
+        )

wagtail/admin/tests/test_icon_sprite.py

@@ -1,9 +1,10 @@
 from django.test import SimpleTestCase
 
-from wagtail.admin.icons import get_icon_sprite_hash, get_icon_sprite_url
+from wagtail.admin.icons import get_icon_sprite_hash, get_icon_sprite_url, get_icons
+from wagtail.test.utils.wagtail_tests import WagtailTestUtils
 
 
-class TestIconSpriteView(SimpleTestCase):
+class TestIconSpriteView(WagtailTestUtils, SimpleTestCase):
     def test_content_type(self):
         response = self.client.get(get_icon_sprite_url())
         self.assertEqual(
@@ -15,6 +16,25 @@ class TestIconSpriteView(SimpleTestCase):
         response = self.client.get(get_icon_sprite_url())
         self.assertNotContains(response, "<!--")
 
+    def test_register_icons_hook(self):
+        get_icons.cache_clear()
+        self.addCleanup(get_icons.cache_clear)
+
+        def register_icons(icons):
+            # The hook should receive the existing icons as an argument
+            self.assertIn("wagtailadmin/icons/wagtail.svg", icons)
+            return icons + [
+                "tests/icons/single-quotes.svg",  # id='icon-single-quotes'
+            ]
+
+        with self.register_hook("register_icons", register_icons):
+            response = self.client.get(get_icon_sprite_url())
+
+        self.assertEqual(response.status_code, 200)
+        soup = self.get_soup(response.content)
+        icon = soup.select_one("symbol#icon-single-quotes")
+        self.assertIsNotNone(icon)
+
 
 class TestIconSpriteHash(SimpleTestCase):
     def test_hash(self):

wagtail/admin/tests/test_page_chooser.py

@@ -28,7 +28,7 @@ class TestChooserBrowse(WagtailTestUtils, TestCase):
 
         self.login()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailadmin_choose_page"), params)
 
     def test_simple(self):
@@ -103,7 +103,7 @@ class TestCanChooseRootFlag(WagtailTestUtils, TestCase):
     def setUp(self):
         self.login()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailadmin_choose_page"), params)
 
     def test_cannot_choose_root_by_default(self):
@@ -125,12 +125,12 @@ class TestChooserBrowseChild(WagtailTestUtils, TestCase):
 
         self.login()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(
             reverse("wagtailadmin_choose_page_child", args=(self.root_page.id,)), params
         )
 
-    def get_invalid(self, params={}):
+    def get_invalid(self, params=None):
         return self.client.get(
             reverse("wagtailadmin_choose_page_child", args=(9999999,)), params
         )
@@ -574,7 +574,7 @@ class TestAutomaticRootPageDetection(WagtailTestUtils, TestCase):
         )
         return event_index
 
-    def get_best_root(self, params={}):
+    def get_best_root(self, params=None):
         response = self.client.get(reverse("wagtailadmin_choose_page"), params)
         return response.context["parent_page"].specific
 
@@ -629,12 +629,12 @@ class TestChooserExternalLink(WagtailTestUtils, TestCase):
         self.internal_page = SimplePage(title="About", content="About Foo")
         Page.objects.get(pk=2).add_child(instance=self.internal_page)
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(
             reverse("wagtailadmin_choose_page_external_link"), params
         )
 
-    def post(self, post_data={}, url_params={}):
+    def post(self, post_data=None, url_params=None):
         url = reverse("wagtailadmin_choose_page_external_link")
         if url_params:
             url += "?" + urlencode(url_params)
@@ -1141,10 +1141,10 @@ class TestChooserAnchorLink(WagtailTestUtils, TestCase):
     def setUp(self):
         self.login()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailadmin_choose_page_anchor_link"), params)
 
-    def post(self, post_data={}, url_params={}):
+    def post(self, post_data=None, url_params=None):
         url = reverse("wagtailadmin_choose_page_anchor_link")
         if url_params:
             url += "?" + urlencode(url_params)
@@ -1224,10 +1224,10 @@ class TestChooserEmailLink(WagtailTestUtils, TestCase):
     def setUp(self):
         self.login()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailadmin_choose_page_email_link"), params)
 
-    def post(self, post_data={}, url_params={}):
+    def post(self, post_data=None, url_params=None):
         url = reverse("wagtailadmin_choose_page_email_link")
         if url_params:
             url += "?" + urlencode(url_params)
@@ -1380,10 +1380,10 @@ class TestChooserPhoneLink(WagtailTestUtils, TestCase):
     def setUp(self):
         self.login()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailadmin_choose_page_phone_link"), params)
 
-    def post(self, post_data={}, url_params={}):
+    def post(self, post_data=None, url_params=None):
         url = reverse("wagtailadmin_choose_page_phone_link")
         if url_params:
             url += "?" + urlencode(url_params)

wagtail/admin/tests/test_reports_views.py

@@ -50,17 +50,20 @@ class BaseReportViewTestCase(AdminTemplateTestUtils, WagtailTestUtils, TestCase)
             cls.header_buttons_parent_selector = (
                 '[data-controller="w-teleport"]'
                 '[data-w-teleport-target-value="#w-slim-header-buttons"]'
+                '[data-w-teleport-mode-value="innerHTML"]'
             )
             cls.drilldown_selector = (
                 '[data-controller="w-teleport"]'
                 '[data-w-teleport-target-value="#filters-drilldown"]'
+                '[data-w-teleport-mode-value="innerHTML"]'
             )
             cls.extra_params = "&_w_filter_fragment=true"
 
     def setUp(self):
         self.user = self.login()
 
-    def get(self, params={}, **kwargs):
+    def get(self, params=None, **kwargs):
+        params = params or {}
         if self.results_only:
             params["_w_filter_fragment"] = "true"
         return self.client.get(self.url, params, **kwargs)

wagtail/admin/tests/test_userbar.py

@@ -9,6 +9,7 @@ from django.utils.translation import gettext
 
 from wagtail import hooks
 from wagtail.admin.staticfiles import versioned_static
+from wagtail.admin.ui.components import Component
 from wagtail.admin.userbar import AccessibilityItem, Userbar
 from wagtail.coreutils import get_dummy_request
 from wagtail.models import PAGE_TEMPLATE_VAR, Locale, Page, Site
@@ -761,7 +762,7 @@ class TestUserbarComponent(WagtailTestUtils, TestCase):
         self.assertIsNotNone(button)
         self.assertEqual(
             button.get_text(separator=" | ", strip=True).strip(),
-            "Issues found | Accessibility",
+            "Issues found | Checks",
         )
 
     def test_render_no_request(self):
@@ -790,7 +791,7 @@ class TestUserbarComponent(WagtailTestUtils, TestCase):
         self.assertIsNotNone(button)
         self.assertEqual(
             button.get_text(separator=" | ", strip=True).strip(),
-            "Issues found | Accessibility",
+            "Issues found | Checks",
         )
 
         css_links = soup.select("link[rel='stylesheet']")
@@ -842,7 +843,7 @@ class TestUserbarComponent(WagtailTestUtils, TestCase):
         self.assertIsNotNone(button)
         self.assertEqual(
             button.get_text(separator=" | ", strip=True).strip(),
-            "Issues found | Accessibility",
+            "Issues found | Checks",
         )
 
     def test_render_with_page(self):
@@ -868,7 +869,7 @@ class TestUserbarComponent(WagtailTestUtils, TestCase):
         self.assertIsNotNone(accessibility_button)
         self.assertEqual(
             accessibility_button.get_text(separator=" | ", strip=True).strip(),
-            "Issues found | Accessibility",
+            "Issues found | Checks",
         )
 
     @override_settings(WAGTAILADMIN_BASE_URL=None)
@@ -899,7 +900,7 @@ class TestUserbarComponent(WagtailTestUtils, TestCase):
         self.assertIsNotNone(button)
         self.assertEqual(
             button.get_text(separator=" | ", strip=True).strip(),
-            "Issues found | Accessibility",
+            "Issues found | Checks",
         )
 
         css_links = soup.select("link[rel='stylesheet']")
@@ -923,3 +924,66 @@ class TestUserbarComponent(WagtailTestUtils, TestCase):
                 versioned_static("wagtailadmin/js/userbar.js"),
             ],
         )
+
+    def test_component_item(self):
+        class TestItem(Component):
+            def render_html(self, parent_context):
+                return "<li><a href='#test-item'>Test item</a></li>"
+
+        def hook(request, items, page):
+            items.append(TestItem())
+
+        with hooks.register_temporarily("construct_wagtail_userbar", hook):
+            rendered = Userbar(object=self.homepage).render_html(
+                {"request": self.request, PAGE_TEMPLATE_VAR: self.homepage}
+            )
+            soup = self.get_soup(rendered)
+
+        self.assertIsNotNone(soup.css.select_one("a[href='#test-item']"))
+
+    def test_legacy_render_item(self):
+        class LegacyItem:
+            def render(self, request):
+                return "<li><a href='#legacy-item'>Legacy item</a></li>"
+
+        def hook(request, items, page):
+            items.append(LegacyItem())
+
+        with (
+            self.assertWarnsMessage(
+                RemovedInWagtail80Warning,
+                "Userbar items now use the `render_html(parent_context)` method instead of `render(request)` - "
+                "view the `construct_wagtail_userbar` docs to update LegacyItem",
+            ),
+            hooks.register_temporarily(
+                "construct_wagtail_userbar",
+                hook,
+            ),
+        ):
+            rendered = Userbar(object=self.homepage).render_html(
+                {"request": self.request, PAGE_TEMPLATE_VAR: self.homepage}
+            )
+            soup = self.get_soup(rendered)
+
+        self.assertIsNotNone(soup.css.select_one("a[href='#legacy-item']"))
+
+    def test_component_item_with_media(self):
+        class ItemWithMedia(Component):
+            def render_html(self, parent_context):
+                return ""
+
+            class Media:
+                js = ["custom-item.js"]
+                css = {"all": ["custom-item.css"]}
+
+        def hook(request, items, page):
+            items.append(ItemWithMedia())
+
+        with hooks.register_temporarily("construct_wagtail_userbar", hook):
+            rendered = Userbar(object=self.homepage).render_html(
+                {"request": self.request, PAGE_TEMPLATE_VAR: self.homepage}
+            )
+            soup = self.get_soup(rendered)
+
+        self.assertIsNotNone(soup.css.select_one('script[src$="custom-item.js"]'))
+        self.assertIsNotNone(soup.css.select_one('link[href$="custom-item.css"]'))

wagtail/admin/tests/test_views_generic.py

@@ -9,7 +9,7 @@ from wagtail.test.utils import WagtailTestUtils
 class TestGenericIndexView(WagtailTestUtils, TestCase):
     fixtures = ["test.json"]
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("testapp_generic_index"), params)
 
     def test_non_integer_primary_key(self):
@@ -28,7 +28,7 @@ class TestGenericIndexView(WagtailTestUtils, TestCase):
 class TestGenericIndexViewWithoutModel(WagtailTestUtils, TestCase):
     fixtures = ["test.json"]
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("testapp_generic_index_without_model"), params)
 
     def test_non_integer_primary_key(self):
@@ -41,7 +41,7 @@ class TestGenericIndexViewWithoutModel(WagtailTestUtils, TestCase):
 class TestGenericCreateView(WagtailTestUtils, TestCase):
     fixtures = ["test.json"]
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("testapp_generic_create"), params)
 
     def test_get_create_view(self):
@@ -74,7 +74,7 @@ class TestGenericCreateView(WagtailTestUtils, TestCase):
 class TestGenericEditView(WagtailTestUtils, TestCase):
     fixtures = ["test.json"]
 
-    def get(self, object_pk, params={}):
+    def get(self, object_pk, params=None):
         return self.client.get(
             reverse("testapp_generic_edit", args=(object_pk,)), params
         )
@@ -120,7 +120,7 @@ class TestGenericEditView(WagtailTestUtils, TestCase):
 class TestGenericDeleteView(WagtailTestUtils, TestCase):
     fixtures = ["test.json"]
 
-    def get(self, object_pk, params={}):
+    def get(self, object_pk, params=None):
         return self.client.get(
             reverse("testapp_generic_edit", args=(object_pk,)), params
         )

wagtail/admin/tests/test_workflows.py

@@ -143,7 +143,7 @@ class TestWorkflowsIndexView(AdminTemplateTestUtils, WagtailTestUtils, TestCase)
         ]
         WorkflowPage.objects.bulk_create(workflow_pages)
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailadmin_workflows:index"), params)
 
     def test_simple(self):
@@ -339,7 +339,7 @@ class TestWorkflowPermissions(WagtailTestUtils, TestCase):
     def setUp(self):
         self.user = self.login()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse(self.url_name), params)
 
     def test_simple(self):
@@ -417,10 +417,10 @@ class TestWorkflowsCreateView(AdminTemplateTestUtils, WagtailTestUtils, TestCase
             FullFeaturedSnippet
         )
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailadmin_workflows:add"), params)
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(reverse("wagtailadmin_workflows:add"), post_data)
 
     def test_get(self):
@@ -676,12 +676,12 @@ class TestWorkflowsEditView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
         moderators.user_set.add(self.moderator)
         moderators.permissions.add(Permission.objects.get(codename="change_workflow"))
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(
             reverse("wagtailadmin_workflows:edit", args=[self.workflow.id]), params
         )
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(
             reverse("wagtailadmin_workflows:edit", args=[self.workflow.id]), post_data
         )
@@ -1028,7 +1028,7 @@ class TestRemoveWorkflow(WagtailTestUtils, TestCase):
         moderators.user_set.add(self.moderator)
         moderators.permissions.add(Permission.objects.get(codename="change_workflow"))
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(
             reverse(
                 "wagtailadmin_workflows:remove", args=[self.page.id, self.workflow.id]
@@ -1082,7 +1082,7 @@ class TestTaskIndexView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
         moderators.user_set.add(self.moderator)
         moderators.permissions.add(Permission.objects.get(codename="change_task"))
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailadmin_workflows:task_index"), params)
 
     def test_simple(self):
@@ -1370,7 +1370,7 @@ class TestCreateTaskView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
         moderators.user_set.add(self.moderator)
         moderators.permissions.add(Permission.objects.get(codename="add_task"))
 
-    def get(self, url_kwargs=None, params={}):
+    def get(self, url_kwargs=None, params=None):
         url_kwargs = url_kwargs or {}
         url_kwargs.setdefault("app_label", SimpleTask._meta.app_label)
         url_kwargs.setdefault("model_name", SimpleTask._meta.model_name)
@@ -1378,7 +1378,7 @@ class TestCreateTaskView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
             reverse("wagtailadmin_workflows:add_task", kwargs=url_kwargs), params
         )
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(
             reverse(
                 "wagtailadmin_workflows:add_task",
@@ -1504,12 +1504,12 @@ class TestEditTaskView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
         moderators.user_set.add(self.moderator)
         moderators.permissions.add(Permission.objects.get(codename="change_task"))
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(
             reverse("wagtailadmin_workflows:edit_task", args=[self.task.id]), params
         )
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(
             reverse("wagtailadmin_workflows:edit_task", args=[self.task.id]), post_data
         )
@@ -3434,10 +3434,12 @@ class TestPageWorkflowReportResults(TestPageWorkflowReport):
     header_buttons_parent_selector = (
         '[data-controller="w-teleport"]'
         '[data-w-teleport-target-value="#w-slim-header-buttons"]'
+        '[data-w-teleport-mode-value="innerHTML"]'
     )
     drilldown_selector = (
         '[data-controller="w-teleport"]'
         '[data-w-teleport-target-value="#filters-drilldown"]'
+        '[data-w-teleport-mode-value="innerHTML"]'
     )
     extra_params = "&_w_filter_fragment=true"
 

wagtail/admin/tests/viewsets/test_model_viewset.py

@@ -91,6 +91,19 @@ class TestModelViewSetGroup(WagtailTestUtils, TestCase):
         self.assertNotContains(response, reverse("blockcounts_streammodel:index"))
 
 
+class TestAdminURLs(WagtailTestUtils, TestCase):
+    def setUp(self):
+        self.user = self.login()
+
+    def test_cannot_reverse_mismatched_converter_value(self):
+        with self.assertRaises(NoReverseMatch):
+            reverse("streammodel:edit", kwargs={"pk": "123abc"})
+
+    def test_404_on_mismatched_converter_value(self):
+        response = self.client.get("/admin/streammodel/edit/123abc/")
+        self.assertEqual(response.status_code, 404)
+
+
 class TestTemplateConfiguration(WagtailTestUtils, TestCase):
     def setUp(self):
         self.user = self.login()

wagtail/admin/ui/autosave.py

@@ -0,0 +1,5 @@
+from wagtail.admin.ui.components import Component
+
+
+class AutosaveIndicator(Component):
+    template_name = "wagtailadmin/shared/autosave/indicator.html"

wagtail/admin/ui/editing_sessions.py

@@ -13,12 +13,14 @@ class EditingSessionsModule(Component):
         release_url,
         other_sessions,
         revision_id=None,
+        revision_created_at=None,
     ):
         self.current_session = current_session
         self.ping_url = ping_url
         self.release_url = release_url
         self.sessions_list = EditingSessionsList(current_session, other_sessions)
         self.revision_id = revision_id
+        self.revision_created_at = revision_created_at
 
     def get_context_data(self, parent_context):
         ping_interval = getattr(
@@ -33,6 +35,7 @@ class EditingSessionsModule(Component):
             "ping_interval": ping_interval,
             "sessions_list": self.sessions_list,
             "revision_id": self.revision_id,
+            "revision_created_at": self.revision_created_at,
         }