From 9c618d8b21a888be59852bede70d2c6678617022 Mon Sep 17 00:00:00 2001 From: Diederik van der Boor Date: Mon, 1 May 2017 12:33:44 +0200 Subject: [PATCH] Improve tests to use a unified qrepr() output --- polymorphic/tests/test_orm.py | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/polymorphic/tests/test_orm.py b/polymorphic/tests/test_orm.py index f284033..65601a5 100644 --- a/polymorphic/tests/test_orm.py +++ b/polymorphic/tests/test_orm.py @@ -360,11 +360,7 @@ class PolymorphicTests(TestCase): # no pretty printing ModelShow1_plain.objects.create(field1='abc') ModelShow2_plain.objects.create(field1='abc', field2='def') - # repr classnames are not hardcoded in Django 1.11+ - if django.VERSION >= (1, 11): - self.assertEqual(qrepr(ModelShow1_plain.objects.all()), ', ]>') - else: - self.assertEqual(qrepr(ModelShow1_plain.objects.all()), ', ]>') + self.assertEqual(qrepr(ModelShow1_plain.objects.all()), ', ]>') def test_extra_method(self): self.create_model2abcd() @@ -813,9 +809,13 @@ def qrepr(data): Ensure consistent repr() output for the QuerySet object. """ if isinstance(data, QuerySet): - if django.VERSION >= (1, 10): + if django.VERSION >= (1, 11): return repr(data) + elif django.VERSION >= (1, 10): + # Django 1.11 still shows "' % data + # Simulate Django 1.11 behavior for older Django versions. + return '<{} {}>'.format(data.__class__.__name__, repr(data)) return repr(data)