wagtail 6.4.1__py3-none-any.whl → 7.0rc1__py3-none-any.whl

wagtail/admin/tests/test_paginator.py

@@ -0,0 +1,86 @@
+from django.test import TestCase
+
+from wagtail.admin.paginator import WagtailPaginator
+
+
+class TestWagtailPaginator(TestCase):
+    def test_elided_page_range(self):
+        """
+        Test the get_elided_page_range method of WagtailPaginator.
+
+        For example, if there are 10 pages, the output should be:
+        At page 1:  1 2 3 4 … 10
+        At page 6:  1 … 6 7 … 10
+        At page 10: 1 … 7 8 9 10
+        """
+
+        ellipsis = WagtailPaginator.ELLIPSIS
+
+        test_cases = [
+            # Format: (total pages, current page, num_page_buttons, [expected elided page range])
+            # Examples of test cases that should fail:
+            # (3, 1, 6, [1, 2]), # Too few pages
+            # (10, 1, 6, [1, 2, 3, 4, ellipsis, 9, 10]), # Too many pages
+            # (10, 6, 6, [1, ellipsis, 5, 6, ellipsis, 10]), # Wrong middle position
+            # (10, 10, 6, [1, ellipsis, 7, 8, 9]), # Too few pages
+            # num_page_buttons=4
+            # If the number of page buttons is less than 5, the elided range should be empty.
+            (10, 3, 4, []),
+            # num_page_buttons=5
+            (1, 1, 5, [1]),
+            (3, 1, 5, [1, 2, 3]),
+            (3, 3, 5, [1, 2, 3]),
+            (5, 2, 5, [1, 2, 3, 4, 5]),
+            (10, 1, 5, [1, 2, 3, ellipsis, 10]),
+            (10, 6, 5, [1, ellipsis, 6, ellipsis, 10]),
+            (10, 10, 5, [1, ellipsis, 8, 9, 10]),
+            # Our default number of page buttons.
+            # num_page_buttons=6
+            (1, 1, 6, [1]),
+            (3, 1, 6, [1, 2, 3]),
+            (3, 3, 6, [1, 2, 3]),
+            (6, 2, 6, [1, 2, 3, 4, 5, 6]),
+            (6, 6, 6, [1, 2, 3, 4, 5, 6]),
+            (10, 1, 6, [1, 2, 3, 4, ellipsis, 10]),
+            (10, 6, 6, [1, ellipsis, 6, 7, ellipsis, 10]),
+            (10, 10, 6, [1, ellipsis, 7, 8, 9, 10]),
+            # num_page_buttons=7
+            (7, 2, 7, [1, 2, 3, 4, 5, 6, 7]),
+            (10, 1, 7, [1, 2, 3, 4, 5, ellipsis, 10]),
+            (10, 6, 7, [1, ellipsis, 5, 6, 7, ellipsis, 10]),
+            (10, 10, 7, [1, ellipsis, 6, 7, 8, 9, 10]),
+            # num_page_buttons=8
+            (8, 3, 8, [1, 2, 3, 4, 5, 6, 7, 8]),
+            (10, 1, 8, [1, 2, 3, 4, 5, 6, ellipsis, 10]),
+            (20, 6, 8, [1, ellipsis, 5, 6, 7, 8, ellipsis, 20]),
+            (10, 10, 8, [1, ellipsis, 5, 6, 7, 8, 9, 10]),
+            # num_page_buttons=9
+            (9, 3, 9, [1, 2, 3, 4, 5, 6, 7, 8, 9]),
+            (10, 1, 9, [1, 2, 3, 4, 5, 6, 7, ellipsis, 10]),
+            (10, 6, 9, [1, ellipsis, 4, 5, 6, 7, 8, 9, 10]),
+            (20, 6, 9, [1, ellipsis, 4, 5, 6, 7, 8, ellipsis, 20]),
+            (10, 10, 9, [1, ellipsis, 4, 5, 6, 7, 8, 9, 10]),
+            # num_page_buttons=10
+            (10, 3, 10, [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]),
+            (10, 1, 10, [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]),
+            (10, 6, 10, [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]),
+            (20, 6, 10, [1, ellipsis, 4, 5, 6, 7, 8, 9, ellipsis, 20]),
+            (10, 10, 10, [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]),
+        ]
+
+        for (
+            total_pages,
+            current_page,
+            num_page_buttons,
+            expected_elided_page_range,
+        ) in test_cases:
+            paginator = WagtailPaginator(
+                list(range(total_pages)), 1
+            )  # 1 object per page
+            paginator.num_page_buttons = num_page_buttons
+            elided_page_range = paginator.get_elided_page_range(current_page)
+            self.assertSequenceEqual(
+                list(elided_page_range),
+                expected_elided_page_range,
+                f"Elided page range failed for total_pages={total_pages}, current_page={current_page}, num_page_buttons={num_page_buttons}",
+            )

