diff options
| author | Clifford Gama <cliffygamy@gmail.com> | 2025-06-12 09:35:07 +0200 |
|---|---|---|
| committer | Sarah Boyce <42296566+sarahboyce@users.noreply.github.com> | 2025-06-16 10:41:24 +0200 |
| commit | 1d89691c7481fbcef6a7bc396e41a651372c1cf3 (patch) | |
| tree | 51220e906f893aba57cc15d455c46a351d8dbb0b /tests | |
| parent | 4de4edf2c05cc80c514c989db480f2fe23ad5ee2 (diff) | |
[5.2.x] Fixed #36453 -- Made When.condition resolve with for_save=False.
Value(None, JSONField()) when used in When.condition incorrectly resolved with
for_save=True, resulting in the value being serialized as SQL NULL instead of
JSON null.
Regression in c1fa3fdd040718356e5a3b9a0fe699d73f47a940.
Thanks to Thomas McKay for the report, and to David Sanders and Simon Charettes
for the review.
Co-authored-by: Sarah Boyce <42296566+sarahboyce@users.noreply.github.com>
Backport of 104cbfd44b9eff010daf0ef0e1ce434385855b13 from main.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/expressions/tests.py | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/tests/expressions/tests.py b/tests/expressions/tests.py index 89601de85b..a06fbc6b95 100644 --- a/tests/expressions/tests.py +++ b/tests/expressions/tests.py @@ -29,6 +29,7 @@ from django.db.models import ( FloatField, Func, IntegerField, + JSONField, Max, Min, Model, @@ -81,6 +82,7 @@ from .models import ( Company, Employee, Experiment, + JSONFieldModel, Manager, Number, RemoteEmployee, @@ -365,6 +367,21 @@ class BasicExpressionsTests(TestCase): Number.objects.all(), [None, None], lambda n: n.float, ordered=False ) + @skipUnlessDBFeature("supports_json_field") + def test_update_jsonfield_case_when_key_is_null(self): + obj = JSONFieldModel.objects.create(data={"key": None}) + updated = JSONFieldModel.objects.update( + data=Case( + When( + data__key=Value(None, JSONField()), + then=Value({"key": "something"}, JSONField()), + ), + ) + ) + self.assertEqual(updated, 1) + obj.refresh_from_db() + self.assertEqual(obj.data, {"key": "something"}) + def test_filter_with_join(self): # F Expressions can also span joins Company.objects.update(point_of_contact=F("ceo")) |
