summaryrefslogtreecommitdiffstats
path: root/Source/CTest
Commit message (Collapse)AuthorAgeFilesLines
* clang-tidy: Use default member initializationRegina Pfeifer2018-12-152-12/+5
|
* clang-tidy: Remove redundant member initializationsRegina Pfeifer2018-12-152-4/+1
|
* Merge topic 'submit-method'Brad King2018-12-074-771/+6
|\ | | | | | | | | | | | | | | | | | | | | | | | | 9501a6145d CTest: Remove special case for Dart version <= 1 e184603378 CTest: Add release note about removed submission methods 8c25aa2feb CTest: Remove submission trigger a688c6b928 CTest: Remove submit method 'ftp' c339b64cf2 CTest: Remove submit method 'scp' aebdad399e CTest: Remove submit method 'cp' c2f6872c4e CTest: Remove submit method 'xmlrpc' Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2672
| * CTest: Remove special case for Dart version <= 1Regina Pfeifer2018-11-282-16/+1
| |
| * CTest: Remove submission triggerRegina Pfeifer2018-11-283-167/+0
| |
| * CTest: Remove submit method 'ftp'Regina Pfeifer2018-11-282-239/+5
| |
| * CTest: Remove submit method 'scp'Regina Pfeifer2018-11-283-139/+0
| |
| * CTest: Remove submit method 'cp'Regina Pfeifer2018-11-282-68/+0
| |
| * CTest: Remove submit method 'xmlrpc'Regina Pfeifer2018-11-282-142/+0
| |
* | cmSystemTools: Add EncodeURL helperBrad King2018-11-281-20/+1
|/ | | | | | | Factor a URL encoding implementation out of CTest. Add an option to not escape slashes. Suggested-by: Daniel Pfeifer <daniel@pfeifer-mail.de>
* Merge topic 'readability-static-accessed-through-instance'Brad King2018-11-286-11/+14
|\ | | | | | | | | | | | | 3e60580784 clang-tidy: Fix readability-static-accessed-through-instance Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2647
| * clang-tidy: Fix readability-static-accessed-through-instanceRegina Pfeifer2018-11-216-11/+14
| | | | | | | | Enable the check in .clang-tidy and fix all warnings.
* | Merge topic 'ctest-updates'Brad King2018-11-273-23/+12
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | d9195ab081 Tests: Teach run_ctest to handle removal of CTestConfig.cmake 1ca53f5ef1 Remove unnecessary CTEST_PROJECT_NAME variables 15ac4aae0e Remove warning when no CTestConfig.cmake file exists 3125c47d27 ctest_build: Do not require unnecessary [CTEST_]PROJECT_NAME value 36bbd07a76 CDashUpload: Use the query part of the submit url as field e1dfe8cee6 CTest: Don't require 'submit.php?' in submit location Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2640
| * | ctest_build: Do not require unnecessary [CTEST_]PROJECT_NAME valueRegina Pfeifer2018-11-201-14/+2
| | | | | | | | | | | | | | | | | | | | | | | | Since commit v3.0.0-rc1~260^2~32 (ctest_build: Use "cmake --build" to launch the native build tool, 2013-11-14) we no longer need to use the project name in `ctest_build()`. Fixes: #18612
| * | CDashUpload: Use the query part of the submit url as fieldRegina Pfeifer2018-11-202-7/+6
| | |
| * | CTest: Don't require 'submit.php?' in submit locationRegina Pfeifer2018-11-201-2/+4
| |/ | | | | | | Fixes: #18611
* | clang-tidy: Enable checks from bugprone setRegina Pfeifer2018-11-211-2/+2
|/
* Merge topic 'ctest-stdin'Brad King2018-11-191-1/+2
|\ | | | | | | | | | | | | | | 86e8315482 CTest: Restore inheritance of stdin by test processes bdec3bd896 Tests: Teach RunCMake infrastructure to optionally provide stdin Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2618
| * CTest: Restore inheritance of stdin by test processesBrad King2018-11-151-1/+2
| | | | | | | | | | | | | | | | | | | | | | Since commit v3.11.0-rc1~117^2 (CTest: Re-implement test process handling using libuv, 2017-12-10) we do not give the child test processes any stdin. Prior to that change we let the child test processes inherit stdin from ctest itself. Tests that run serially might be able to use the real stdin meaningfully, so restore that behavior and add a test case. Fixes: #18591
| * Merge branch 'set_directory_properties-script-mode' into release-3.13Brad King2018-10-291-0/+1
| |\ | | | | | | | | | Merge-request: !2544
* | | CTest: Add colored output on tests summary where supportedSylvain Joubert2018-11-092-10/+31
| | | | | | | | | | | | | | | | | | | | | | | | - Number of passed/failed tests is colored according to the whole outcome - Individual listed tested are colored according to their completion status: * Disabled: blue * Failed: red * Not Run: yellow
* | | Merge topic 'set_directory_properties-script-mode'Brad King2018-10-301-0/+1
|\ \ \ | | |/ | |/| | | | | | | | | | | | | 50572d638b set_directory_properties: Restore in script mode Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2544
| * | set_directory_properties: Restore in script modeBrad King2018-10-291-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit v3.10.0-rc1~391^2~3 (Add directory property 'LABELS' and CMAKE_DIRECTORY_LABELS variable, 2017-06-23) this command was accidentally not allowed in script mode. It was dropped because `ctest -S` mode needs to start with CMake's normal script mode and then replace the `set_directory_properties` implementation. Restore the normal `set_directory_properties` in script mode and then add special logic to replace it in ctest. Also add a test case. Fixes: #18523
* | | Merge topic 'ctest-coverage-gtm-percent'Brad King2018-10-171-2/+7
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | f7d92deff4 CTest: Fix GTM coverage handling of entry point named "%" Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2489
| * | | CTest: Fix GTM coverage handling of entry point named "%"Joseph Snyder2018-10-161-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Removing the "%" character from the name of the routine in the line parser causes CTest to be unable to find a routine entry point that is only named "%". Instead leave it during line parsing and handle routine names ending in "%" explicitly when loading files.
* | | | ctest_submit: Generate Done.xml at submit timeZack Galbreath2018-10-151-1/+11
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a bug where calling `ctest_submit()` would result in a Done.xml file with an empty <buildId> element. CDash responds with a buildId when CTest submits a file. For this reason, Done.xml needs to generated after CTest has submitted some other file to CDash. This change also has the benefit of making Done.xml's timestamp as late as possible, giving us a more accurate record of how long the entire build took to complete.
* | | ctest_submit: Add support for a "Done" partBetsy McPhail2018-10-101-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Teach CTest to submit Done.xml. Submission of this file indicates to CDash that a build is complete and no more files will be uploaded. It contains the build id returned by CDash and the current time. This file is submitted last for a given build when using the `ctest_submit()` command. If submitting by PARTS, use `ctest_submit(PARTS Done)`.
* | | ctest_submit: Refactor file list to use a vector instead of a setBetsy McPhail2018-10-102-28/+35
| |/ |/| | | | | Remove duplicates in a way that preserves order.
* | Fix misc. typosluz.paz2018-10-032-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Found via `codespell -q 3 -I ../cmake-whitelist.txt --skip="./Utilities"` where the whitelist consists of ``` aci ans behaviour buil convertor dum earch ect emmited emmitted helpfull iff isnt ith lowercased mose nd nknown nto objext ot pathes pevents splitted substract superceded supercedes te tim todays uint upto whitespaces ```
* | Merge topic 'ctest-num-width'Brad King2018-10-031-2/+6
|\ \ | | | | | | | | | | | | | | | | | | 6d28884617 cmCTestRunTest: Avoid float/int conversions in number width logic Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2433
| * | cmCTestRunTest: Avoid float/int conversions in number width logicBrad King2018-10-021-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use of `std::log10` added by commit 02c5091c90 (cmCTestRunTest: Simplify number width computation, 2018-09-08) broke our number width computation on some platforms where static_cast<int>(std::log10(static_cast<size_t>(10))) somehow produces `0` instead of `1`. Re-implement the logic to avoid floating-point computations.
* | | Merge topic 'no_buildid_for_files'Brad King2018-10-011-20/+28
|\ \ \ | |/ / |/| | | | | | | | | | | | | | c49d13f94b ctest: only create buildid when submitting from Testing/ dir Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2423
| * | ctest: only create buildid when submitting from Testing/ dirZack Galbreath2018-09-281-20/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In 7f530cc we taught CTest to pass extra information to CDash at submit time. This extra info is used by CDash to initialize a buildid. `ctest_submit(FILES)` can be used to send specific files to CDash. These files are not necessarily associated with the build currently being performed. For this reason, we modify the behavior of ctest_submit() to only specify this extra info when we are submitting files from the current build's Testing directory.
* | | Merge topic 'add_progress_summary_console_logging_mode'Brad King2018-10-013-58/+111
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b3d5b8b3fb ctest: Add option for live progress summary in terminal 62fbe5002a cmCTestRunTest: Thread number of completed tests through start APIs 02c5091c90 cmCTestRunTest: Simplify number width computation 6a285bb737 cmCTestRunTest: Buffer test result output before printing Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2240
| * | | ctest: Add option for live progress summary in terminalMichael Wegner2018-09-262-18/+75
| | | |
| * | | cmCTestRunTest: Thread number of completed tests through start APIsMichael Wegner2018-09-253-7/+8
| | | |
| * | | cmCTestRunTest: Simplify number width computationMichael Wegner2018-09-251-8/+2
| | | | | | | | | | | | | | | | Use a real logarithm implementation.
| * | | cmCTestRunTest: Buffer test result output before printingMichael Wegner2018-09-251-28/+29
| | | |
* | | | ctest: Fix generator platform under --build-nocmakeBrad King2018-09-251-3/+9
| |/ / |/| | | | | | | | | | | | | | | | | | | | When constructing a global generator instance in `--build-and-test` mode we need to set the platform passed by `--build-generator-platform` directly on the generator. The old code that set it on the `cmake` class instance did nothing because that only affects cache initialization, which is not done by `--build-nocmake`.
* | | Merge topic 'ctest-more_submit_params'Brad King2018-09-191-0/+23
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 7f530cc54e ctest_submit: pass additional info to CDash Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2380
| * | | ctest_submit: pass additional info to CDashZack Galbreath2018-09-171-0/+23
| | | | | | | | | | | | | | | | | | | | Specify buildname, site, and buildstamp to CDash upon submission. CDash will use this extra info to assign and report back a buildid.
* | | | Merge topic 'getsafedef-stdstring'Brad King2018-09-181-2/+2
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | f4ff60a803 cmMakefile: Make GetSafeDefinition return std::string const& Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2350
| * | | cmMakefile: Make GetSafeDefinition return std::string const&Vitaly Stakhovsky2018-09-181-2/+2
| |/ /
* | | CTest: Improve stop-time implementationBrad King2018-09-124-18/+44
|/ / | | | | | | | | | | | | | | | | | | The CTestTestStopTime test has been failing sporadically because the stop time causes the first internal test to have a timeout short enough that we might hit it and start the second test just before the stop time is reached. Instead we should track when a timeout is shortened in order to stay within the stop time. If a test times out for this reason then we should consider the stop time reached and not start any more tests.
* | Merge topic 'ctest-fix-test-load'Brad King2018-09-112-20/+44
|\ \ | | | | | | | | | | | | | | | | | | 292ec157b6 CTest: Fix --test-load regression Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2362
| * | CTest: Fix --test-load regressionBrad King2018-09-102-20/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Remove unnecessary c_str() callsVitaly Stakhovsky2018-09-054-7/+7
| | | | | | | | | | | | Use the new IsOn(),IsOff() overloads.
* | | Merge topic 'no-sanitizer-logfile'Brad King2018-08-311-2/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 5aaaee5e9e ctest_memcheck: Change failure to find log file from error to warning Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2323
| * | | ctest_memcheck: Change failure to find log file from error to warningBetsy McPhail2018-08-281-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sanitizers do not create a log file when no defects are found. Therefore, it is currently impossible for ctest_memcheck to set both `CAPTURE_CMAKE_ERROR` and `RETURN_VALUE` to zero. With defects, `CAPTURE_CMAKE_ERROR`=0 and `RETURN_VALUE`=-1, as expected. With no defects, `CAPTURE_CMAKE_ERROR`=-1 and `RETURN_VALUE`=0.
* | | | CTest: Add error exception for sphinx-build WARNING messagesTim2018-08-301-0/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sphinx-build warnings look like this: ../path/to/file.rst:105: WARNING: This is not an error. CTest detects these as errors by this regex: ([^ :]+):([0-9]+): ([^ \\t]) Add an exception as we already have for lower-case `: warning`.