Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Fixed #21403: Corrected test code

A test for annotations incorrectly assumed that the first instance
(in the test) of a model using AutoField for PK will always get pk=1.
The test was changed to compare against actual instance id instead.
  • Loading branch information...
commit 3411af3cd195ef79f3320ee9b2ab261e9ee5a3f5 1 parent 70e6131
@shaib shaib authored
Showing with 3 additions and 3 deletions.
  1. +3 −3 tests/generic_relations_regress/tests.py
View
6 tests/generic_relations_regress/tests.py
@@ -222,10 +222,10 @@ def test_annotate(self):
Link.objects.create(content_object=b)
qs = HasLinkThing.objects.annotate(Sum('links'))
# If content_type restriction isn't in the query's join condition,
- # then wrong results are produced here as b will also match (it has
- # same pk).
+ # then wrong results are produced here as the link to b will also match
+ # (b and hs1 have equal pks).
self.assertEqual(qs.count(), 1)
- self.assertEqual(qs[0].links__sum, 1)
+ self.assertEqual(qs[0].links__sum, l.id)
l.delete()
# Now if we don't have proper left join, we will not produce any
# results at all here.
Please sign in to comment.
Something went wrong with that request. Please try again.