diff options
| author | Brad King <brad.king@kitware.com> | 2018-09-09 15:11:38 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2018-09-10 11:32:16 (GMT) |
| commit | 292ec157b67b5570d5bc2e00bba554cc157d9dae (patch) | |
| tree | 47976098727357d9d9117ae3f320940ec8edd670 /Tests/RunCMake/ctest_test/CTestTestLoadFail-stdout.txt | |
| parent | f478fa633daeb1432805821adddc40730ffd283d (diff) | |
| download | CMake-292ec157b67b5570d5bc2e00bba554cc157d9dae.zip CMake-292ec157b67b5570d5bc2e00bba554cc157d9dae.tar.gz CMake-292ec157b67b5570d5bc2e00bba554cc157d9dae.tar.bz2 | |
CTest: Fix --test-load regression
The `ctest --test-load` option is implemented in `StartNextTests` by not
starting any tests when the load is too high and instead sleeping and
then returning. Prior to commit v3.11.0-rc1~117^2 (CTest: Re-implement
test process handling using libuv, 2017-12-10) our outer loop in
`RunTests` would immediately call `StartNextTests` again. However, now
the `uv_run` loop may simply terminate if there are no tests running
because no events are left pending.
Fix this by converting the sleep in `StartNextTests` into a libuv timer
that it starts instead. This avoids leaving `uv_run` with no pending
events. In the case that there are other running tests this also allows
CTest to detect when they finish even if it during the wait period where
we previously slept.
This regression was not caught by the test suite because it only
verified that we do not start new tests when the load was too high and
not that we proceed to start tests when the load drops. Revise the test
suite to cover both.
Fixes: #18338
Diffstat (limited to 'Tests/RunCMake/ctest_test/CTestTestLoadFail-stdout.txt')
| -rw-r--r-- | Tests/RunCMake/ctest_test/CTestTestLoadFail-stdout.txt | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/Tests/RunCMake/ctest_test/CTestTestLoadFail-stdout.txt b/Tests/RunCMake/ctest_test/CTestTestLoadFail-stdout.txt deleted file mode 100644 index e203c10..0000000 --- a/Tests/RunCMake/ctest_test/CTestTestLoadFail-stdout.txt +++ /dev/null @@ -1,2 +0,0 @@ -Test project .*/Tests/RunCMake/ctest_test/CTestTestLoadFail-build -\*\*\*\*\* WAITING, System Load: 5, Max Allowed Load: 4, Smallest test RunCMakeVersion requires 1\*\*\*\*\*$ |