summaryrefslogtreecommitdiffstats
path: root/Source/CTest
Commit message (Collapse)AuthorAgeFilesLines
...
* | CTest: Remove unfinished batch test modeBrad King2017-12-084-161/+1
| | | | | | | | | | This was partially implemented by commit v2.8.0~154 (Added some ctest batch capabilities, 2009-09-10) but never finished.
* | Merge topic 'fix-ctest-chrono'Brad King2017-12-071-3/+4
|\ \ | |/ | | | | | | | | | | | | be470278 CTest: Fix regression in test timeout compuatation Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: dublet <github@dublet.org> Merge-request: !1568
| * CTest: Fix regression in test timeout compuatationBrad King2017-12-071-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | Refactoring in commit 66419bc046 (CTest: convert timeouts to std::chrono::duration, 2017-11-20) accidentally changed the logic used to compute the timeout for a test when it starts. It incorrectly limits the maximum possible timeout to 2 minutes rather than 2 minutes less than the total allowed test time remaining. Update the new logic to restore the original behavior. Avoid subtracting 2 minutes from our "infinite" timeout value to avoid creating very large timeouts that are not "infinite" and may exceed integer type ranges.
* | Merge topic 'ctest-chrono'Brad King2017-12-0515-98/+171
|\ \ | |/ | | | | | | | | | | | | 66419bc0 CTest: convert timeouts to std::chrono::duration 923b8fad IWYU: Add mappings for std::enable_if on chrono durations Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1513
| * CTest: convert timeouts to std::chrono::durationWouter Klouwen2017-12-0413-90/+171
| | | | | | | | | | | | | | | | | | This commit continues the refactoring of CTest to adopt std::chrono. After the last sets of changes that introduced std::chrono::steady_clock and std::chrono::system_clock respectively, it makes sense to have all the timeouts be stored as std::chrono::duration. No functional change intended.
| * IWYU: Add mappings for std::enable_if on chrono durationsBrad King2017-11-288-8/+0
| | | | | | | | | | | | | | | | IWYU incorrectly classifies this internal STL type as not internal, and suggests including `<type_traits>` for it. Work around the problem by mapping the offending names to a file that we always include. See include-what-you-use issue 434.
* | CTest: Check failed fixture dependencies before test command and argsCraig Scott2017-11-281-2/+6
|/
* CTest: adopt std::chrono::system_clockWouter Klouwen2017-11-178-31/+24
| | | | | | | | | | | | | | | | | | After the refactor to make CTest use std::chrono::steady_clock for the keeping of time for test duration, there are still references to cmSystemTools::GetTime() left. To further adopt std::chrono for time related activities, this commit changes those remaining references to std::chrono::system_clock::now() calls and alters the storage from either unsigned int or double to std::chrono::system_clock::time_point. For ease of conversion, a converter method is added to cmXMLWriter that converts from a std::chrono::system_clock::time_point to the number of seconds since the UN*X epoch as that is expected behaviour. This means no more casts as required. Functionally should be no difference as the system_clock is implemented in the same terms.
* CTest: use std::chrono::steady_clock for time keepingWouter Klouwen2017-11-1415-96/+155
| | | | | | | | | | | | It was reported in issue #17345 that CTest does not use monotonic time to report test duration. Monotonic clocks are not affected by large NTP adjustments or things like daylight savings time. As CMake 3.10 requires C++11, which introduced std::chrono, this commit moves the time keeping in CTest from cmSystemTools::GetTime() to std::chrono::steady_clock. Fixes: #17345
* CTest: correct misleading warning message for RUN_SERIAL testsWouter Klouwen2017-11-091-0/+12
| | | | | | | | | As reported in issue 17167, when only RUN_SERIAL tests remain, CTest can display a misleading message that it is waiting for the load to come down when in fact, it cannot start any new tests. This commit fixes that by determining whether this circumstance is what's happening and adds an additional warning message in this case.
* Fix trivial typos in textluzpaz2017-11-034-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | Some are user facing. Found using codespell -q 3 --skip="./Utilities" -I .cmake-whitelist.txt` whereby the whitelist contained: ans dum helpfull emmited emmitted buil iff isnt nto ot pathes substract te todays upto whitespaces
* Improve several occurrences of vector::push_back in loopsMatthias Maennich2017-09-284-0/+4
| | | | | | | Fix issues diagnosed by clang-tidy by pre-allocating the vector capacity before the loop [performance-inefficient-vector-operation]. Signed-off-by: Matthias Maennich <matthias@maennich.net>
* Retire std::auto_ptr and its macro CM_AUTO_PTRMatthias Maennich2017-09-253-18/+19
| | | | Signed-off-by: Matthias Maennich <matthias@maennich.net>
* Replace empty-string comparisons with checking against `empty()`.Pavel Solodovnikov2017-09-219-17/+18
|
* Merge topic 'CTest-cov-string'Brad King2017-09-202-11/+10
|\ | | | | | | | | | | | | 062776e7 CTest: pass std::string to cmCTestCoverageHandler::ShouldIDoCoverage() Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1290
| * CTest: pass std::string to cmCTestCoverageHandler::ShouldIDoCoverage()Rolf Eike Beer2017-09-192-11/+10
| | | | | | | | Avoid needless conversion back and forth.
* | Merge topic 'CTest-crash-missing-glob'Brad King2017-09-201-0/+2
|\ \ | |/ |/| | | | | | | | | 6db63012 CTest: fix crash if source file for coverage cannot be found Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1289
| * CTest: fix crash if source file for coverage cannot be foundRolf Eike Beer2017-09-191-0/+2
| | | | | | | | | | | | The opened XML elements were not closed, so an assert was triggered when the file was finally closed. If CMake is built with assertions disabled then an invalid XML file will be produced.
* | Merge topic 'string-clear'Brad King2017-09-1916-81/+81
|\ \ | | | | | | | | | | | | | | | | | | 5db3aac1 Meta: replace empty-string assignments with `clear()`. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1276
| * | Meta: replace empty-string assignments with `clear()`.Pavel Solodovnikov2017-09-1516-81/+81
| | |
* | | Merge topic 'ctest-runcommand'Brad King2017-09-181-9/+36
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 67529aab Doc: document that CoverageExtraFlags will come first 69fac3c3 pass arguments as vector to cmCTest::RunCommand() Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1261
| * | | pass arguments as vector to cmCTest::RunCommand()Rolf Eike Beer2017-09-151-9/+36
| |/ / | | | | | | | | | | | | | | | | | | The only 2 callers took care to construct a properly escaped string, but not using the documented way, and that string was passed only to be immediately split into tokens again. Start with a vector and join it only for logging, avoiding needless quotes during that.
* | | Use C++11 override instead of CM_OVERRIDEBrad King2017-09-1546-184/+183
|/ / | | | | | | | | | | | | | | We now require C++11 support including `override`. Drop use of the old compatibility macro. Convert references as follows: git grep -l CM_OVERRIDE -- '*.h' '*.hxx' '*.cxx' | xargs sed -i 's/CM_OVERRIDE/override/g'
* | Meta: modernize old-fashioned loops to range-based `for` (CTest).Pavel Solodovnikov2017-09-1432-824/+573
| | | | | | | | | | | | Changes done via `clang-tidy` with some manual fine-tuning for the variable naming and `auto` type deduction where appropriate.
* | CTest: Weight reported test times by PROCESSORS in summariesBill Hoffman2017-09-081-2/+2
| | | | | | | | | | | | This commit changes the times reported by labels and subprojects to be weighted by the PROCESSORS property. It is reported with `sec*proc` instead of just `sec`.
* | CTest: Split out labels and subproject labels in summary outputBill Hoffman2017-09-082-99/+37
| | | | | | | | | | | | | | | | This commit splits out the reporting of labels and labels used for sub projects. If a label is a sub project label it will not be included in the label summary. To implement this the commit creates PrintLabelOrSubprojectSummary which is able to do the work of both PrintLabelSummary and PrintSubprojectSummary avoiding code duplication.
* | use static_cast<> for casts from void*Rolf Eike Beer2017-09-012-4/+4
| |
* | clang-format: format all code as Cpp11Daniel Pfeifer2017-08-303-9/+9
| |
* | Merge topic 'cstyle-casts'Daniel Pfeifer2017-08-307-29/+30
|\ \ | | | | | | | | | | | | | | | | | | 190e3825 Replace C-style casts Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1176
| * | Replace C-style castsDaniel Pfeifer2017-08-277-29/+30
| | |
* | | IWYU: Mark cmConfigure.h with pragma: keepDaniel Pfeifer2017-08-2643-45/+35
|/ / | | | | | | Also remove `#include "cmConfigure.h"` from most source files.
* | Use C++11 nullptrDaniel Pfeifer2017-08-2434-165/+160
| |
* | CTest: Report lines matched by launchersBill Hoffman2017-08-071-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Since launchers show the entire stderr/stdout for a command and only skip it if all of the lines are suppressed, it can be hard to tell if individual suppressions are working. Mark up the output reported to CDash with a prefix for each line indicating whether it was a matched or suppressed warning. Lines that are suppressed are prefixed with `[CTest: warning suppressed]` and lines that match the warning expression are prefixed with `[CTest: warning matched]`. This should make it easier for people to add suppressions as they will know what they need to match and what they have already suppressed. This is an issue because if there is more than one warning in a single compiler output you have to suppress all of them before it will go away.
* | Merge topic 'add_test_processors_prop_to_cdash'Brad King2017-07-211-0/+6
|\ \ | | | | | | | | | | | | | | | | | | 614921b4 Add the test property PROCESSORS to the CDash xml as a NamedMeasurement. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1066
| * | Add the test property PROCESSORS to the CDash xml as a NamedMeasurement.Bill Hoffman2017-07-181-0/+6
| | | | | | | | | | | | | | | | | | This commit forwards the test property PROCESSORS to CDash in the xml produced by ctest. This is to allow CDash to know how much processor time was allocated to a test.
* | | Merge topic 'cmake-E-sha'Brad King2017-07-181-7/+5
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | c4de0a25 Add sha1sum, sha224sum, sha256sum, sha384sum and sha512sum to command mode c4647d84 Change ComputeFileMD5 to ComputeFileHash 501a4fee Add some unit tests for md5sum Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1021
| * | Change ComputeFileMD5 to ComputeFileHashAndré Klitzing2017-07-141-7/+5
| |/ | | | | | | | | | | | | * Use a parameter to select hash algorithm * Return a std::string as result or an empty string if it fails * Avoids unnecessary copy of hash value
* | Merge topic 'better-exception-reporting'Brad King2017-07-135-14/+26
|\ \ | | | | | | | | | | | | | | | | | | 0c650f39 CTest: Report more detail about system exceptions in tests Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1046
| * | CTest: Report more detail about system exceptions in testsBill Hoffman2017-07-125-14/+26
| |/ | | | | | | | | This passes the system exception string up to CDash and to the command line instead of just printing "other".
* | Display subproject timing summaryBetsy McPhail2017-07-102-1/+86
| | | | | | | | Use the '--no-subproject-summary' option to disable timing summary.
* | Add directory property 'LABELS' and CMAKE_DIRECTORY_LABELS variableBetsy McPhail2017-07-102-1/+99
| | | | | | | | | | The specified LABELS will be passed down to subdirectories as well as any targets or tests in the directory.
* | Add CTEST_LABELS_FOR_SUBPROJECTS as a CTest module and script variableBetsy McPhail2017-07-108-1/+23
|/ | | | | Use this variable to specify a list of labels that will be reported to CDash as subprojects.
* Merge topic 'display-ctest-disabled'Brad King2017-06-051-2/+6
|\ | | | | | | | | | | | | | | f59d729c ctest: Display disabled status when ctest -N is called Acked-by: Kitware Robot <kwrobot@kitware.com> Reviewed-by: Craig Scott <craig.scott@crascit.com> Merge-request: !923
| * ctest: Display disabled status when ctest -N is calledChuck Atkins2017-06-021-2/+6
| |
* | Remove second arg: npos in substr usagesPavel Solodovnikov2017-06-011-2/+2
| |
* | Access string npos without instancePavel Solodovnikov2017-06-0110-78/+80
| |
* | cmParseBlanketJSCoverage: Pass unmodified parameter as const&Brad King2017-06-011-1/+1
| |
* | cmCommand: remove unused methods from interface and all implementationsDaniel Pfeifer2017-05-119-55/+6
| |
* | CTest: use new methods for builtin commandsDaniel Pfeifer2017-05-113-27/+26
| |
* | cmState: remove RemoveUnscriptableCommandsDaniel Pfeifer2017-05-081-4/+0
| |