[Python-checkins] cpython (2.7): Issue #26309: Rewrite test in main thread and avoid race condition
martin.panter
python-checkins at python.org
Thu Feb 18 21:43:34 EST 2016
https://hg.python.org/cpython/rev/cba717fa8e10
changeset: 100262:cba717fa8e10
branch: 2.7
parent: 100259:e0fbd25f0b36
user: Martin Panter <vadmium+py at gmail.com>
date: Fri Feb 19 02:16:42 2016 +0000
summary:
Issue #26309: Rewrite test in main thread and avoid race condition
files:
Lib/test/test_socketserver.py | 23 ++++++++++++-----------
1 files changed, 12 insertions(+), 11 deletions(-)
diff --git a/Lib/test/test_socketserver.py b/Lib/test/test_socketserver.py
--- a/Lib/test/test_socketserver.py
+++ b/Lib/test/test_socketserver.py
@@ -326,28 +326,29 @@
SocketServer.TCPServer((HOST, -1),
SocketServer.StreamRequestHandler)
+
+class MiscTestCase(unittest.TestCase):
+
def test_shutdown_request_called_if_verify_request_false(self):
# Issue #26309: BaseServer should call shutdown_request even if
# verify_request is False
- result = {"shutdown_called": False}
class MyServer(SocketServer.TCPServer):
def verify_request(self, request, client_address):
return False
+ shutdown_called = 0
def shutdown_request(self, request):
- result["shutdown_called"] = True
+ self.shutdown_called += 1
SocketServer.TCPServer.shutdown_request(self, request)
- def connect_to_server(proto, addr):
- s = socket.socket(proto, socket.SOCK_STREAM)
- s.connect(addr)
- s.close()
-
- self.run_server(MyServer,
- SocketServer.StreamRequestHandler,
- connect_to_server)
- self.assertEqual(result["shutdown_called"], True)
+ server = MyServer((HOST, 0), SocketServer.StreamRequestHandler)
+ s = socket.socket(server.address_family, socket.SOCK_STREAM)
+ s.connect(server.server_address)
+ s.close()
+ server.handle_request()
+ self.assertEqual(server.shutdown_called, 1)
+ server.server_close()
def test_main():
--
Repository URL: https://hg.python.org/cpython
More information about the Python-checkins
mailing list