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

wagtail/admin/tests/test_editing_sessions.py

@@ -282,12 +282,16 @@ class TestPingView(WagtailTestUtils, TestCase):
         self.assertIn("Vic Otheruser saved a new version", session_text)
         self.assertNotIn("Currently viewing", session_text)
         dialog_title = soup.select_one(
-            'template[data-w-teleport-target-value="#title-text-w-overwrite-changes-dialog"]'
+            "template"
+            '[data-w-teleport-target-value="#title-text-w-overwrite-changes-dialog"]'
+            '[data-w-teleport-mode-value="textContent"]'
         )
         self.assertIsNotNone(dialog_title)
         self.assertIn("Vic Otheruser saved a new version", dialog_title.string)
         dialog_subtitle = soup.select_one(
-            'template[data-w-teleport-target-value="#subtitle-w-overwrite-changes-dialog"]'
+            "template"
+            '[data-w-teleport-target-value="#subtitle-w-overwrite-changes-dialog"]'
+            '[data-w-teleport-mode-value="textContent"]'
         )
         self.assertIsNotNone(dialog_subtitle)
         self.assertIn(
@@ -328,6 +332,179 @@ class TestPingView(WagtailTestUtils, TestCase):
             ],
         )
 
+        soup = self.get_soup(response_json["html"])
+        rendered_sessions = soup.select("ol.w-editing-sessions__list li")
+        self.assertEqual(len(rendered_sessions), 1)
+        session_text = rendered_sessions[0].text
+        self.assertIn("Vic Otheruser saved a new version", session_text)
+        self.assertNotIn("Currently viewing", session_text)
+        dialog_title = soup.select_one(
+            "template"
+            '[data-w-teleport-target-value="#title-text-w-overwrite-changes-dialog"]'
+            '[data-w-teleport-mode-value="textContent"]'
+        )
+        self.assertIsNotNone(dialog_title)
+        self.assertIn(
+            "Vic Otheruser saved a new version",
+            dialog_title.string,
+        )
+        dialog_subtitle = soup.select_one(
+            "template"
+            '[data-w-teleport-target-value="#subtitle-w-overwrite-changes-dialog"]'
+            '[data-w-teleport-mode-value="textContent"]'
+        )
+        self.assertIsNotNone(dialog_subtitle)
+        self.assertIn(
+            "Proceeding will overwrite the changes made by Vic Otheruser. "
+            "Refreshing the page will show you the new changes, but you will lose any of your unsaved changes.",
+            dialog_subtitle.string,
+        )
+
+        self.session.refresh_from_db()
+        self.assertEqual(self.session.last_seen_at, TIMESTAMP_NOW)
+        self.assertFalse(self.session.is_editing)
+
+    @freeze_time(TIMESTAMP_NOW)
+    def test_ping_with_overwritten_revision(self):
+        # Simulate other user creating a new revision to be used for autosave
+        with freeze_time(TIMESTAMP_1):
+            loaded_revision = self.page.save_revision(user=self.other_user)
+
+        loaded_timestamp = loaded_revision.created_at.isoformat()
+        response = self.client.post(
+            reverse(
+                "wagtailadmin_editing_sessions:ping",
+                args=("wagtailcore", "page", self.page.id, self.session.id),
+            ),
+            {
+                "revision_id": self.original_revision.id,
+                "revision_created_at": loaded_timestamp,
+            },
+        )
+        self.assertEqual(response.status_code, 200)
+        response_json = response.json()
+        self.assertEqual(response_json["session_id"], self.session.id)
+
+        # no revisions have been saved since the original revision
+        self.assertEqual(
+            response_json["other_sessions"],
+            [
+                {
+                    "session_id": self.other_session.id,
+                    "user": "Vic Otheruser",
+                    "last_seen_at": TIMESTAMP_2.isoformat(),
+                    "is_editing": False,
+                    "revision_id": None,
+                },
+            ],
+        )
+
+        soup = self.get_soup(response_json["html"])
+        rendered_sessions = soup.select("ol.w-editing-sessions__list li")
+        self.assertEqual(len(rendered_sessions), 1)
+        session_text = rendered_sessions[0].text
+        self.assertIn("Vic Otheruser", session_text)
+        self.assertIn("Currently viewing", session_text)
+        self.assertNotIn("saved a new version", session_text)
+
+        self.session.refresh_from_db()
+        self.assertEqual(self.session.last_seen_at, TIMESTAMP_NOW)
+        self.assertFalse(self.session.is_editing)
+
+        # Simulate other user doing an autosave by overwriting the same revision
+        # that we have loaded
+        with freeze_time(TIMESTAMP_3):
+            self.page.save_revision(
+                user=self.other_user,
+                overwrite_revision=loaded_revision,
+            )
+
+        response = self.client.post(
+            reverse(
+                "wagtailadmin_editing_sessions:ping",
+                args=("wagtailcore", "page", self.page.id, self.session.id),
+            ),
+            {
+                "revision_id": self.original_revision.id,
+                "revision_created_at": loaded_timestamp,
+            },
+        )
+        self.assertEqual(response.status_code, 200)
+        response_json = response.json()
+        self.assertEqual(response_json["session_id"], self.session.id)
+
+        # the overwritten revision should be indicated in the response
+        # (and last_seen_at should reflect it), even though it has the same ID
+        # as the loaded revision
+        self.assertEqual(
+            response_json["other_sessions"],
+            [
+                {
+                    "session_id": self.other_session.id,
+                    "user": "Vic Otheruser",
+                    "last_seen_at": TIMESTAMP_3.isoformat(),
+                    "is_editing": False,
+                    "revision_id": loaded_revision.id,
+                },
+            ],
+        )
+
+        soup = self.get_soup(response_json["html"])
+        rendered_sessions = soup.select("ol.w-editing-sessions__list li")
+        self.assertEqual(len(rendered_sessions), 1)
+        session_text = rendered_sessions[0].text
+        self.assertIn("Vic Otheruser saved a new version", session_text)
+        self.assertNotIn("Currently viewing", session_text)
+        dialog_title = soup.select_one(
+            'template[data-w-teleport-target-value="#title-text-w-overwrite-changes-dialog"]'
+        )
+        self.assertIsNotNone(dialog_title)
+        self.assertIn("Vic Otheruser saved a new version", dialog_title.string)
+        dialog_subtitle = soup.select_one(
+            'template[data-w-teleport-target-value="#subtitle-w-overwrite-changes-dialog"]'
+        )
+        self.assertIsNotNone(dialog_subtitle)
+        self.assertIn(
+            "Proceeding will overwrite the changes made by Vic Otheruser. "
+            "Refreshing the page will show you the new changes, but you will lose any of your unsaved changes.",
+            dialog_subtitle.string,
+        )
+
+        self.session.refresh_from_db()
+        self.assertEqual(self.session.last_seen_at, TIMESTAMP_NOW)
+        self.assertFalse(self.session.is_editing)
+
+        self.other_session.delete()
+
+        response = self.client.post(
+            reverse(
+                "wagtailadmin_editing_sessions:ping",
+                args=("wagtailcore", "page", self.page.id, self.session.id),
+            ),
+            {
+                "revision_id": self.original_revision.id,
+                "revision_created_at": loaded_timestamp,
+            },
+        )
+        self.assertEqual(response.status_code, 200)
+        response_json = response.json()
+        self.assertEqual(response_json["session_id"], self.session.id)
+
+        # the overwritten revision should still appear as an "other session" in
+        # the response, even though the editing session record has been deleted
+        self.assertEqual(
+            response_json["other_sessions"],
+            [
+                {
+                    "session_id": None,
+                    "user": "Vic Otheruser",
+                    "last_seen_at": TIMESTAMP_3.isoformat(),
+                    "is_editing": False,
+                    "revision_id": loaded_revision.id,
+                },
+            ],
+        )
+
         soup = self.get_soup(response_json["html"])
         rendered_sessions = soup.select("ol.w-editing-sessions__list li")
         self.assertEqual(len(rendered_sessions), 1)