wagtail/admin/tests/test_privacy.py

@@ -2,6 +2,7 @@ from django.contrib.auth.models import Group
 from django.test import TestCase, override_settings
 from django.urls import reverse
 
+from wagtail.admin.staticfiles import versioned_static
 from wagtail.models import Page, PageViewRestriction
 from wagtail.test.testapp.models import SimplePage
 from wagtail.test.utils import WagtailTestUtils
@@ -120,7 +121,8 @@ class TestSetPrivacyView(WagtailTestUtils, TestCase):
         )
         html = response.json()["html"]
         self.assertIn(
-            f'<span>Privacy is inherited from the ancestor page - <a href="{parent_edit_url }">Private page (simple page)</a></span>',
+            f"<span>Privacy is inherited from the ancestor page - "
+            f'<a href="{parent_edit_url}">Private page (simple page)</a></span>',
             html,
         )
 
@@ -503,9 +505,13 @@ class TestPrivacyIndicators(WagtailTestUtils, TestCase):
 
         soup = self.get_soup(response.content)
 
+        privacy_switch_js = versioned_static("wagtailadmin/js/privacy-switch.js")
+
         public_link = soup.select_one('[data-w-zone-switch-key-value="isPublic"]')
         private_link = soup.select_one('[data-w-zone-switch-key-value="!isPublic"]')
+        scripts = soup.select(f"script[src='{privacy_switch_js}']")
 
+        self.assertEqual(len(scripts), 1)
         # Check the privacy indicator is public
         self.assertEqual(public_link["class"], ["page-status-tag"])
 
@@ -524,8 +530,13 @@ class TestPrivacyIndicators(WagtailTestUtils, TestCase):
 
         soup = self.get_soup(response.content)
 
+        privacy_switch_js = versioned_static("wagtailadmin/js/privacy-switch.js")
+
         public_link = soup.select_one('[data-w-zone-switch-key-value="isPublic"]')
         private_link = soup.select_one('[data-w-zone-switch-key-value="!isPublic"]')
+        scripts = soup.select(f"script[src='{privacy_switch_js}']")
+
+        self.assertEqual(len(scripts), 1)
 
         # Check the privacy indicator is private
         self.assertEqual(private_link["class"], ["page-status-tag"])

wagtail/admin/tests/test_reports_views.py

@@ -33,7 +33,6 @@ from wagtail.test.testapp.models import (
 )
 from wagtail.test.utils import WagtailTestUtils
 from wagtail.test.utils.template_tests import AdminTemplateTestUtils
