Skip to content

Commit 44cd86b

Browse files
corona10miss-islington
authored andcommitted
bpo-37805: Add tests for json.dump(..., skipkeys=True) (GH-15489)
https://bugs.python.org/issue37805 Automerge-Triggered-By: @methane
1 parent 6b16d93 commit 44cd86b

File tree

2 files changed

+11
-0
lines changed

2 files changed

+11
-0
lines changed

Lib/test/test_json/test_dump.py

+10
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,16 @@ def test_dump(self):
1212
def test_dumps(self):
1313
self.assertEqual(self.dumps({}), '{}')
1414

15+
def test_dump_skipkeys(self):
16+
v = {b'invalid_key': False, 'valid_key': True}
17+
with self.assertRaises(TypeError):
18+
self.json.dumps(v)
19+
20+
s = self.json.dumps(v, skipkeys=True)
21+
o = self.json.loads(s)
22+
self.assertIn('valid_key', o)
23+
self.assertNotIn(b'invalid_key', o)
24+
1525
def test_encode_truefalse(self):
1626
self.assertEqual(self.dumps(
1727
{True: False, False: True}, sort_keys=True),
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Add tests for json.dump(..., skipkeys=True). Patch by Dong-hee Na.

0 commit comments

Comments
 (0)