Add a timeout to test_sslv2_fail so it passes on cygwin.

bdarnell · bdarnell · commit 463baf434793 · 2012-01-22T18:15:01.000-08:00
diff --git a/tornado/test/httpserver_test.py b/tornado/test/httpserver_test.py
@@ -113,7 +113,12 @@ def test_sslv2_fail(self):
             # we've got all the other ssl version tests here.
             # Clients should have SSLv2 disabled by default.
             try:
-                response = self.fetch('/')
+                # The server simply closes the connection when it gets
+                # an SSLv2 ClientHello packet.
+                # request_timeout is needed here because on some platforms
+                # (cygwin, but not native windows python), the close is not
+                # detected promptly.
+                response = self.fetch('/', request_timeout=1)
             except ssl.SSLError:
                 # In some python/ssl builds the PROTOCOL_SSLv2 constant
                 # exists but SSLv2 support is still compiled out, which