-from wagtail.utils.deprecation import RemovedInWagtail70Warning
 
 
 class BaseReportViewTestCase(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
@@ -793,13 +792,14 @@ class TestFilteredLogEntriesView(BaseReportViewTestCase):
     def test_decorated_queryset(self):
         # Ensure that decorate_paginated_queryset is only called with the queryset for the current
         # page, instead of all objects over all pages.
-        with mock.patch.object(
-            LogEntriesView,
-            "decorate_paginated_queryset",
-            side_effect=LogEntriesView.decorate_paginated_queryset,
-            autospec=True,
-        ) as decorate_paginated_queryset, mock.patch.object(
-            LogEntriesView, "paginate_by", return_value=1
+        with (
+            mock.patch.object(
+                LogEntriesView,
+                "decorate_paginated_queryset",
+                side_effect=LogEntriesView.decorate_paginated_queryset,
+                autospec=True,
+            ) as decorate_paginated_queryset,
+            mock.patch.object(LogEntriesView, "paginate_by", return_value=1),
         ):
             response = self.get()
             decorate_paginated_queryset.assert_called_once()
@@ -808,41 +808,6 @@ class TestFilteredLogEntriesView(BaseReportViewTestCase):
 
         self.assertEqual(response.status_code, 200)
 
-    def test_deprecated_title_attribute(self):
-        # Remove this test when the deprecation ends
-        with mock.patch.object(
-            LogEntriesView,
-            "page_title",
-            return_value=None,
-            new_callable=mock.PropertyMock,
-        ), mock.patch.object(
-            LogEntriesView,
-            "title",
-            return_value="Deprecated page title",
-            new_callable=mock.PropertyMock,
-        ):
-            with self.assertWarnsMessage(
-                RemovedInWagtail70Warning,
-                "The `title` attribute in `LogEntriesView` (a `ReportView` subclass) is "
-                "deprecated. Use `page_title` instead.",
-            ):
-                self.assertEqual(LogEntriesView.title, "Deprecated page title")
-                self.assertIsNone(LogEntriesView.page_title)
-                response = self.get()
-                self.assertEqual(response.status_code, 200)
-                self.assertPageTitle(
-                    self.get_soup(response.content),
-                    "Deprecated page title - Wagtail",
-                )
-                self.assertEqual(
-                    response.context["page_title"],
-                    "Deprecated page title",
-                )
-                self.assertEqual(
-                    response.context["title"],
-                    "Deprecated page title",
-                )
-
 
 class TestFilteredLogEntriesResultsView(TestFilteredLogEntriesView):
     url_name = "wagtailadmin_reports:site_history_results"

wagtail/admin/tests/test_rich_text.py

@@ -4,6 +4,7 @@ from django.conf import settings
 from django.test import SimpleTestCase, TestCase
 from django.test.utils import override_settings
 from django.urls import reverse, reverse_lazy
+from django.utils import translation
 
 from wagtail.admin.rich_text import DraftailRichTextArea, get_rich_text_editor_widget
 from wagtail.admin.rich_text.converters.editor_html import (
@@ -636,3 +637,16 @@ class TestRichTextChooserUrls(WagtailTestUtils, BaseRichTextEditHandlerTestCase)
         self.assertIn("/admin/images/chooser/", html)
         self.assertIn("/admin/embeds/chooser/", html)
         self.assertIn("/admin/documents/chooser/", html)
+
+
+class TestDraftailLazyTranslations(SimpleTestCase):
+    def test_context_i18n(self):
+        widget = DraftailRichTextArea(features=["h2"])
+        context_default_language = widget.get_context(None, None, {})
+        with translation.override("de"):
+            context_de = widget.get_context(None, None, {})
+        # At least the description of the h2 feature should be different
+        self.assertNotEqual(
+            context_default_language["widget"]["attrs"]["data-w-init-detail-value"],
+            context_de["widget"]["attrs"]["data-w-init-detail-value"],
+        )

wagtail/admin/tests/test_templatetags.py

@@ -1,4 +1,3 @@
-import json
 import os
 import unittest
 from datetime import datetime, timedelta
@@ -12,7 +11,6 @@ from django.test.utils import override_settings
 from django.utils import timezone
 from freezegun import freeze_time
 
-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,
@@ -22,14 +20,12 @@ from wagtail.admin.templatetags.wagtailadmin_tags import (
     timesince_last_update,
     timesince_simple,
 )
-from wagtail.admin.templatetags.wagtailadmin_tags import locales as locales_tag
 from wagtail.coreutils import get_dummy_request
 from wagtail.images.tests.utils import get_test_image_file
 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 TestAvatarUrlInterceptTemplateTag(WagtailTestUtils, TestCase):
@@ -398,24 +394,6 @@ class TestInternationalisationTags(TestCase):
         with override_settings(WAGTAIL_I18N_ENABLED=True):
             self.assertTrue(i18n_enabled())
 
-    def test_locales(self):
-        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),
-            [
-                {"code": "en", "display_name": "English"},
-                {"code": "fr", "display_name": "French"},
-                {"code": "ro", "display_name": "Romanian"},
-                {"code": "ru", "display_name": "Russian"},
-            ],
-        )
-
     def test_locale_label_from_id(self):
         with self.assertNumQueries(1):
             self.assertEqual(locale_label_from_id(self.locale_ids[0]), "English")
@@ -427,20 +405,6 @@ 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):
@@ -1068,3 +1032,53 @@ class PageBreadcrumbsTagTest(AdminTemplateTestUtils, WagtailTestUtils, TestCase)
         self.assertEqual(len(invalid_icons), 0)
         icon = soup.select_one("ol li:last-child svg use[href='#icon-site']")
         self.assertIsNotNone(icon)
