summaryrefslogtreecommitdiffstats
path: root/Lib/json
diff options
context:
space:
mode:
authorSerhiy Storchaka <storchaka@gmail.com>2017-11-25 15:38:20 (GMT)
committerGitHub <noreply@github.com>2017-11-25 15:38:20 (GMT)
commitcfa797c0681b7fef47cf93955fd06b54ddd09a7f (patch)
tree6e4ecf5ed329d339b26f1eb2bf1f73950abbee72 /Lib/json
parent5b48dc638b7405fd9bde4d854bf477dfeaaddf44 (diff)
downloadcpython-cfa797c0681b7fef47cf93955fd06b54ddd09a7f.zip
cpython-cfa797c0681b7fef47cf93955fd06b54ddd09a7f.tar.gz
cpython-cfa797c0681b7fef47cf93955fd06b54ddd09a7f.tar.bz2
bpo-24641: Improved error message for JSON unserializible keys. (#4364)
Also updated an example for default() in the module docstring. Removed quotes around type name in other error messages.
Diffstat (limited to 'Lib/json')
-rw-r--r--Lib/json/__init__.py7
-rw-r--r--Lib/json/encoder.py7
2 files changed, 8 insertions, 6 deletions
diff --git a/Lib/json/__init__.py b/Lib/json/__init__.py
index 6d0511e..a566009 100644
--- a/Lib/json/__init__.py
+++ b/Lib/json/__init__.py
@@ -76,7 +76,8 @@ Specializing JSON object encoding::
>>> def encode_complex(obj):
... if isinstance(obj, complex):
... return [obj.real, obj.imag]
- ... raise TypeError(repr(obj) + " is not JSON serializable")
+ ... raise TypeError(f'Object of type {obj.__class__.__name__} '
+ ... f'is not JSON serializable')
...
>>> json.dumps(2 + 1j, default=encode_complex)
'[2.0, 1.0]'
@@ -344,8 +345,8 @@ def loads(s, *, encoding=None, cls=None, object_hook=None, parse_float=None,
s, 0)
else:
if not isinstance(s, (bytes, bytearray)):
- raise TypeError('the JSON object must be str, bytes or bytearray, '
- 'not {!r}'.format(s.__class__.__name__))
+ raise TypeError(f'the JSON object must be str, bytes or bytearray, '
+ f'not {s.__class__.__name__}')
s = s.decode(detect_encoding(s), 'surrogatepass')
if (cls is None and object_hook is None and
diff --git a/Lib/json/encoder.py b/Lib/json/encoder.py
index 41a497c..fb083ed 100644
--- a/Lib/json/encoder.py
+++ b/Lib/json/encoder.py
@@ -176,8 +176,8 @@ class JSONEncoder(object):
return JSONEncoder.default(self, o)
"""
- raise TypeError("Object of type '%s' is not JSON serializable" %
- o.__class__.__name__)
+ raise TypeError(f'Object of type {o.__class__.__name__} '
+ f'is not JSON serializable')
def encode(self, o):
"""Return a JSON string representation of a Python data structure.
@@ -373,7 +373,8 @@ def _make_iterencode(markers, _default, _encoder, _indent, _floatstr,
elif _skipkeys:
continue
else:
- raise TypeError("key " + repr(key) + " is not a string")
+ raise TypeError(f'keys must be str, int, float, bool or None, '
+ f'not {key.__class__.__name__}')
if first:
first = False
else: