From 58de6ee8711a079da0b555221ae739d6a69df2f7 Mon Sep 17 00:00:00 2001 From: Nadeem Vawda Date: Tue, 19 Apr 2011 01:38:47 +0200 Subject: Fix sporadic failure in test_startfile. Wait for the child process to terminate before ending the test, so that the regrtest cleanup code doesn't get an error when it tries to delete the temporary CWD. --- Lib/test/test_startfile.py | 5 +++++ Misc/NEWS | 2 ++ 2 files changed, 7 insertions(+) diff --git a/Lib/test/test_startfile.py b/Lib/test/test_startfile.py index 7a003eb..dd505bf 100644 --- a/Lib/test/test_startfile.py +++ b/Lib/test/test_startfile.py @@ -11,6 +11,7 @@ import unittest from test import support import os from os import path +from time import sleep startfile = support.get_attribute(os, 'startfile') @@ -23,6 +24,10 @@ class TestCase(unittest.TestCase): empty = path.join(path.dirname(__file__), "empty.vbs") startfile(empty) startfile(empty, "open") + # Give the child process some time to exit before we finish. + # Otherwise the cleanup code will not be able to delete the cwd, + # because it is still in use. + sleep(0.1) def test_main(): support.run_unittest(TestCase) diff --git a/Misc/NEWS b/Misc/NEWS index 022ad20..8063d02 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -315,6 +315,8 @@ Build Tests ----- +- Fix test_startfile to wait for child process to terminate before finishing. + - Issue #11719: Fix message about unexpected test_msilib skip on non-Windows platforms. Patch by Nadeem Vawda. -- cgit v0.12