+
+
+class ThemeColorSchemeTest(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
+    def setUp(self):
+        self.request = get_dummy_request()
+        self.user = self.login()
+        self.request.user = self.user
+        self.profile = UserProfile.get_for_user(self.user)
+
+    def test_default_mode(self):
+        template = """
+            {% load wagtailadmin_tags %}
+            <meta name="color-scheme" content="{% admin_theme_color_scheme %}">
+        """
+        rendered = Template(template).render(Context({"request": self.request}))
+
+        soup = self.get_soup(rendered)
+        meta_tag = soup.find("meta", {"name": "color-scheme"})
+        self.assertIsNotNone(meta_tag)
+        self.assertEqual(meta_tag["content"], "dark light")
+
+    def test_dark_mode(self):
+        self.profile.theme = "dark"
+        self.profile.save()
+
+        template = """
+            {% load wagtailadmin_tags %}
+            <meta name="color-scheme" content="{% admin_theme_color_scheme %}">
+        """
+        rendered = Template(template).render(Context({"request": self.request}))
+
+        soup = self.get_soup(rendered)
+        meta_tag = soup.find("meta", {"name": "color-scheme"})
+        self.assertIsNotNone(meta_tag)
+        self.assertEqual(meta_tag["content"], "dark")
+
+    def test_light_mode(self):
+        self.profile.theme = "light"
+        self.profile.save()
+
+        template = """
+            {% load wagtailadmin_tags %}
+            <meta name="color-scheme" content="{% admin_theme_color_scheme %}">
+        """
+        rendered = Template(template).render(Context({"request": self.request}))
+
+        soup = self.get_soup(rendered)
+        meta_tag = soup.find("meta", {"name": "color-scheme"})
+        self.assertIsNotNone(meta_tag)
+        self.assertEqual(meta_tag["content"], "light")

wagtail/admin/tests/test_userbar.py

@@ -2,16 +2,20 @@ import json
 
 from django.contrib.auth.models import AnonymousUser, Permission
 from django.template import Context, Template
-from django.test import TestCase
+from django.test import TestCase, override_settings
 from django.urls import reverse
+from django.utils import translation
+from django.utils.translation import gettext
 
 from wagtail import hooks
 from wagtail.admin.userbar import AccessibilityItem
 from wagtail.coreutils import get_dummy_request
-from wagtail.models import PAGE_TEMPLATE_VAR, Page, Site
+from wagtail.models import PAGE_TEMPLATE_VAR, Locale, Page, Site
+from wagtail.test.context_processors import get_call_count, reset_call_count
 from wagtail.test.testapp.models import BusinessChild, BusinessIndex, SimplePage
 from wagtail.test.utils import WagtailTestUtils
-from wagtail.utils.deprecation import RemovedInWagtail70Warning
+from wagtail.users.models import UserProfile
+from wagtail.utils.deprecation import RemovedInWagtail80Warning
 
 
 class TestUserbarTag(WagtailTestUtils, TestCase):
@@ -441,11 +445,13 @@ class TestUserbarInPageServe(WagtailTestUtils, TestCase):
         self.homepage.add_child(instance=self.page)
 
     def test_userbar_rendered(self):
+        reset_call_count()
         response = self.page.serve(self.request)
         response.render()
 
         self.assertEqual(response.status_code, 200)
         self.assertContains(response, '<template id="wagtail-userbar-template">')
+        self.assertEqual(get_call_count(), 1)
 
     def test_userbar_anonymous_user_cannot_see(self):
         self.request.user = AnonymousUser()
@@ -479,18 +485,72 @@ class TestUserbarInPageServe(WagtailTestUtils, TestCase):
             kwargs["called"] = True
             return items
 
-        with self.assertWarnsMessage(
-            RemovedInWagtail70Warning,
-            "`construct_wagtail_userbar` hook functions should accept a `page` argument in third position",
-        ), hooks.register_temporarily(
-            "construct_wagtail_userbar",
-            construct_wagtail_userbar,
+        with (
+            self.assertWarnsMessage(
+                RemovedInWagtail80Warning,
+                "`construct_wagtail_userbar` hook functions should accept a "
+                "`page` argument in third position",
+            ),
+            hooks.register_temporarily(
+                "construct_wagtail_userbar",
+                construct_wagtail_userbar,
+            ),
         ):
             response = self.page.serve(self.request)
             response.render()
 
             self.assertTrue(kwargs.get("called"))
 
+    @override_settings(
+        WAGTAIL_I18N_ENABLED=True,
+        WAGTAIL_CONTENT_LANGUAGES=[("en", "English"), ("fr", "French")],
+        WAGTAILADMIN_PERMITTED_LANGUAGES=[("en", "English")],
+        LANGUAGE_CODE="en",
+    )
+    def test_userbar_rendered_in_admin_permitted_language_if_user_has_no_language(self):
+        french = Locale.objects.create(language_code="fr")
+        self.homepage.copy_for_translation(french)
+        french_page = self.page.copy_for_translation(french)
+
+        with translation.override("fr"):
+            response = french_page.serve(self.request)
+            response.render()
+
+        self.assertContains(response, "Go to Wagtail admin")
+
+    @override_settings(
+        WAGTAIL_I18N_ENABLED=True,
+        WAGTAIL_CONTENT_LANGUAGES=[("en", "English"), ("fr", "French")],
+        LANGUAGE_CODE="en",
+    )
+    def test_userbar_rendered_in_active_language_if_admin_permitted(self):
+        french = Locale.objects.create(language_code="fr")
+        self.homepage.copy_for_translation(french)
+        french_page = self.page.copy_for_translation(french)
+
+        with translation.override("fr"):
+            response = french_page.serve(self.request)
+            response.render()
+
+        with translation.override("fr"):
+            expected_text = gettext("Go to Wagtail admin")
+        self.assertContains(response, expected_text)
+
+    @override_settings(LANGUAGE_CODE="en")
+    def test_userbar_rendered_in_user_preferred_language(self):
+        profile = UserProfile.get_for_user(self.user)
+        profile.preferred_language = "fr"
+        profile.save()
+
+        response = self.page.serve(self.request)
+        response.render()
+
+        # Ensure we have a French string without tests failing if translation changes
+        with translation.override("fr"):
+            expected_text = gettext("Go to Wagtail admin")
+        self.assertNotEqual(expected_text, "Go to Wagtail admin")
+        self.assertContains(response, expected_text)
+
 
 class TestUserbarHooksForChecksPanel(WagtailTestUtils, TestCase):
     def setUp(self):
@@ -522,12 +582,16 @@ class TestUserbarHooksForChecksPanel(WagtailTestUtils, TestCase):
             kwargs["called"] = True
             return items
 
-        with self.assertWarnsMessage(
-            RemovedInWagtail70Warning,
-            "`construct_wagtail_userbar` hook functions should accept a `page` argument in third position",
-        ), hooks.register_temporarily(
-            "construct_wagtail_userbar",
-            construct_wagtail_userbar,
+        with (
+            self.assertWarnsMessage(
+                RemovedInWagtail80Warning,
+                "`construct_wagtail_userbar` hook functions should accept a "
+                "`page` argument in third position",
+            ),
+            hooks.register_temporarily(
+                "construct_wagtail_userbar",
+                construct_wagtail_userbar,
+            ),
         ):
             response = self.client.get(
                 reverse("wagtailadmin_pages:edit", args=(self.homepage.id,))

wagtail/admin/tests/test_views.py

@@ -6,6 +6,7 @@ from wagtail.admin.forms.auth import PasswordResetForm
 from wagtail.admin.tests.test_forms import CustomLoginForm, CustomPasswordResetForm
 from wagtail.models import Page
 from wagtail.test.utils import WagtailTestUtils
+from wagtail.users.models import UserProfile
 
 
 class TestLoginView(WagtailTestUtils, TestCase):
@@ -224,10 +225,24 @@ class TestPasswordResetView(TestCase):
         )
 
 
-class TestJsi18nView(TestCase):
+class TestJsi18nView(WagtailTestUtils, TestCase):
     def test_jsi18n_does_not_require_login(self):
         response = self.client.get(reverse("wagtailadmin_javascript_catalog"))
         self.assertEqual(response.status_code, 200)
         # get content type header without the "charset" suffix
         content_type = response["content-type"].split(";")[0]
         self.assertEqual(content_type, "text/javascript")
+
+    def test_jsi18n_is_localized(self):
+        user = self.login()
+        profile = UserProfile.get_for_user(user)
+        # If a non-default language is activated, the response should contain this:
+        pattern = "const newcatalog = "
+        # If the preferred language is the default language, the catalog should be empty
+        response = self.client.get(reverse("wagtailadmin_javascript_catalog"))
+        self.assertNotContains(response, pattern)
+        # Changing the language should make the catalog appear
+        profile.preferred_language = "de"
+        profile.save()
+        response = self.client.get(reverse("wagtailadmin_javascript_catalog"))
+        self.assertContains(response, pattern)

wagtail/admin/tests/test_widgets.py

@@ -12,6 +12,7 @@ from wagtail.admin.forms.tags import TagField
 from wagtail.models import Page
 from wagtail.test.testapp.forms import AdminStarDateInput
 from wagtail.test.testapp.models import EventPage, RestaurantTag, SimplePage
+from wagtail.utils.deprecation import RemovedInWagtail80Warning
 
 
 class TestAdminPageChooserWidget(TestCase):
@@ -430,9 +431,9 @@ class TestAdminTagWidget(TestCase):
             ],
         )
 
-    @override_settings(TAG_SPACES_ALLOWED=False)
+    @override_settings(WAGTAIL_TAG_SPACES_ALLOWED=False)
     def test_render_js_init_no_spaces_allowed(self):
-        """Checks that the 'w-tag' controller attributes are correctly added to the tag widgets based  on TAG_SPACES_ALLOWED in settings"""
+        """Checks that the 'w-tag' controller attributes are correctly added to the tag widgets based  on WAGTAIL_TAG_SPACES_ALLOWED in settings"""
         widget = widgets.AdminTagWidget()
 
         html = widget.render("tags", None, attrs={"id": "alpha"})
@@ -447,9 +448,9 @@ class TestAdminTagWidget(TestCase):
             ],
         )
 
