diff options
| author | Kieran Bingham <kieran.bingham@ideasonboard.com> | 2019-09-10 09:52:30 +0100 |
|---|---|---|
| committer | Kieran Bingham <kieran.bingham@ideasonboard.com> | 2019-09-13 09:21:40 +0100 |
| commit | ff09b6d3788731eadc31b6f73252243577b3d3d3 (patch) | |
| tree | 4f4d16637f3ec0a54917fefc7f202280b9b0b7a3 | |
| parent | 122892d1bccf0495579918a4fcdf74a793e9fb36 (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.cpp | 4 |
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) { |