From 5692dcf9a017a80a58e4bd18af1968a2c8e5b204 Mon Sep 17 00:00:00 2001 From: James Prestwood Date: Tue, 19 Jul 2022 11:52:21 -0700 Subject: [PATCH] test-runner: allow infinite process wait subprocess.Popen's wait() method was overwritten to be non-blocking but in certain circumstances you do want to wait forever. Fix this to allow timeout=None, which calls the parent wait() method directly. --- tools/utils.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tools/utils.py b/tools/utils.py index bc030230..7a182848 100644 --- a/tools/utils.py +++ b/tools/utils.py @@ -246,6 +246,10 @@ class Process(subprocess.Popen): # Override wait() so it can do so non-blocking def wait(self, timeout=10): + if timeout == None: + super().wait() + return + Namespace.non_block_wait(self.__wait, timeout, 1) self._cleanup()