Commit d3543af7 authored by Tim Graham's avatar Tim Graham
Browse files

Added sorting to some admin_view tests to prevent transient failures; refs #23380.

parent 16548cfc

tests/admin_filters/tests.py

+3 −3
Original line number Diff line number Diff line
@@ -438,7 +438,7 @@ class ListFiltersTests(TestCase): 
        # Make sure that all users are present in the contrib's list filter

        filterspec = changelist.get_filters(request)[0][2]

        expected = [(self.alfred.pk, 'alfred'), (self.bob.pk, 'bob'), (self.lisa.pk, 'lisa')]

        self.assertEqual(filterspec.lookup_choices, expected)

        self.assertEqual(sorted(filterspec.lookup_choices), sorted(expected))



        request = self.request_factory.get('/', {'contributors__isnull': 'True'})

        changelist = self.get_changelist(request, Book, modeladmin)
@@ -526,7 +526,7 @@ class ListFiltersTests(TestCase): 
        # Make sure that only actual authors are present in author's list filter

        filterspec = changelist.get_filters(request)[0][4]

        expected = [(self.alfred.pk, 'alfred'), (self.bob.pk, 'bob')]

        self.assertEqual(filterspec.lookup_choices, expected)

        self.assertEqual(sorted(filterspec.lookup_choices), sorted(expected))



    def test_relatedonlyfieldlistfilter_manytomany(self):

        modeladmin = BookAdminRelatedOnlyFilter(Book, site)
@@ -537,7 +537,7 @@ class ListFiltersTests(TestCase): 
        # Make sure that only actual contributors are present in contrib's list filter

        filterspec = changelist.get_filters(request)[0][5]

        expected = [(self.bob.pk, 'bob'), (self.lisa.pk, 'lisa')]

        self.assertEqual(filterspec.lookup_choices, expected)

        self.assertEqual(sorted(filterspec.lookup_choices), sorted(expected))



    def test_booleanfieldlistfilter(self):

        modeladmin = BookAdmin(Book, site)