From: Victor Stinner <victor.stinner@gmail.com>
Date: Wed, 9 Aug 2017 15:44:19 +0000 (+0200)
Subject: bpo-31160: Fix test_builtin for zombie process (#3043)
X-Git-Tag: v3.7.0a1~300
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=4baca1b0f7325032598cd38e7ceffc79b616d255;p=thirdparty%2FPython%2Fcpython.git

bpo-31160: Fix test_builtin for zombie process (#3043)

PtyTests.run_child() now calls os.waitpid() to read the exit status
of the child process to avoid creating zombie process and leaking
processes in the background.
---

diff --git a/Lib/test/test_builtin.py b/Lib/test/test_builtin.py
index a83d20f43439..219a08abd3ff 100644
--- a/Lib/test/test_builtin.py
+++ b/Lib/test/test_builtin.py
@@ -1568,6 +1568,10 @@ class PtyTests(unittest.TestCase):
             self.fail("got %d lines in pipe but expected 2, child output was:\n%s"
                       % (len(lines), child_output))
         os.close(fd)
+
+        pid, status = os.waitpid(pid, 0)
+        self.assertEqual(status, 0)
+
         return lines
 
     def check_input_tty(self, prompt, terminal_input, stdio_encoding=None):