summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorJames Timmins <jameshtimmins@gmail.com>2019-09-19 19:20:40 -0700
committerMariusz Felisiak <felisiak.mariusz@gmail.com>2019-09-20 11:14:39 +0200
commit3697ddbf7526e85483067db5a2fde93f661da360 (patch)
tree29fe1740f7109dc9cd51f621cfdd5d91873b7632 /tests
parentdef1857d9b7c79a48c0c3ecf997321fa76b479d9 (diff)
[3.0.x] Fixed #30771 -- Fixed exact lookup against queries with selected columns.
Use pre-existing select fields (and thereby GROUP BY fields) from subquery if they were specified, instead of always defaulting to pk. Thanks Aur Saraf for the report and Simon Charette for guidance. Backport of 0719edcd5fed56157ffb3323a8f634aa5e8f9a80 from master
Diffstat (limited to 'tests')
-rw-r--r--tests/lookup/tests.py13
1 files changed, 13 insertions, 0 deletions
diff --git a/tests/lookup/tests.py b/tests/lookup/tests.py
index a603824c0d..1958b995b2 100644
--- a/tests/lookup/tests.py
+++ b/tests/lookup/tests.py
@@ -5,6 +5,7 @@ from operator import attrgetter
from django.core.exceptions import FieldError
from django.db import connection
+from django.db.models import Max
from django.db.models.expressions import Exists, OuterRef
from django.db.models.functions import Substr
from django.test import TestCase, skipUnlessDBFeature
@@ -956,3 +957,15 @@ class LookupTests(TestCase):
),
)
self.assertEqual(qs.get(has_author_alias_match=True), tag)
+
+ def test_exact_query_rhs_with_selected_columns(self):
+ newest_author = Author.objects.create(name='Author 2')
+ authors_max_ids = Author.objects.filter(
+ name='Author 2',
+ ).values(
+ 'name',
+ ).annotate(
+ max_id=Max('id'),
+ ).values('max_id')
+ authors = Author.objects.filter(id=authors_max_ids[:1])
+ self.assertEqual(authors.get(), newest_author)