]> granicus.if.org Git - python/commitdiff
bpo-33023: Fix NotImplemented to NotImplementedError. (GH-10934)
authorSerhiy Storchaka <storchaka@gmail.com>
Thu, 6 Dec 2018 20:36:55 +0000 (22:36 +0200)
committerGitHub <noreply@github.com>
Thu, 6 Dec 2018 20:36:55 +0000 (22:36 +0200)
Lib/idlelib/debugger_r.py
Lib/ssl.py
Lib/test/test_ssl.py

index 01a3bd25998fc04ba1bc4acc7814f5b9d0fbf142..0e6dcfbd12c2ab781abe4810b548b9cd9f2cd8d9 100644 (file)
@@ -157,7 +157,7 @@ class IdbAdapter:
     #----------called by a DictProxy----------
 
     def dict_keys(self, did):
-        raise NotImplemented("dict_keys not public or pickleable")
+        raise NotImplementedError("dict_keys not public or pickleable")
 ##         dict = dicttable[did]
 ##         return dict.keys()
 
index 8f6d402209b15f26a7cb1acc91cade81fb538fdd..e6e3a6d0fa8d186e598d22ff12a7461c5d1bd539 100644 (file)
@@ -884,8 +884,8 @@ class SSLSocket(socket):
             return self._sslobj.session_reused
 
     def dup(self):
-        raise NotImplemented("Can't dup() %s instances" %
-                             self.__class__.__name__)
+        raise NotImplementedError("Can't dup() %s instances" %
+                                  self.__class__.__name__)
 
     def _checkClosed(self, msg=None):
         # raise an exception here if you wish to check for spurious closes
index 74a91f6c23961e2a8d990ff367e8903e35f2df06..7f6b93148f452769685f31c64597135662d28181 100644 (file)
@@ -463,8 +463,12 @@ class BasicSocketTests(unittest.TestCase):
             self.assertRaises(OSError, ss.recvfrom_into, bytearray(b'x'), 1)
             self.assertRaises(OSError, ss.send, b'x')
             self.assertRaises(OSError, ss.sendto, b'x', ('0.0.0.0', 0))
+            self.assertRaises(NotImplementedError, ss.dup)
             self.assertRaises(NotImplementedError, ss.sendmsg,
                               [b'x'], (), 0, ('0.0.0.0', 0))
+            self.assertRaises(NotImplementedError, ss.recvmsg, 100)
+            self.assertRaises(NotImplementedError, ss.recvmsg_into,
+                              [bytearray(100)])
 
     def test_timeout(self):
         # Issue #8524: when creating an SSL socket, the timeout of the
@@ -3382,10 +3386,11 @@ class ThreadedTests(unittest.TestCase):
             # Make sure sendmsg et al are disallowed to avoid
             # inadvertent disclosure of data and/or corruption
             # of the encrypted data stream
+            self.assertRaises(NotImplementedError, s.dup)
             self.assertRaises(NotImplementedError, s.sendmsg, [b"data"])
             self.assertRaises(NotImplementedError, s.recvmsg, 100)
             self.assertRaises(NotImplementedError,
-                              s.recvmsg_into, bytearray(100))
+                              s.recvmsg_into, [bytearray(100)])
             s.write(b"over\n")
 
             self.assertRaises(ValueError, s.recv, -1)