diff options
| author | Jacob Walls <jacobtylerwalls@gmail.com> | 2025-02-16 21:35:12 -0500 |
|---|---|---|
| committer | Sarah Boyce <42296566+sarahboyce@users.noreply.github.com> | 2025-02-18 17:28:24 +0100 |
| commit | 0bf412111be686b6b23e00863f5d449d63557dbf (patch) | |
| tree | 549c19f89fa6b93369cb8c2a0a938ef1c668f2a7 /tests/model_fields | |
| parent | 9d22a7d8f0e814a596ecbeb6efd051262f6a03e3 (diff) | |
Fixed #35167 -- Delegated to super() in JSONField.get_db_prep_save().
Avoids reports of bulk_update() sending Cast expressions
to JSONField.get_prep_value().
Co-authored-by: Simon Charette <charette.s@gmail.com>
Diffstat (limited to 'tests/model_fields')
| -rw-r--r-- | tests/model_fields/models.py | 11 | ||||
| -rw-r--r-- | tests/model_fields/test_jsonfield.py | 19 |
2 files changed, 29 insertions, 1 deletions
diff --git a/tests/model_fields/models.py b/tests/model_fields/models.py index fdea06b23d..299e927615 100644 --- a/tests/model_fields/models.py +++ b/tests/model_fields/models.py @@ -430,6 +430,17 @@ class RelatedJSONModel(models.Model): required_db_features = {"supports_json_field"} +class CustomSerializationJSONModel(models.Model): + class StringifiedJSONField(models.JSONField): + def get_prep_value(self, value): + return json.dumps(value, cls=self.encoder) + + json_field = StringifiedJSONField() + + class Meta: + required_db_features = {"supports_json_field"} + + class AllFieldsModel(models.Model): big_integer = models.BigIntegerField() binary = models.BinaryField() diff --git a/tests/model_fields/test_jsonfield.py b/tests/model_fields/test_jsonfield.py index 5a9cf9ad7a..267b9a0e66 100644 --- a/tests/model_fields/test_jsonfield.py +++ b/tests/model_fields/test_jsonfield.py @@ -40,7 +40,13 @@ from django.db.models.functions import Cast from django.test import SimpleTestCase, TestCase, skipIfDBFeature, skipUnlessDBFeature from django.test.utils import CaptureQueriesContext -from .models import CustomJSONDecoder, JSONModel, NullableJSONModel, RelatedJSONModel +from .models import ( + CustomJSONDecoder, + CustomSerializationJSONModel, + JSONModel, + NullableJSONModel, + RelatedJSONModel, +) @skipUnlessDBFeature("supports_json_field") @@ -298,6 +304,17 @@ class TestSaveLoad(TestCase): obj.refresh_from_db() self.assertEqual(obj.value, value) + def test_bulk_update_custom_get_prep_value(self): + objs = CustomSerializationJSONModel.objects.bulk_create( + [CustomSerializationJSONModel(pk=1, json_field={"version": "1"})] + ) + objs[0].json_field["version"] = "1-alpha" + CustomSerializationJSONModel.objects.bulk_update(objs, ["json_field"]) + self.assertSequenceEqual( + CustomSerializationJSONModel.objects.values("json_field"), + [{"json_field": '{"version": "1-alpha"}'}], + ) + @skipUnlessDBFeature("supports_json_field") class TestQuerying(TestCase): |
