From 14553b08a1f93bb4ec8f8c2239ef240f634cc2da Mon Sep 17 00:00:00 2001 From: Facundo Batista Date: Fri, 23 Mar 2007 20:23:08 +0000 Subject: [PATCH] Surrounded with try/finally to socket's default timeout setting changes in the tests, so failing one test won't produce strange results in others. Also relaxed the timeout settings in the test (where actually the value didn't mean anything). --- Lib/test/test_httplib.py | 16 +++++++++------- Lib/test/test_socket.py | 20 +++++++++++--------- 2 files changed, 20 insertions(+), 16 deletions(-) diff --git a/Lib/test/test_httplib.py b/Lib/test/test_httplib.py index a39a3eb501..78aa6519b8 100644 --- a/Lib/test/test_httplib.py +++ b/Lib/test/test_httplib.py @@ -176,17 +176,19 @@ class TimeoutTest(TestCase): self.assertTrue(httpConn.sock.gettimeout() is None) # a value - httpConn = httplib.HTTPConnection(HOST, PORT, timeout=10) + httpConn = httplib.HTTPConnection(HOST, PORT, timeout=30) httpConn.connect() - self.assertEqual(httpConn.sock.gettimeout(), 10) + self.assertEqual(httpConn.sock.gettimeout(), 30) # None, having other default previous = socket.getdefaulttimeout() - socket.setdefaulttimeout(10) - httpConn = httplib.HTTPConnection(HOST, PORT, timeout=None) - httpConn.connect() - socket.setdefaulttimeout(previous) - self.assertEqual(httpConn.sock.gettimeout(), 10) + socket.setdefaulttimeout(30) + try: + httpConn = httplib.HTTPConnection(HOST, PORT, timeout=None) + httpConn.connect() + finally: + socket.setdefaulttimeout(previous) + self.assertEqual(httpConn.sock.gettimeout(), 30) def test_main(verbose=None): diff --git a/Lib/test/test_socket.py b/Lib/test/test_socket.py index a7ebe04496..ade524f807 100644 --- a/Lib/test/test_socket.py +++ b/Lib/test/test_socket.py @@ -844,22 +844,24 @@ class NetworkConnectionAttributesTest(unittest.TestCase): self.assertTrue(sock.gettimeout() is None) # a value, named - sock = socket.create_connection((HOST, PORT), timeout=10) - self.assertEqual(sock.gettimeout(), 10) + sock = socket.create_connection((HOST, PORT), timeout=30) + self.assertEqual(sock.gettimeout(), 30) # a value, just the value - sock = socket.create_connection((HOST, PORT), 10) - self.assertEqual(sock.gettimeout(), 10) + sock = socket.create_connection((HOST, PORT), 30) + self.assertEqual(sock.gettimeout(), 30) # None, having other default previous = socket.getdefaulttimeout() - socket.setdefaulttimeout(10) - sock = socket.create_connection((HOST, PORT), timeout=None) - socket.setdefaulttimeout(previous) - self.assertEqual(sock.gettimeout(), 10) + socket.setdefaulttimeout(30) + try: + sock = socket.create_connection((HOST, PORT), timeout=None) + finally: + socket.setdefaulttimeout(previous) + self.assertEqual(sock.gettimeout(), 30) def testFamily(self): - sock = socket.create_connection((HOST, PORT), timeout=10) + sock = socket.create_connection((HOST, PORT), timeout=30) self.assertEqual(sock.family, 2) -- 2.50.1