]> granicus.if.org Git - python/commitdiff
Merged revisions 84021-84023 via svnmerge from
authorAntoine Pitrou <solipsis@pitrou.net>
Sat, 14 Aug 2010 17:26:19 +0000 (17:26 +0000)
committerAntoine Pitrou <solipsis@pitrou.net>
Sat, 14 Aug 2010 17:26:19 +0000 (17:26 +0000)
svn+ssh://pythondev@svn.python.org/python/branches/py3k

........
  r84021 | antoine.pitrou | 2010-08-14 18:24:38 +0200 (sam., 14 août 2010) | 4 lines

  Add a reasonable timeout to network SSL tests, so as to avoid buildbot timeouts
  when the remote server doesn't answer.
........
  r84022 | antoine.pitrou | 2010-08-14 18:32:10 +0200 (sam., 14 août 2010) | 4 lines

  This is a better resolution than r84021 (because it will also affect
  ssl.get_server_certificate()).
........
  r84023 | antoine.pitrou | 2010-08-14 18:33:38 +0200 (sam., 14 août 2010) | 3 lines

  There doesn't seem to be a reason for this test to be disabled.
........

Lib/test/test_ssl.py

index 0165de86b92c704dc868619557a777d43e2b8a76..193978cb3d5bdcb2ff30b82d8c3b61aef1b684d1 100644 (file)
@@ -103,6 +103,12 @@ class BasicTests(unittest.TestCase):
 
 
 class NetworkedTests(unittest.TestCase):
+    def setUp(self):
+        self.old_timeout = socket.getdefaulttimeout()
+        socket.setdefaulttimeout(30)
+
+    def tearDown(self):
+        socket.setdefaulttimeout(self.old_timeout)
 
     def test_connect(self):
         s = ssl.wrap_socket(socket.socket(socket.AF_INET),
@@ -183,8 +189,6 @@ class NetworkedTests(unittest.TestCase):
         if not pem:
             self.fail("No server certificate on svn.python.org:443!")
 
-        return
-
         try:
             pem = ssl.get_server_certificate(("svn.python.org", 443), ca_certs=CERTFILE)
         except ssl.SSLError as x: