From: Victor Stinner <victor.stinner@gmail.com>
Date: Wed, 13 Sep 2017 00:05:53 +0000 (-0700)
Subject: bpo-31249: Fix test_concurrent_futures dangling thread (#3521)
X-Git-Tag: v3.7.0a1~65
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=3bcf157c115ba3e48bce62ac8cb13c703475a113;p=python

bpo-31249: Fix test_concurrent_futures dangling thread (#3521)

ProcessPoolShutdownTest.test_del_shutdown() now closes the call queue
and joins its thread, to prevent leaking a dangling thread.
---

diff --git a/Lib/test/test_concurrent_futures.py b/Lib/test/test_concurrent_futures.py
index a888dcacc4..7bc733efb1 100644
--- a/Lib/test/test_concurrent_futures.py
+++ b/Lib/test/test_concurrent_futures.py
@@ -222,11 +222,14 @@ class ProcessPoolShutdownTest(ProcessPoolMixin, ExecutorShutdownTest, BaseTestCa
         list(executor.map(abs, range(-5, 5)))
         queue_management_thread = executor._queue_management_thread
         processes = executor._processes
+        call_queue = executor._call_queue
         del executor
 
         queue_management_thread.join()
         for p in processes.values():
             p.join()
+        call_queue.close()
+        call_queue.join_thread()
 
 
 class WaitTests: