',
+ )
def assert_fieldline_hidden(self, response):
- self.assertContains(response, '
',
+ )
@override_settings(ROOT_URLCONF="admin_views.urls")
diff --git a/tests/admin_widgets/tests.py b/tests/admin_widgets/tests.py
index e0ae5b77471c..5a8b29b83a1f 100644
--- a/tests/admin_widgets/tests.py
+++ b/tests/admin_widgets/tests.py
@@ -1203,6 +1203,41 @@ def test_calendar_show_date_from_input(self):
# The right month and year are displayed.
self.wait_for_text("#calendarin0 caption", expected_caption)
+ @override_settings(TIME_ZONE="Asia/Seoul")
+ def test_timezone_warning_message(self):
+ from selenium.webdriver.common.by import By
+
+ self.admin_login(username="super", password="secret", login_url="/")
+
+ self.selenium.get(
+ self.live_server_url + reverse("admin:admin_widgets_member_add")
+ )
+
+ datetime = self.selenium.find_element(By.CSS_SELECTOR, "p.datetime")
+ warnings = self.selenium.find_elements(
+ By.CSS_SELECTOR, "div.field-birthdate div.timezonewarning"
+ )
+ self.assertEqual(len(warnings), 1)
+
+ warning = warnings[0]
+ self.assertTrue(warning.is_displayed())
+ next_element = warning.find_element(By.XPATH, "./following-sibling::*[1]")
+ # Warning messages are generally located just above the field block.
+ self.assertEqual(next_element, datetime)
+
+ date = datetime.find_element(By.TAG_NAME, "input")
+ date.send_keys("invalid")
+ with self.wait_page_loaded():
+ self.selenium.find_element(By.NAME, "_save").click()
+
+ errors = self.selenium.find_element(By.ID, "id_birthdate_error")
+ warning = self.selenium.find_element(
+ By.CSS_SELECTOR, "div.help.timezonewarning"
+ )
+ next_element = warning.find_element(By.XPATH, "./following-sibling::*[1]")
+ # warning message appears above the error message.
+ self.assertEqual(next_element, errors)
+
@requires_tz_support
@override_settings(TIME_ZONE="Asia/Singapore")
diff --git a/tests/auth_tests/operations_migrations/0001_initial.py b/tests/auth_tests/operations_migrations/0001_initial.py
new file mode 100644
index 000000000000..49a475653c1c
--- /dev/null
+++ b/tests/auth_tests/operations_migrations/0001_initial.py
@@ -0,0 +1,14 @@
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+ initial = True
+
+ operations = [
+ migrations.CreateModel(
+ name="OldModel",
+ fields=[
+ ("id", models.AutoField(primary_key=True)),
+ ],
+ ),
+ ]
diff --git a/tests/auth_tests/operations_migrations/0002_rename_oldmodel_to_newmodel.py b/tests/auth_tests/operations_migrations/0002_rename_oldmodel_to_newmodel.py
new file mode 100644
index 000000000000..a23761e8fdeb
--- /dev/null
+++ b/tests/auth_tests/operations_migrations/0002_rename_oldmodel_to_newmodel.py
@@ -0,0 +1,14 @@
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+ dependencies = [
+ ("auth_tests", "0001_initial"),
+ ]
+
+ operations = [
+ migrations.RenameModel(
+ old_name="OldModel",
+ new_name="NewModel",
+ ),
+ ]
diff --git a/tests/auth_tests/operations_migrations/__init__.py b/tests/auth_tests/operations_migrations/__init__.py
new file mode 100644
index 000000000000..e69de29bb2d1
diff --git a/tests/auth_tests/test_management.py b/tests/auth_tests/test_management.py
index f5027da4e5eb..985dfd79f8e7 100644
--- a/tests/auth_tests/test_management.py
+++ b/tests/auth_tests/test_management.py
@@ -7,8 +7,12 @@
from unittest import mock
from django.apps import apps
+from django.conf import settings
from django.contrib.auth import get_permission_codename, management
-from django.contrib.auth.management import create_permissions, get_default_username
+from django.contrib.auth.management import (
+ create_permissions,
+ get_default_username,
+)
from django.contrib.auth.management.commands import changepassword, createsuperuser
from django.contrib.auth.models import Group, Permission, User
from django.contrib.contenttypes.models import ContentType
@@ -16,6 +20,7 @@
from django.core.management.base import CommandError
from django.db import migrations
from django.test import TestCase, override_settings
+from django.test.testcases import TransactionTestCase
from django.utils.translation import gettext_lazy as _
from .models import (
@@ -1528,6 +1533,276 @@ def test_permission_with_proxy_content_type_created(self):
)
+@override_settings(
+ MIGRATION_MODULES=dict(
+ settings.MIGRATION_MODULES,
+ auth_tests="auth_tests.operations_migrations",
+ ),
+)
+class PermissionRenameOperationsTests(TransactionTestCase):
+ available_apps = [
+ "django.contrib.contenttypes",
+ "django.contrib.auth",
+ "auth_tests",
+ ]
+
+ databases = {"default", "other"}
+
+ def setUp(self):
+ self.stdout = StringIO()
+ self.addCleanup(self.stdout.close)
+
+ def test_permission_rename(self):
+ # Create initial content type and permissions for OldModel.
+ call_command("migrate", "auth_tests", "0001", verbosity=0)
+ # Apply the migration that renames OldModel to NewModel.
+ call_command("migrate", "auth_tests", "0002", verbosity=0)
+
+ actions = ContentType._meta.default_permissions
+
+ for action in actions:
+ self.assertFalse(
+ Permission.objects.filter(codename=f"{action}_oldmodel").exists()
+ )
+ self.assertTrue(
+ Permission.objects.filter(codename=f"{action}_newmodel").exists()
+ )
+
+ # Unapply that migration, renaming NewModel back to OldModel.
+ call_command(
+ "migrate",
+ "auth_tests",
+ "0001",
+ database="default",
+ interactive=False,
+ verbosity=0,
+ )
+
+ for action in actions:
+ self.assertTrue(
+ Permission.objects.filter(codename=f"{action}_oldmodel").exists()
+ )
+ self.assertFalse(
+ Permission.objects.filter(codename=f"{action}_newmodel").exists()
+ )
+
+ call_command(
+ "migrate",
+ "auth_tests",
+ "zero",
+ database="default",
+ interactive=False,
+ verbosity=0,
+ )
+
+ @mock.patch(
+ "django.db.router.allow_migrate_model",
+ return_value=False,
+ )
+ def test_rename_skipped_if_router_disallows(self, _):
+ # Create them manually, auto permissions won't create
+ # since router disallows
+
+ ct = ContentType.objects.create(app_label="auth_tests", model="oldmodel")
+ Permission.objects.create(
+ codename="change_oldmodel",
+ name="Can change old model",
+ content_type=ct,
+ )
+
+ # Create initial content type and permissions for OldModel.
+ call_command("migrate", "auth_tests", "0001", verbosity=0)
+ # Apply the migration that renames OldModel to NewModel.
+ call_command("migrate", "auth_tests", "0002", verbosity=0)
+
+ self.assertTrue(Permission.objects.filter(codename="change_oldmodel").exists())
+ self.assertFalse(Permission.objects.filter(codename="change_newmodel").exists())
+
+ call_command(
+ "migrate",
+ "auth_tests",
+ "zero",
+ database="default",
+ interactive=False,
+ verbosity=0,
+ )
+
+ def test_rename_backward_without_permissions(self):
+ """
+ Backward migration handles the case where permissions
+ don't exist (e.g., they were manually deleted).
+ """
+ # Create initial content type and permissions for OldModel.
+ call_command("migrate", "auth_tests", "0001", verbosity=0)
+ # Apply the migration that renames OldModel to NewModel.
+ call_command("migrate", "auth_tests", "0002", verbosity=0)
+
+ Permission.objects.filter(content_type__app_label="auth_tests").delete()
+
+ call_command(
+ "migrate",
+ "auth_tests",
+ "zero",
+ database="default",
+ interactive=False,
+ verbosity=0,
+ )
+ self.assertFalse(
+ Permission.objects.filter(
+ codename__in=["change_oldmodel", "change_newmodel"]
+ ).exists()
+ )
+
+ call_command(
+ "migrate",
+ "auth_tests",
+ "zero",
+ database="default",
+ interactive=False,
+ verbosity=0,
+ )
+
+ def test_rename_permission_conflict(self):
+ # Create initial content type and permissions for OldModel.
+ call_command("migrate", "auth_tests", "0001", verbosity=0)
+
+ ct = ContentType.objects.get(app_label="auth_tests", model="oldmodel")
+ old_perm = Permission.objects.get(
+ codename="change_oldmodel",
+ name="Can change old model",
+ )
+ conflicting_perm = Permission.objects.create(
+ codename="change_newmodel",
+ name="Can change new model",
+ content_type=ct,
+ )
+
+ with self.assertRaises(RuntimeError):
+ # Apply the migration that renames OldModel to NewModel.
+ call_command(
+ "migrate",
+ "auth_tests",
+ "0002",
+ database="default",
+ interactive=False,
+ stdout=self.stdout,
+ )
+
+ command_output = self.stdout.getvalue()
+
+ self.assertIn(
+ f"Failed to rename permission {old_perm.pk} "
+ f"from '{old_perm.codename}' to '{conflicting_perm.codename}'. "
+ f"Please resolve the conflict manually.",
+ command_output,
+ )
+
+ self.assertTrue(Permission.objects.filter(codename="change_oldmodel").exists())
+
+ with self.assertRaises(RuntimeError):
+ call_command(
+ "migrate",
+ "auth_tests",
+ "zero",
+ database="default",
+ interactive=False,
+ verbosity=0,
+ )
+
+ def test_permission_rename_respects_other_db(self):
+ # Create initial content type and permissions for OldModel.
+ call_command("migrate", "auth_tests", "0001", verbosity=0)
+
+ permission = Permission.objects.using("default").get(
+ codename="add_oldmodel",
+ name="Can add old model",
+ )
+
+ # Create initial content type and permissions for OldModel.
+ call_command("migrate", "auth_tests", "0001", verbosity=0, database="other")
+ # Apply the migration that renames OldModel to NewModel.
+ call_command("migrate", "auth_tests", "0002", verbosity=0, database="other")
+
+ permission.refresh_from_db()
+ self.assertEqual(permission.codename, "add_oldmodel")
+ self.assertFalse(
+ Permission.objects.using("other").filter(codename="add_oldmodel").exists()
+ )
+ self.assertTrue(
+ Permission.objects.using("other").filter(codename="add_newmodel").exists()
+ )
+
+ call_command(
+ "migrate",
+ "auth_tests",
+ "zero",
+ database="other",
+ interactive=False,
+ verbosity=0,
+ )
+ call_command(
+ "migrate",
+ "auth_tests",
+ "zero",
+ database="default",
+ interactive=False,
+ verbosity=0,
+ )
+
+ def test_verbosity_prints(self):
+ # Create initial content type and permissions for OldModel.
+ call_command("migrate", "auth_tests", "0001", verbosity=0)
+ # Apply the migration that renames OldModel to NewModel.
+ call_command("migrate", "auth_tests", "0002", verbosity=2, stdout=self.stdout)
+
+ command_output = self.stdout.getvalue()
+ self.assertIn(
+ "Renamed permission(s): auth_tests.add_oldmodel → add_newmodel",
+ command_output,
+ )
+ self.assertIn(
+ "Renamed permission(s): auth_tests.change_oldmodel → change_newmodel",
+ command_output,
+ )
+ self.assertIn(
+ "Renamed permission(s): auth_tests.view_oldmodel → view_newmodel",
+ command_output,
+ )
+ self.assertIn(
+ "Renamed permission(s): auth_tests.delete_oldmodel → delete_newmodel",
+ command_output,
+ )
+
+ call_command("migrate", "auth_tests", "0001", verbosity=2, stdout=self.stdout)
+
+ command_output = self.stdout.getvalue()
+ self.assertIn(
+ "Renamed permission(s): auth_tests.add_newmodel → add_oldmodel",
+ command_output,
+ )
+ self.assertIn(
+ "Renamed permission(s): auth_tests.change_newmodel → change_oldmodel",
+ command_output,
+ )
+ self.assertIn(
+ "Renamed permission(s): auth_tests.view_newmodel → view_oldmodel",
+ command_output,
+ )
+ self.assertIn(
+ "Renamed permission(s): auth_tests.delete_newmodel → delete_oldmodel",
+ command_output,
+ )
+
+ call_command(
+ "migrate",
+ "auth_tests",
+ "zero",
+ database="default",
+ interactive=False,
+ verbosity=0,
+ )
+
+
class DefaultDBRouter:
"""Route all writes to default."""
diff --git a/tests/auth_tests/test_templates.py b/tests/auth_tests/test_templates.py
index edde6ca6b4e3..775409af597e 100644
--- a/tests/auth_tests/test_templates.py
+++ b/tests/auth_tests/test_templates.py
@@ -37,6 +37,20 @@ def test_password_reset_view(self):
)
self.assertContains(response, "
Password reset ")
+ def test_password_reset_view_error_form(self):
+ response = self.client.post(reverse("password_reset"), {})
+ self.assertContains(
+ response,
+ '
'
+ '
Email address: '
+ '
'
+ "This field is required. "
+ '
',
+ html=True,
+ )
+
def test_password_reset_view_error_title(self):
response = self.client.post(reverse("password_reset"), {})
self.assertContains(
@@ -96,6 +110,38 @@ def test_password_reset_confirm_view_error_title(self):
response, "
Error: Enter new password | Django site admin "
)
+ def test_password_reset_confirm_view_error_form(self):
+ client = PasswordResetConfirmClient()
+ default_token_generator = PasswordResetTokenGenerator()
+ token = default_token_generator.make_token(self.user)
+ uidb64 = urlsafe_base64_encode(str(self.user.pk).encode())
+ url = reverse(
+ "password_reset_confirm", kwargs={"uidb64": uidb64, "token": token}
+ )
+ response = client.post(url, {})
+ self.assertContains(
+ response,
+ '
'
+ '
New password: '
+ '
'
+ "This field is required. "
+ '
',
+ html=True,
+ )
+ self.assertContains(
+ response,
+ '
'
+ '
Confirm password: '
+ '
'
+ "This field is required. "
+ '
',
+ html=True,
+ )
+
@override_settings(AUTH_USER_MODEL="auth_tests.CustomUser")
def test_password_reset_confirm_view_custom_username_hint(self):
custom_user = CustomUser.custom_objects.create_user(
diff --git a/tests/auth_tests/test_views.py b/tests/auth_tests/test_views.py
index d7d59d9fc033..a3863b6233c0 100644
--- a/tests/auth_tests/test_views.py
+++ b/tests/auth_tests/test_views.py
@@ -1555,7 +1555,8 @@ def test_user_with_usable_password_change_password(self):
# Usable password field.
self.assertContains(
response,
- "
Password-based authentication: ",
+ ''
+ "Password-based authentication: ",
)
# Submit buttons
self.assertContains(response, '