From: Antoine Pitrou <solipsis@pitrou.net>
Date: Sat, 24 Apr 2010 11:13:53 +0000 (+0000)
Subject: Merged revisions 80434 via svnmerge from
X-Git-Tag: v3.2a1~1023
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e322024449dddfa9f5d3341e8d6fc370b7918b64;p=python

Merged revisions 80434 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk

........
  r80434 | antoine.pitrou | 2010-04-24 12:43:57 +0200 (sam., 24 avril 2010) | 5 lines

  Make test_makefile_close a networked test (can't read() from a non-connected
  socket under OS X), and skip it under Windows (where sockets can't be read()
  from using their fds).
........
---

diff --git a/Lib/test/test_ssl.py b/Lib/test/test_ssl.py
index 53b71b9630..2b1ba8a0ce 100644
--- a/Lib/test/test_ssl.py
+++ b/Lib/test/test_ssl.py
@@ -151,22 +151,6 @@ class BasicTests(unittest.TestCase):
         del ss
         self.assertEqual(wr(), None)
 
-    def test_makefile_close(self):
-        # Issue #5238: creating a file-like object with makefile() shouldn't
-        # leak the underlying file descriptor.
-        ss = ssl.wrap_socket(socket.socket(socket.AF_INET))
-        fd = ss.fileno()
-        f = ss.makefile()
-        f.close()
-        # The fd is still open
-        os.read(fd, 0)
-        # Closing the SSL socket should close the fd too
-        ss.close()
-        gc.collect()
-        with self.assertRaises(OSError) as e:
-            os.read(fd, 0)
-        self.assertEqual(e.exception.errno, errno.EBADF)
-
 
 class NetworkedTests(unittest.TestCase):
 
@@ -200,6 +184,25 @@ class NetworkedTests(unittest.TestCase):
         finally:
             s.close()
 
+    @unittest.skipIf(os.name == "nt", "Can't use a socket as a file under Windows")
+    def test_makefile_close(self):
+        # Issue #5238: creating a file-like object with makefile() shouldn't
+        # delay closing the underlying "real socket" (here tested with its
+        # file descriptor, hence skipping the test under Windows).
+        ss = ssl.wrap_socket(socket.socket(socket.AF_INET))
+        ss.connect(("svn.python.org", 443))
+        fd = ss.fileno()
+        f = ss.makefile()
+        f.close()
+        # The fd is still open
+        os.read(fd, 0)
+        # Closing the SSL socket should close the fd too
+        ss.close()
+        gc.collect()
+        with self.assertRaises(OSError) as e:
+            os.read(fd, 0)
+        self.assertEqual(e.exception.errno, errno.EBADF)
+
     def testNonBlockingHandshake(self):
         s = socket.socket(socket.AF_INET)
         s.connect(("svn.python.org", 443))