@@ -417,7 +594,9 @@ class TestPingView(WagtailTestUtils, TestCase):
         self.assertIn("Gordon Thirduser saved a new version", session_text)
         self.assertNotIn("Currently viewing", session_text)
         dialog_title = soup.select_one(
-            'template[data-w-teleport-target-value="#title-text-w-overwrite-changes-dialog"]'
+            "template"
+            '[data-w-teleport-target-value="#title-text-w-overwrite-changes-dialog"]'
+            '[data-w-teleport-mode-value="textContent"]'
         )
         self.assertIsNotNone(dialog_title)
         self.assertIn(
@@ -425,7 +604,9 @@ class TestPingView(WagtailTestUtils, TestCase):
             dialog_title.string,
         )
         dialog_subtitle = soup.select_one(
-            'template[data-w-teleport-target-value="#subtitle-w-overwrite-changes-dialog"]'
+            "template"
+            '[data-w-teleport-target-value="#subtitle-w-overwrite-changes-dialog"]'
+            '[data-w-teleport-mode-value="textContent"]'
         )
         self.assertIsNotNone(dialog_subtitle)
         self.assertIn(
@@ -487,7 +668,9 @@ class TestPingView(WagtailTestUtils, TestCase):
         self.assertIn("System saved a new version", session_text)
         self.assertNotIn("Currently viewing", session_text)
         dialog_title = soup.select_one(
-            'template[data-w-teleport-target-value="#title-text-w-overwrite-changes-dialog"]'
+            "template"
+            '[data-w-teleport-target-value="#title-text-w-overwrite-changes-dialog"]'
+            '[data-w-teleport-mode-value="textContent"]'
         )
         self.assertIsNotNone(dialog_title)
         self.assertIn(
@@ -495,7 +678,9 @@ class TestPingView(WagtailTestUtils, TestCase):
             dialog_title.string,
         )
         dialog_subtitle = soup.select_one(
-            'template[data-w-teleport-target-value="#subtitle-w-overwrite-changes-dialog"]'
+            "template"
+            '[data-w-teleport-target-value="#subtitle-w-overwrite-changes-dialog"]'
+            '[data-w-teleport-mode-value="textContent"]'
         )
         self.assertIsNotNone(dialog_subtitle)
         self.assertIn(
@@ -730,11 +915,15 @@ class TestPingView(WagtailTestUtils, TestCase):
         self.assertIn("You have unsaved changes in another window", session_text)
         self.assertNotIn("Currently viewing", session_text)
         dialog_title = soup.select_one(
-            'template[data-w-teleport-target-value="#title-text-w-overwrite-changes-dialog"]'
+            "template"
+            '[data-w-teleport-target-value="#title-text-w-overwrite-changes-dialog"]'
+            '[data-w-teleport-mode-value="textContent"]'
         )
         self.assertIsNone(dialog_title)
         dialog_subtitle = soup.select_one(
-            'template[data-w-teleport-target-value="#subtitle-w-overwrite-changes-dialog"]'
+            "template"
+            '[data-w-teleport-target-value="#subtitle-w-overwrite-changes-dialog"]'
+            '[data-w-teleport-mode-value="textContent"]'
         )
         self.assertIsNone(dialog_subtitle)
         other_session_text = rendered_sessions[1].text
@@ -937,6 +1126,32 @@ class TestPingView(WagtailTestUtils, TestCase):
             ],
         )
 
