Ticket #11970: 11970-with-test3.diff

django/core/serializers/json.py

@@ -8 +8 @@
 
 from django.core.serializers.python import Serializer as PythonSerializer
 from django.core.serializers.python import Deserializer as PythonDeserializer
+from django.core.serializers import base
 from django.utils import simplejson
 from django.utils.timezone import is_aware
 
@@ -27 +28 @@
         if callable(getattr(self.stream, 'getvalue', None)):
             return self.stream.getvalue()
 
+
 def Deserializer(stream_or_string, **options):
     """
     Deserialize a stream or string of JSON data.
@@ -35 +37 @@
         stream = StringIO(stream_or_string)
     else:
         stream = stream_or_string
-    for obj in PythonDeserializer(simplejson.load(stream), **options):
-        yield obj
+    try:
+        for obj in PythonDeserializer(simplejson.load(stream), **options):
+            yield obj
+    except ValueError, e:
+        # Map to deserializer error
+        raise base.DeserializationError(e.args)
+
 
 class DjangoJSONEncoder(simplejson.JSONEncoder):
     """

tests/regressiontests/serializers_regress/tests.py

@@ -19 +19 @@
 
 from django.core import serializers
 from django.core.serializers import SerializerDoesNotExist
+from django.core.serializers.base import DeserializationError
 from django.db import connection, models
 from django.test import TestCase
 from django.utils.functional import curry
@@ -390 +391 @@
         with self.assertRaises(SerializerDoesNotExist):
             serializers.get_deserializer("nonsense")
 
+    def test_json_deserializer_exception(self):
+        with self.assertRaises(DeserializationError):
+            data = serializers.deserialize("json", """[{"pk":1}""")
+            for obj in data:
+                pass
+
 def serializerTest(format, self):
 
     # Create all the objects defined in the test data