diff options
| author | Luke Plant <L.Plant.98@cantab.net> | 2012-12-24 01:33:44 +0000 |
|---|---|---|
| committer | Tim Graham <timograham@gmail.com> | 2013-08-16 17:55:08 -0400 |
| commit | d5da495a2edc741b6496821baa0d6bcee9dce9bb (patch) | |
| tree | 4664fa9e0e83ea71a50a694bbb2b112430efe4c2 | |
| parent | bf611f14ec13312aa822beec72c63bd04950613c (diff) | |
[1.4.x] Fixed #20906 -- Fixed a dependence on set-ordering in tests
Backport of 1ae64e96c1 from master
| -rw-r--r-- | tests/regressiontests/admin_filters/tests.py | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/tests/regressiontests/admin_filters/tests.py b/tests/regressiontests/admin_filters/tests.py index c75ba6c23d..7eb0c6ab0a 100644 --- a/tests/regressiontests/admin_filters/tests.py +++ b/tests/regressiontests/admin_filters/tests.py @@ -82,11 +82,11 @@ class DepartmentListFilterLookupWithNonStringValue(SimpleListFilter): parameter_name = 'department' def lookups(self, request, model_admin): - return set([ + return sorted(set([ (employee.department.id, # Intentionally not a string (Refs #19318) employee.department.code) for employee in model_admin.queryset(request).all() - ]) + ])) def queryset(self, request, queryset): if self.value(): @@ -681,10 +681,9 @@ class ListFiltersTests(TestCase): filterspec = changelist.get_filters(request)[0][-1] self.assertEqual(force_unicode(filterspec.title), u'department') choices = list(filterspec.choices(changelist)) - - self.assertEqual(choices[2]['display'], u'DEV') - self.assertEqual(choices[2]['selected'], True) - self.assertEqual(choices[2]['query_string'], '?department=%s' % self.john.pk) + self.assertEqual(choices[1]['display'], 'DEV') + self.assertEqual(choices[1]['selected'], True) + self.assertEqual(choices[1]['query_string'], '?department=%s' % self.john.pk) def test_fk_with_to_field(self): """ |
