]> granicus.if.org Git - python/commitdiff
bpo-32618: Fix test_mutatingdecodehandler not testing test.mutating (#5269)
authorXiang Zhang <angwerzx@126.com>
Tue, 23 Jan 2018 14:50:50 +0000 (22:50 +0800)
committerGitHub <noreply@github.com>
Tue, 23 Jan 2018 14:50:50 +0000 (22:50 +0800)
* bpo-32618: Fix test_mutatingdecodehandler not testing test.mutating

It should test both test.replacing and test.mutating instead of test test.replacing twice.

Lib/test/test_codeccallbacks.py

index 6a3e993265687bfc83fe0c95ee67afd7f03f548a..0c066e633fa846d28218caaa02dd6ec2cda2eefa 100644 (file)
@@ -1032,7 +1032,7 @@ class CodecCallbackTest(unittest.TestCase):
 
         def mutating(exc):
             if isinstance(exc, UnicodeDecodeError):
-                exc.object[:] = b""
+                exc.object = b""
                 return ("\u4242", 0)
             else:
                 raise TypeError("don't know how to handle %r" % exc)
@@ -1042,8 +1042,7 @@ class CodecCallbackTest(unittest.TestCase):
         with test.support.check_warnings():
             # unicode-internal has been deprecated
             for (encoding, data) in baddata:
-                with self.assertRaises(TypeError):
-                    data.decode(encoding, "test.replacing")
+                self.assertEqual(data.decode(encoding, "test.mutating"), "\u4242")
 
     def test_fake_error_class(self):
         handlers = [