From: Ezio Melotti <ezio.melotti@gmail.com>
Date: Mon, 4 Mar 2013 13:17:56 +0000 (+0200)
Subject: #17346: make sure pickle tests are run against all protocols.  Initial patch by Mariu... 
X-Git-Tag: v3.2.4rc1~60
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=aaef34483c294c79dfc4e96f3168eea0f6b0e3f8;p=python

#17346: make sure pickle tests are run against all protocols.  Initial patch by Marius Gedminas.
---

diff --git a/Lib/test/pickletester.py b/Lib/test/pickletester.py
index 4d491b0173..1ce01c3d07 100644
--- a/Lib/test/pickletester.py
+++ b/Lib/test/pickletester.py
@@ -577,10 +577,10 @@ class AbstractPickleTests(unittest.TestCase):
         i = C()
         i.attr = i
         for proto in protocols:
-            s = self.dumps(i, 2)
+            s = self.dumps(i, proto)
             x = self.loads(s)
             self.assertEqual(dir(x), dir(i))
-            self.assertTrue(x.attr is x)
+            self.assertIs(x.attr, x)
 
     def test_recursive_multi(self):
         l = []
@@ -637,13 +637,13 @@ class AbstractPickleTests(unittest.TestCase):
     def test_bytes(self):
         for proto in protocols:
             for s in b'', b'xyz', b'xyz'*100:
-                p = self.dumps(s)
+                p = self.dumps(s, proto)
                 self.assertEqual(self.loads(p), s)
             for s in [bytes([i]) for i in range(256)]:
-                p = self.dumps(s)
+                p = self.dumps(s, proto)
                 self.assertEqual(self.loads(p), s)
             for s in [bytes([i, i]) for i in range(256)]:
-                p = self.dumps(s)
+                p = self.dumps(s, proto)
                 self.assertEqual(self.loads(p), s)
 
     def test_ints(self):