-    @override_settings(TAG_LIMIT=5)
+    @override_settings(WAGTAIL_TAG_LIMIT=5)
     def test_render_js_init_with_tag_limit(self):
-        """Checks that the 'w-tag' controller attributes are correctly added to the tag widget using options based on TAG_LIMIT in settings"""
+        """Checks that the 'w-tag' controller attributes are correctly added to the tag widget using options based on WAGTAIL_TAG_LIMIT in settings"""
 
         widget = widgets.AdminTagWidget()
 
@@ -517,9 +518,9 @@ class TestAdminTagWidget(TestCase):
             ],
         )
 
-    @override_settings(TAG_SPACES_ALLOWED=True)
+    @override_settings(WAGTAIL_TAG_SPACES_ALLOWED=True)
     def test_tags_help_text_spaces_allowed(self):
-        """Checks that the tags help text html element content is correct when TAG_SPACES_ALLOWED is True"""
+        """Checks that the tags help text html element content is correct when WAGTAIL_TAG_SPACES_ALLOWED is True"""
         widget = widgets.AdminTagWidget()
         help_text = widget.get_context(None, None, {})["widget"]["help_text"]
 
@@ -535,9 +536,9 @@ class TestAdminTagWidget(TestCase):
             html,
         )
 
-    @override_settings(TAG_SPACES_ALLOWED=False)
+    @override_settings(WAGTAIL_TAG_SPACES_ALLOWED=False)
     def test_tags_help_text_no_spaces_allowed(self):
