summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorJacob Walls <jacobtylerwalls@gmail.com>2025-10-13 17:22:17 -0400
committerJacob Walls <jacobtylerwalls@gmail.com>2025-10-14 15:50:26 -0400
commit8baee531d40cd6ee8b342dbcf2c7924d20694969 (patch)
treea4d8c8d0aa4180b20e219d9b3e62d5f57f4d5c38 /tests
parent94cbd67d9eb9712be5518c3d5b4c17eac2e63629 (diff)
[5.2.x] Fixed #36648, Refs #33772 -- Accounted for composite pks in first()/last() when aggregating.
Backport of 02eed4f37879b2077496f86bb1378a076b981233 from main.
Diffstat (limited to 'tests')
-rw-r--r--tests/composite_pk/test_aggregate.py20
1 files changed, 20 insertions, 0 deletions
diff --git a/tests/composite_pk/test_aggregate.py b/tests/composite_pk/test_aggregate.py
index d852fdce30..8a2067cb90 100644
--- a/tests/composite_pk/test_aggregate.py
+++ b/tests/composite_pk/test_aggregate.py
@@ -141,3 +141,23 @@ class CompositePKAggregateTests(TestCase):
msg = "Max expression does not support composite primary keys."
with self.assertRaisesMessage(ValueError, msg):
Comment.objects.aggregate(Max("pk"))
+
+ def test_first_from_unordered_queryset_aggregation_pk_selected(self):
+ self.assertEqual(
+ Comment.objects.values("pk").annotate(max=Max("id")).first(),
+ {"pk": (1, 1), "max": 1},
+ )
+
+ def test_first_from_unordered_queryset_aggregation_pk_selected_separately(self):
+ self.assertEqual(
+ Comment.objects.values("tenant", "id").annotate(max=Max("id")).first(),
+ {"tenant": 1, "id": 1, "max": 1},
+ )
+
+ def test_first_from_unordered_queryset_aggregation_pk_incomplete(self):
+ msg = (
+ "Cannot use QuerySet.first() on an unordered queryset performing "
+ "aggregation. Add an ordering with order_by()."
+ )
+ with self.assertRaisesMessage(TypeError, msg):
+ Comment.objects.values("tenant").annotate(max=Max("id")).first()