summaryrefslogtreecommitdiffstats
path: root/Source/CTest
Commit message (Collapse)AuthorAgeFilesLines
* cmParseCacheCoverage: use cmSystemTools::SplitStringBen Boeckel2020-04-132-29/+4
|
* nits: replace some "c" instances with 'c'Ben Boeckel2020-04-132-4/+4
|
* clang-tidy: address bugprone-branch-clone lintsBen Boeckel2020-04-133-31/+18
| | | | | | | | | Arguably, many of these are bugs in `clang-tidy`. An if/else tree with other conditionals between cloned blocks may be relying on the intermediate logic to fall out of the case and inverting this logic may be non-trivial. See: https://bugs.llvm.org/show_bug.cgi?id=44165
* CTest: Add CTEST_RESOURCE_SPEC_FILE variableKyle Edwards2020-04-023-16/+30
|
* Merge topic 'cmprop-getglobalprop'Brad King2020-03-271-5/+4
|\ | | | | | | | | | | | | c84cf42897 cmState::GetGlobalProperty: return cmProp Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4521
| * cmState::GetGlobalProperty: return cmPropVitaly Stakhovsky2020-03-251-5/+4
| |
* | Merge branch 'backport-ctest-timeout-report' into ctest-timeout-reportBrad King2020-03-262-8/+14
|\ \
| * | CTest: Fix reported duration on timeout when grindchild keeps pipes openBrad King2020-03-262-8/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit d1976cd1f2 (CTest: Fix timeout when grandchild keeps pipes open, 2020-01-13, v3.17.0-rc1~169^2) we no longer hang, but the test duration we report after the timeout is the amount of time the immediate child ran before exiting. Fix the logic to instead report the actual amount of time we spent monitoring the test before the timeout. Fixes: #20509
* | | use _s to construct static string_views at several placesRolf Eike Beer2020-03-241-29/+31
| | | | | | | | | | | | This should avoid the runtime strlen() call.
* | | check for a valid URL scheme before starting to do any splittingRolf Eike Beer2020-03-231-6/+7
| | |
* | | replace std::string::substr() with operations that do not allocate memoryRolf Eike Beer2020-03-234-5/+11
| | | | | | | | | | | | | | | Modify the original string instead of creating a new copy with substr() when it is not used for anything else afterwards.
* | | replace "std::string::find(x) == 0" with cmHasPrefix()Rolf Eike Beer2020-03-232-14/+20
| | |
* | | replace "substr(0, xx) ==" with cmHasPrefix()Rolf Eike Beer2020-03-232-4/+5
| | |
* | | CTest: avoid repeated string comparesRolf Eike Beer2020-03-231-52/+27
| | | | | | | | | | | | | | | | | | Only one key can match per iteration, avoid any further compares when one match was already found. While at it entirely avoid that the key and value strings are copied.
* | | Mumps coverage: directly pass std::string as argumentRolf Eike Beer2020-03-236-9/+9
| | |
* | | remove pointless return value from cmCTestTestHandler::CleanTestOutput()Rolf Eike Beer2020-03-232-4/+3
| | |
* | | PyCoverage: avoid repeated string splitting, especially for uncovered linesRolf Eike Beer2020-03-231-20/+17
| | |
* | | use std::string::rfind() instead of open coding itRolf Eike Beer2020-03-231-12/+8
| |/ |/| | | | | While at it avoid creating a new string.
* | Merge topic 'modernize-memory-management'Brad King2020-03-126-60/+100
|\ \ | | | | | | | | | | | | | | | | | | f964739ead cmCTestRunTest: modernize memory management Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4454
| * | cmCTestRunTest: modernize memory managementMarc Chevrier2020-03-116-60/+100
| | |
* | | Merge topic 'ctest-curl-debugfunction'Brad King2020-03-122-2/+2
|\ \ \ | | |/ | |/| | | | | | | | | | | | | 7a1cce210b CTest: Fix our internal CURL_DEBUGFUNCTION to conform to CURL docs Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4459
| * | CTest: Fix our internal CURL_DEBUGFUNCTION to conform to CURL docsBetsy McPhail2020-03-112-2/+2
| | | | | | | | | | | | | | | | | | The curl debug callback function must return ``0``. Fixes: #20449
* | | Source: use std::string overloadsVitaly Stakhovsky2020-03-102-2/+2
| |/ |/|
* | Modernize memory managementMarc Chevrier2020-03-084-68/+41
|/ | | | Update internals of various classes.
* CTest: Provide more detailed information on resource allocation errorKyle Edwards2020-02-243-1/+35
|
* Refactor: Provide more detailed error information from TryAllocateResources()Kyle Edwards2020-02-242-16/+37
|
* CTest: Improve error reporting with bad working directory for testsKyle Edwards2020-02-242-2/+2
|
* Refactor: Require detail when calling cmCTestRunTest::StartFailure()Kyle Edwards2020-02-243-8/+11
|
* Add --no-tests=<[error|ignore]> option to CTestStefan Dinkelacker2020-01-311-1/+12
| | | | | | | | | If no tests were found, the default behavior of CTest is to always log an error message but to return an error code in script mode only. This option unifies the behavior of CTest by either returning an error code if no tests were found or by ignoring it. Signed-off-by: Stefan Dinkelacker <s.dinkelacker@dkfz-heidelberg.de>
* CTest: Fix timeout when grandchild keeps pipes openBrad King2020-01-141-3/+0
| | | | | | | | | | | When a test's process creates its own child and exits, the grandchild may keep pipes open. Fix CTest logic to correctly timeout if the grandchild does not exit and close the pipes before the timeout expires. This was broken by commit b5e21d7d2e (CTest: Re-implement test process handling using libuv, 2017-12-10, v3.11.0-rc1~117^2) which added an unnecessary condition to the timeout handling. Fixes: #20116
* Refactoring: suppress cmEraseIf in favor of cm::erase_ifMarc Chevrier2020-01-092-5/+6
|
* GlobalGenerator family: modernize memory managementMarc Chevrier2020-01-074-18/+10
|
* Merge topic 'ctest-resource-allocation-spec-message'Craig Scott2019-12-293-23/+81
|\ | | | | | | | | | | | | b393b32b4b CTest: Improve error handling when reading resource spec file Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4162
| * CTest: Improve error handling when reading resource spec fileKyle Edwards2019-12-273-23/+81
| | | | | | | | Fixes: #20079
* | Refactoring: use append functions from cmext/algorithmMarc Chevrier2019-12-1710-31/+43
| |
* | Merge topic 'ctest-drmemory-support'Brad King2019-12-122-6/+180
|\ \ | | | | | | | | | | | | | | | | | | | | | 676befdf52 ctest: add support for memcheck using Dr. Memory 2db0a65f56 cmCTestMemCheckHandler.cxx: minor refactoring Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4119
| * | ctest: add support for memcheck using Dr. MemoryDietmar Scheidl2019-12-102-0/+177
| | | | | | | | | | | | Fixes: #19788
| * | cmCTestMemCheckHandler.cxx: minor refactoringDietmar Scheidl2019-12-101-6/+3
| | |
* | | ctest: Populate CTEST_CONFIGURATION_TYPE from the -C command line when setJulien Jomier2019-12-091-0/+7
|/ /
* | Merge topic 'ctest-resource-fixes'Brad King2019-11-261-0/+26
|\ \ | |/ | | | | | | | | | | | | | | a033bafbe0 Help: Clarify how tests are run if no resource spec file is specified a64ba0235f CTest: Clarify that resource requirements can be split f9f294f5fa CTest: Add version field to resource spec file Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4080
| * CTest: Add version field to resource spec fileKyle Edwards2019-11-251-0/+26
| | | | | | | | Fixes: #20007
* | ctest_test: Add option to REPEAT testsBrad King2019-11-074-3/+38
| |
* | cmCTestMultiProcessHandler: Hold repeat mode as a memberBrad King2019-11-073-4/+14
| |
* | CTest: Rename internal APIs for --repeat optionsBrad King2019-11-073-13/+13
| | | | | | | | Replace use of the term "rerun" with "repeat" to match the public names.
* | Merge branch 'backport-ctest-resource-groups'Brad King2019-11-0517-272/+278
|\ \ | |/
| * CTest: Rename hardware -> resources for source codeCraig Scott2019-11-0514-180/+185
| |
| * cmCTestMultiProcessHandler: Rename resource locking functionsCraig Scott2019-11-052-6/+6
| | | | | | | | | | | | Renaming these ahead of other refactoring which will use the previous names. The previous names more accurately reflect their purpose after this commit anyway (talking about locking and unlocking rather than allocating and deallocating).
| * CTest: Rename hardware -> resources for CMake variables, command optionsCraig Scott2019-11-053-5/+5
| | | | | | | | | | | | Only changes the user-visible effects of renaming hardware allocation to resource allocation. Code changes are the minimum needed to achieve that.
| * CTest: Rename "Processes" lexer to "ResourceGroups"Brad King2019-11-054-63/+63
| | | | | | | | | | The corresponding test property `PROCESSES` has been renamed to `RESOURCE_GROUPS`.
| * CTest: Rename PROCESSES test property to RESOURCE_GROUPSBrad King2019-11-054-23/+24
| | | | | | | | | | | | | | | | | | | | | | | | The `PROCESSES` test property name added for CMake 3.16 is too close to the existing `PROCESSORS` test property. Furthermore, the property in principle specifies groups of resources organized in a way that is meaningful to a particular test. The groups may often correspond to processes but they could have other meanings. Since the property name `PROCESSES` has not been in a final 3.16 release yet, simply rename it to `RESOURCE_GROUPS`. Fixes: #19914