Browse files

[1.0.X] Fixed #9411 -- Changed the test from r9081 to be more robust …

…across different database backends.

Backport of [9254] from trunk (I neglected to mention in the ticket this affected 1.0.X branch).

Also updated svnmerge metadata; all eligible fixes have been backported and all others blocked.

git-svn-id: bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
1 parent 71ae6cb commit d323ef20d3e7cb22892c45d47708c00bdf3ed3e2 @kmtracey kmtracey committed Oct 24, 2008
Showing with 6 additions and 5 deletions.
  1. +6 −5 tests/regressiontests/queries/
@@ -974,11 +974,12 @@ def __unicode__(self):
>>> expected == result
-Make sure bump_prefix() (an internal Query method) doesn't (re-)break.
->>> query = Tag.objects.values_list('id').order_by().query
->>> query.bump_prefix()
->>> print query.as_sql()[0]
-SELECT U0."id" FROM "queries_tag" U0
+Make sure bump_prefix() (an internal Query method) doesn't (re-)break. It's
+sufficient that this query runs without error.
+>>> qs = Tag.objects.values_list('id', flat=True).order_by('id')
+>>> qs.query.bump_prefix()
+>>> list(qs)
+[1, 2, 3, 4, 5]
Calling order_by() with no parameters removes any existing ordering on the
model. But it should still be possible to add new ordering after that.

0 comments on commit d323ef2

Please sign in to comment.