Add tests for string pk

tests/admin.py

@@ -7,22 +7,20 @@
     CustomAdminModel2,
     InheritedAdminModel,
     FieldsetsAdminModel,
+    StringAdminModel,
 )
 
 
 class CustomAdminForm(forms.ModelForm):
-
     def clean_name(self):
-        """ Fake cleaning for tests
-        """
+        """Fake cleaning for tests"""
         name = self.cleaned_data.get("name")
-        if (self.instance.pk
-                and name == "invalid {}".format(self.instance.pk)):
+        if self.instance.pk and name == "invalid {}".format(self.instance.pk):
             raise forms.ValidationError("Invalid model name")
         return name
 
     class Meta:
-        fields = ("name", )
+        fields = ("name",)
         model = CustomAdminModel
 
 
@@ -31,7 +29,9 @@ class InheritedAdminInline(admin.TabularInline):
 
 
 class CustomAdmin(admin.ModelAdmin):
-    inlines = [InheritedAdminInline, ]
+    inlines = [
+        InheritedAdminInline,
+    ]
     model = CustomAdminModel
     form = CustomAdminForm
 
@@ -45,17 +45,21 @@ class CustomAdminWithCustomTemplate(admin.ModelAdmin):
     change_form_template = "admin/change_form_template.html"
 
 
+class CustomAdminWithStringPK(admin.ModelAdmin):
+    model = StringAdminModel
+
+
 class CustomAdminWithGetFieldsetsAncestor(admin.ModelAdmin):
     """
     Ancestor that defines fieldsets
     which should not be used by the mass_change_view()
     """
+
     model = FieldsetsAdminModel
     fieldsets = ()
 
 
 class CustomAdminWithGetFieldsets(CustomAdminWithGetFieldsetsAncestor):
-
     def get_fieldsets(self, request, obj=None):
         return (
             ("First part of name", {"fields": ("first_name",)}),
@@ -66,18 +70,19 @@ def get_fieldsets(self, request, obj=None):
 admin.site.register(CustomAdminModel, CustomAdmin)
 admin.site.register(CustomAdminModel2, CustomAdminWithCustomTemplate)
 admin.site.register(FieldsetsAdminModel, CustomAdminWithGetFieldsets)
+admin.site.register(StringAdminModel, CustomAdminWithStringPK)
 
 
 class BaseAdmin(admin.ModelAdmin):
-    readonly_fields = ("name", )
+    readonly_fields = ("name",)
 
 
 class InheritedAdmin(BaseAdmin):
     model = InheritedAdminModel
-    raw_id_fields = ("fk_field", )
+    raw_id_fields = ("fk_field",)
 
 
 admin.site.register(InheritedAdminModel, InheritedAdmin)
 
-custom_admin_site = admin.AdminSite(name='myadmin')
+custom_admin_site = admin.AdminSite(name="myadmin")
 custom_admin_site.register(CustomAdminModel, CustomAdmin)

tests/migrations/0004_add_model_with_string_primary_key.py

@@ -0,0 +1,22 @@
+# Generated by Django 4.2.5 on 2023-09-19 10:36
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("tests", "0003_fieldsetsadminmodel_alter_customadminmodel_id_and_more"),
+    ]
+
+    operations = [
+        migrations.CreateModel(
+            name="StringAdminModel",
+            fields=[
+                (
+                    "primary",
+                    models.CharField(max_length=32, primary_key=True, serialize=False),
+                ),
+                ("name", models.CharField(max_length=32)),
+            ],
+        ),
+    ]

tests/models.py

@@ -9,6 +9,16 @@ class Meta:
         app_label = "tests"
 
 
+class StringAdminModel(models.Model):
+    """Special model for testing string primary keys."""
+
+    primary = models.CharField(max_length=32, primary_key=True)
+    name = models.CharField(max_length=32)
+
+    class Meta:
+        app_label = "tests"
+
+
 class CustomAdminModel2(models.Model):
     name = models.CharField(max_length=32)
 

tests/tests.py

@@ -2,6 +2,7 @@
 from django.contrib.auth.models import User
 from django.contrib import admin
 from django.test import TestCase, override_settings, RequestFactory
+
 try:
     from django.urls import reverse
 except ImportError:  # Django<2.0
@@ -14,6 +15,7 @@
     CustomAdminModel2,
     InheritedAdminModel,
     FieldsetsAdminModel,
+    StringAdminModel,
 )
 from .site import CustomAdminSite
 from .mocks import MockRenderMassAdmin
@@ -76,6 +78,20 @@ def test_update(self, admin_name='admin'):
         # all models have changed
         self.assertEqual(list(new_names), ["new name"] * 3)
 
+    def test_update_with_string_primary_key_and_special_chars(self, admin_name='admin'):
+        models = [StringAdminModel.objects.create(
+            primary="{}/3".format(1 + i),
+            name="model {}".format(i),
+        ) for i in range(0, 3)]
+
+        response = self.client.post(get_massadmin_url(models, self.client.session),
+                                    {"_mass_change": "name",
+                                     "name": "new name"})
+        self.assertRedirects(response, get_changelist_url(StringAdminModel, admin_name))
+        new_names = StringAdminModel.objects.order_by("pk").values_list("name", flat=True)
+        # all models have changed
+        self.assertEqual(list(new_names), ["new name"] * 3)
+
     @override_settings(ROOT_URLCONF='tests.urls_custom_admin')
     def test_custom_admin_site_update(self):
         """ We test_update for a custom admin on top of a regular as well
@@ -164,6 +180,7 @@ def test_excluded_queryset(self):
 class CustomizationTestCase(TestCase):
     """ MassAdmin has all customized options from related ModelAdmin
     """
+
     def setUp(self):
         self.user = User.objects.create_superuser(
             'temporary', '[email protected]', 'temporary')