diff options
author | Antoine Pitrou <antoine@python.org> | 2020-04-20 19:22:50 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-04-20 19:22:50 (GMT) |
commit | 887ff8e37e238fbce18c647e588283904f38ab24 (patch) | |
tree | 51e36929d3e7dcf3b89c54054028faf664bc4cf9 /Lib/test | |
parent | 81de3c225774179cdc82a1733a64e4a876ff02b5 (diff) | |
download | cpython-887ff8e37e238fbce18c647e588283904f38ab24.zip cpython-887ff8e37e238fbce18c647e588283904f38ab24.tar.gz cpython-887ff8e37e238fbce18c647e588283904f38ab24.tar.bz2 |
[3.8] bpo-40330: Fix utf-8 size check in ShareableList (GH-19606) (GH-19625)
The item size must be checked after encoding to bytes, not before.
Automerge-Triggered-By: @pitrou.
(cherry picked from commit eba9f6155df59c9beed97fb5764c9f01dd941af0)
Co-authored-by: Antoine Pitrou <antoine@python.org>
Diffstat (limited to 'Lib/test')
-rw-r--r-- | Lib/test/_test_multiprocessing.py | 16 |
1 files changed, 14 insertions, 2 deletions
diff --git a/Lib/test/_test_multiprocessing.py b/Lib/test/_test_multiprocessing.py index e64a8e9..d5336e4 100644 --- a/Lib/test/_test_multiprocessing.py +++ b/Lib/test/_test_multiprocessing.py @@ -3964,9 +3964,21 @@ class _TestSharedMemory(BaseTestCase): sl[4] = 'some' # Change type at a given position. self.assertEqual(sl[4], 'some') self.assertEqual(sl.format, '8s8sdq8sxxxxxxx?q') - with self.assertRaises(ValueError): - sl[4] = 'far too many' # Exceeds available storage. + with self.assertRaisesRegex(ValueError, + "exceeds available storage"): + sl[4] = 'far too many' self.assertEqual(sl[4], 'some') + sl[0] = 'encodés' # Exactly 8 bytes of UTF-8 data + self.assertEqual(sl[0], 'encodés') + self.assertEqual(sl[1], b'HoWdY') # no spillage + with self.assertRaisesRegex(ValueError, + "exceeds available storage"): + sl[0] = 'encodées' # Exactly 9 bytes of UTF-8 data + self.assertEqual(sl[1], b'HoWdY') + with self.assertRaisesRegex(ValueError, + "exceeds available storage"): + sl[1] = b'123456789' + self.assertEqual(sl[1], b'HoWdY') # Exercise count(). with warnings.catch_warnings(): |