-        """Checks that the tags help text html element content is correct when TAG_SPACES_ALLOWED is False"""
+        """Checks that the tags help text html element content is correct when WAGTAIL_TAG_SPACES_ALLOWED is False"""
         widget = widgets.AdminTagWidget()
         help_text = widget.get_context(None, None, {})["widget"]["help_text"]
 
@@ -552,6 +553,44 @@ class TestAdminTagWidget(TestCase):
             html,
         )
 
+    @override_settings(TAG_LIMIT=3)
+    def test_legacy_tag_limit_setting(self):
+        widget = widgets.AdminTagWidget()
+        with self.assertWarnsMessage(
+            RemovedInWagtail80Warning,
+            "The setting 'TAG_LIMIT' is deprecated. "
+            "Please use 'WAGTAIL_TAG_LIMIT' instead.",
+        ):
+            html = widget.render("tags", None, attrs={"id": "alpha"})
+        params = self.get_js_init_params(html)
+        self.assertEqual(
+            params,
+            [
+                "alpha",
+                "/admin/tag-autocomplete/",
+                {"allowSpaces": True, "tagLimit": 3, "autocompleteOnly": False},
+            ],
+        )
+
+    @override_settings(TAG_SPACES_ALLOWED=False)
+    def test_legacy_tag_spaces_allowed_setting(self):
+        widget = widgets.AdminTagWidget()
+        with self.assertWarnsMessage(
+            RemovedInWagtail80Warning,
+            "The setting 'TAG_SPACES_ALLOWED' is deprecated. "
+            "Please use 'WAGTAIL_TAG_SPACES_ALLOWED' instead.",
+        ):
+            html = widget.render("tags", None, attrs={"id": "alpha"})
+        params = self.get_js_init_params(html)
+        self.assertEqual(
+            params,
+            [
+                "alpha",
+                "/admin/tag-autocomplete/",
+                {"allowSpaces": False, "tagLimit": None, "autocompleteOnly": False},
+            ],
+        )
+
 
 class TestTagField(TestCase):
     def setUp(self):