From 1dcea60d318f70573b6c6ac9e0ef76d07183c2b8 Mon Sep 17 00:00:00 2001 From: Serhiy Storchaka Date: Fri, 17 Sep 2021 21:25:54 +0300 Subject: [PATCH] bpo-45187: Fix dangling threads in test_socket.CreateServerFunctionalTest --- Lib/test/test_socket.py | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/Lib/test/test_socket.py b/Lib/test/test_socket.py index d064c6705a484f..eeb8e8c98a1494 100755 --- a/Lib/test/test_socket.py +++ b/Lib/test/test_socket.py @@ -6528,13 +6528,6 @@ def test_dualstack_ipv6_family(self): class CreateServerFunctionalTest(unittest.TestCase): timeout = support.LOOPBACK_TIMEOUT - def setUp(self): - self.thread = None - - def tearDown(self): - if self.thread is not None: - self.thread.join(self.timeout) - def echo_server(self, sock): def run(sock): with sock: @@ -6548,8 +6541,9 @@ def run(sock): event = threading.Event() sock.settimeout(self.timeout) - self.thread = threading.Thread(target=run, args=(sock, )) - self.thread.start() + thread = threading.Thread(target=run, args=(sock, )) + thread.start() + self.addCleanup(thread.join, self.timeout) event.set() def echo_client(self, addr, family):