From 06b308a7eb090d13b06dde2b07d83a5dae166545 Mon Sep 17 00:00:00 2001 From: Serhiy Storchaka Date: Fri, 17 Sep 2021 21:56:41 +0300 Subject: [PATCH] bpo-45187: Fix dangling threads in test_socket.CreateServerFunctionalTest (GH-28422) (cherry picked from commit 51ebb7f4f5e9bdcf8279a1d91be9569706f6bead) Co-authored-by: Serhiy Storchaka --- 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 54adf103e7e338..5712b46f7f111a 100755 --- a/Lib/test/test_socket.py +++ b/Lib/test/test_socket.py @@ -6482,13 +6482,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: @@ -6502,8 +6495,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):