+        # The rendered HTML should use special messaging if the newer revision
+        # is made by the same user ("in another window").
+        soup = self.get_soup(response_json["html"])
+        rendered_sessions = soup.select("ol.w-editing-sessions__list li")
+        self.assertEqual(len(rendered_sessions), 2)
+        session_text = rendered_sessions[0].text
+        self.assertIn("You saved a new version in another window", session_text)
+        self.assertNotIn("Currently viewing", session_text)
+        dialog_title = soup.select_one(
+            'template[data-w-teleport-target-value="#title-text-w-overwrite-changes-dialog"]'
+        )
+        self.assertIsNotNone(dialog_title)
+        self.assertIn(
+            "You saved a new version in another window", dialog_title.text.strip()
+        )
+        dialog_subtitle = soup.select_one(
+            'template[data-w-teleport-target-value="#subtitle-w-overwrite-changes-dialog"]'
+        )
+        self.assertIsNotNone(dialog_subtitle)
+        self.assertIn(
+            "Proceeding will overwrite the changes you made in that window. "
+            "Refreshing the page will show you the new changes, but you will "
+            "lose any of your unsaved changes in the current window.",
+            dialog_subtitle.text.strip(),
+        )
+
     @freeze_time(TIMESTAMP_NOW)
     def test_user_must_have_edit_permission_on_page(self):
         # make user a member of Editors
@@ -1331,6 +1546,13 @@ class TestModuleInEditView(WagtailTestUtils, TestCase):
             revision_input.get("value"),
             str(self.object.latest_revision.id),
         )
+        revision_created_at = soup.select_one('input[name="revision_created_at"]')
+        self.assertIsNotNone(revision_created_at)
+        self.assertEqual(revision_created_at.get("type"), "hidden")
+        self.assertEqual(
+            revision_created_at.get("value"),
+            self.object.latest_revision.created_at.isoformat(),
+        )
 
     @freeze_time(TIMESTAMP_NOW)
     def test_edit_view_with_default_interval(self):

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)