diff options
| author | Jonathan Richards <jonathan@golorry.com> | 2021-03-14 14:00:40 -0700 |
|---|---|---|
| committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2021-04-14 19:46:45 +0200 |
| commit | d0267690f8a8e83065459d13a5a6f29e75640f78 (patch) | |
| tree | 0002b1dc6c6638c1a86f65de2e367cb0b4ded30d /tests | |
| parent | 65dfb06a1ab56c238cc80f5e1c31f61210c4577d (diff) | |
[3.2.x] Fixed #32548 -- Fixed crash when combining Q() objects with boolean expressions.
Backport of 00b0786de533dbb3f6208d8d5eaddbf765b4e5b8 from main.
Regression in 466920f6d726eee90d5566e0a9948e92b33a122e.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/expressions/tests.py | 4 | ||||
| -rw-r--r-- | tests/queries/test_q.py | 22 |
2 files changed, 18 insertions, 8 deletions
diff --git a/tests/expressions/tests.py b/tests/expressions/tests.py index 1c32ab045b..9a34242de7 100644 --- a/tests/expressions/tests.py +++ b/tests/expressions/tests.py @@ -821,6 +821,10 @@ class BasicExpressionsTests(TestCase): Q() & Exists(is_poc), Exists(is_poc) | Q(), Q() | Exists(is_poc), + Q(Exists(is_poc)) & Q(), + Q() & Q(Exists(is_poc)), + Q(Exists(is_poc)) | Q(), + Q() | Q(Exists(is_poc)), ] for conditions in tests: with self.subTest(conditions): diff --git a/tests/queries/test_q.py b/tests/queries/test_q.py index 6dcf36ce02..24a705f07f 100644 --- a/tests/queries/test_q.py +++ b/tests/queries/test_q.py @@ -1,6 +1,8 @@ -from django.db.models import F, Q +from django.db.models import Exists, F, OuterRef, Q from django.test import SimpleTestCase +from .models import Tag + class QTests(SimpleTestCase): def test_combine_and_empty(self): @@ -39,17 +41,14 @@ class QTests(SimpleTestCase): q = Q(price__gt=F('discounted_price')) path, args, kwargs = q.deconstruct() self.assertEqual(path, 'django.db.models.Q') - self.assertEqual(args, ()) - self.assertEqual(kwargs, {'price__gt': F('discounted_price')}) + self.assertEqual(args, (('price__gt', F('discounted_price')),)) + self.assertEqual(kwargs, {}) def test_deconstruct_negated(self): q = ~Q(price__gt=F('discounted_price')) path, args, kwargs = q.deconstruct() - self.assertEqual(args, ()) - self.assertEqual(kwargs, { - 'price__gt': F('discounted_price'), - '_negated': True, - }) + self.assertEqual(args, (('price__gt', F('discounted_price')),)) + self.assertEqual(kwargs, {'_negated': True}) def test_deconstruct_or(self): q1 = Q(price__gt=F('discounted_price')) @@ -88,6 +87,13 @@ class QTests(SimpleTestCase): self.assertEqual(args, (Q(price__gt=F('discounted_price')),)) self.assertEqual(kwargs, {}) + def test_deconstruct_boolean_expression(self): + tagged = Tag.objects.filter(category=OuterRef('pk')) + q = Q(Exists(tagged)) + _, args, kwargs = q.deconstruct() + self.assertEqual(args, (Exists(tagged),)) + self.assertEqual(kwargs, {}) + def test_reconstruct(self): q = Q(price__gt=F('discounted_price')) path, args, kwargs = q.deconstruct() |
