diff options
| author | Anton Samarchyan <anton.samarchyan@savoirfairelinux.com> | 2016-11-30 10:16:40 -0500 |
|---|---|---|
| committer | Tim Graham <timograham@gmail.com> | 2016-12-22 09:32:11 -0500 |
| commit | da792400503265a80e5307f17e59b65ec88694aa (patch) | |
| tree | 9af9409153805dbb845bde8d79008eb10fbe4afe /tests/modeladmin | |
| parent | bf4516a6280fd9381a84402702917452c2805d75 (diff) | |
Fixed #26632 -- Skipped admin.E111 list_display_links check if get_list_display() is overridden.
Diffstat (limited to 'tests/modeladmin')
| -rw-r--r-- | tests/modeladmin/test_checks.py | 29 |
1 files changed, 29 insertions, 0 deletions
diff --git a/tests/modeladmin/test_checks.py b/tests/modeladmin/test_checks.py index 89effa9bff..0c66544b6b 100644 --- a/tests/modeladmin/test_checks.py +++ b/tests/modeladmin/test_checks.py @@ -526,6 +526,35 @@ class ListDisplayLinksCheckTests(CheckTestCase): self.assertIsValid(TestModelAdmin, ValidationTestModel) + def test_list_display_links_check_skipped_if_get_list_display_overridden(self): + """ + list_display_links check is skipped if get_list_display() is overridden. + """ + class TestModelAdmin(ModelAdmin): + list_display_links = ['name', 'subtitle'] + + def get_list_display(self, request): + pass + + self.assertIsValid(TestModelAdmin, ValidationTestModel) + + def test_list_display_link_checked_for_list_tuple_if_get_list_display_overridden(self): + """ + list_display_links is checked for list/tuple/None even if + get_list_display() is overridden. + """ + class TestModelAdmin(ModelAdmin): + list_display_links = 'non-list/tuple' + + def get_list_display(self, request): + pass + + self.assertIsInvalid( + TestModelAdmin, ValidationTestModel, + "The value of 'list_display_links' must be a list, a tuple, or None.", + 'admin.E110' + ) + class ListFilterTests(CheckTestCase): |
