summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorPrzemysław Buczkowski <przemub@creditdigital.co.uk>2018-09-13 13:36:14 +0100
committerTim Graham <timograham@gmail.com>2018-10-02 09:17:23 -0400
commit70d0a1ca02f42c0f8984b6234ca0f9d7e354a135 (patch)
tree11a9e6d5c92e55124567da316204f977bd3ef0ca /tests
parent7598cd4748dc402b0209e5eedb6d2a83c3da1620 (diff)
Fixed #29711 -- Added a system check for uniquness of admin actions' __name__.
Diffstat (limited to 'tests')
-rw-r--r--tests/modeladmin/test_checks.py27
1 files changed, 27 insertions, 0 deletions
diff --git a/tests/modeladmin/test_checks.py b/tests/modeladmin/test_checks.py
index 6a10441471..89fde35d3c 100644
--- a/tests/modeladmin/test_checks.py
+++ b/tests/modeladmin/test_checks.py
@@ -1309,3 +1309,30 @@ class ActionsCheckTests(CheckTestCase):
'custom_permission_action action.',
id='admin.E129',
)
+
+ def test_actions_not_unique(self):
+ def action(modeladmin, request, queryset):
+ pass
+
+ class BandAdmin(ModelAdmin):
+ actions = (action, action)
+
+ self.assertIsInvalid(
+ BandAdmin, Band,
+ "__name__ attributes of actions defined in "
+ "<class 'modeladmin.test_checks.ActionsCheckTests."
+ "test_actions_not_unique.<locals>.BandAdmin'> must be unique.",
+ id='admin.E130',
+ )
+
+ def test_actions_unique(self):
+ def action1(modeladmin, request, queryset):
+ pass
+
+ def action2(modeladmin, request, queryset):
+ pass
+
+ class BandAdmin(ModelAdmin):
+ actions = (action1, action2)
+
+ self.assertIsValid(BandAdmin, Band)