diff Lib/test/test_threading.py @ 59200:eabfa778b076 legacy-trunk

Issue #7481: When a threading.Thread failed to start it would leave the instance stuck in initial state and present in threading.enumerate().
author Gregory P. Smith <greg@mad-scientist.com>
date Sun, 28 Feb 2010 18:36:09 +0000
parents 6ffcdafa0118
children c45805ba323f
line wrap: on
line diff
--- a/Lib/test/test_threading.py	Sun Feb 28 18:26:37 2010 +0000
+++ b/Lib/test/test_threading.py	Sun Feb 28 18:36:09 2010 +0000
@@ -249,6 +249,25 @@
             t.join()
         # else the thread is still running, and we have no way to kill it
 
+    def test_limbo_cleanup(self):
+        # Issue 7481: Failure to start thread should cleanup the limbo map.
+        def fail_new_thread(*args):
+            raise thread.error()
+        _start_new_thread = threading._start_new_thread
+        threading._start_new_thread = fail_new_thread
+        try:
+            t = threading.Thread(target=lambda: None)
+            try:
+                t.start()
+                assert False
+            except thread.error:
+                self.assertFalse(
+                    t in threading._limbo,
+                    "Failed to cleanup _limbo map on failure of Thread.start()."
+                )
+        finally:
+            threading._start_new_thread = _start_new_thread
+
     def test_finalize_runnning_thread(self):
         # Issue 1402: the PyGILState_Ensure / _Release functions may be called
         # very late on python exit: on deallocation of a running thread for