summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKieran Bingham <kieran.bingham@ideasonboard.com>2019-09-10 09:52:30 +0100
committerKieran Bingham <kieran.bingham@ideasonboard.com>2019-09-13 09:21:40 +0100
commitff09b6d3788731eadc31b6f73252243577b3d3d3 (patch)
tree4f4d16637f3ec0a54917fefc7f202280b9b0b7a3
parent122892d1bccf0495579918a4fcdf74a793e9fb36 (diff)
test: process: Extend timeout duration
The process test runs for just 100mS. The spawned process runs for at least 50mS. Ordinarily this should allow plenty of time for both the process to be spawned and run, but when adding extra debug instrumentation, the processes can be slowed down, leading to a false negative test failure. Extend the timeout to 2 seconds to allow the short process to be run correctly - but use the now initialised exitStatus_ to exit the event loop as soon as the process has completed. Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
-rw-r--r--test/process/process_test.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/test/process/process_test.cpp b/test/process/process_test.cpp
index f3cabe0a..7e7b3c2c 100644
--- a/test/process/process_test.cpp
+++ b/test/process/process_test.cpp
@@ -56,8 +56,8 @@ protected:
return TestFail;
}
- timeout.start(100);
- while (timeout.isRunning())
+ timeout.start(2000);
+ while (timeout.isRunning() && exitStatus_ == Process::NotExited)
dispatcher->processEvents();
if (exitStatus_ != Process::NormalExit) {