Ticket #25688: empty_list_fails_to_register.diff

django/contrib/admin/decorators.py

@@ -9 +9 @@
 
     A kwarg of `site` can be passed as the admin site, otherwise the default
     admin site will be used.
+
+    At least one model must be passed to register.
     """
     from django.contrib.admin import ModelAdmin
     from django.contrib.admin.sites import site, AdminSite
@@ -22 +24 @@
         if not issubclass(admin_class, ModelAdmin):
             raise ValueError('Wrapped class must subclass ModelAdmin.')
 
+        if not models:
+            raise ValueError('At least one model must be passed to register.')
+
         admin_site.register(models, admin_class=admin_class)
 
         return admin_class

tests/admin_registration/tests.py

@@ -133 +133 @@
     def test_custom_site_not_an_admin_site(self):
         self.assertRaisesMessage(ValueError, 'site must subclass AdminSite',
             register(Person, site=Traveler), NameAdmin)
+
+    def test_empty_list_registration_fails(self):
+        self.assertRaisesMessage(ValueError, 'At least one model must be passed to register.',
+            register(), NameAdmin)
+