From 1a4837641e8fad5d8a0c611ca1c70543c5748fe2 Mon Sep 17 00:00:00 2001 From: Brad King Date: Fri, 8 Mar 2024 16:07:36 -0500 Subject: ctest: Remove unnecessary and ambiguous tests-from-file comment syntax Test names can contain `#`. Since we ignore lines that do not match any test names anyway, "commenting" can still work without explicit syntax. Also drop whitespace trimming for similar reasons. Fixes: #25741 --- Help/manual/ctest.1.rst | 4 ++-- Source/CTest/cmCTestTestHandler.cxx | 7 ++----- Tests/RunCMake/CTestCommandLine/TestsFromFile-TestList.txt | 3 +++ Tests/RunCMake/ctest_test/TestsFromFile-TestList.txt | 3 +++ 4 files changed, 10 insertions(+), 7 deletions(-) diff --git a/Help/manual/ctest.1.rst b/Help/manual/ctest.1.rst index 602a82d..aa95ac6 100644 --- a/Help/manual/ctest.1.rst +++ b/Help/manual/ctest.1.rst @@ -242,7 +242,7 @@ Run Tests This option tells CTest to run the tests which are listed in the given file. The file must contain one exact test name per line. - Lines can be commented out using a ``#``. + Lines that do not exactly match any test names are ignored. This option can be combined with the other options like ``-R``, ``-E``, ``-L`` or ``-LE``. @@ -254,7 +254,7 @@ Run Tests This option tells CTest to NOT run the tests which are listed in the given file. The file must contain one exact test name per line. - Lines can be commented out using a ``#``. + Lines that do not exactly match any test names are ignored. This option can be combined with the other options like ``-R``, ``-E``, ``-L`` or ``-LE``. diff --git a/Source/CTest/cmCTestTestHandler.cxx b/Source/CTest/cmCTestTestHandler.cxx index 63ff8d1..dae205c 100644 --- a/Source/CTest/cmCTestTestHandler.cxx +++ b/Source/CTest/cmCTestTestHandler.cxx @@ -2037,12 +2037,9 @@ cm::optional> cmCTestTestHandler::ReadTestListFile( std::set testNames; std::string line; while (cmSystemTools::GetLineFromStream(ifs, line)) { - std::string trimmed = cmTrimWhitespace(line); - if (trimmed.empty() || (trimmed[0] == '#')) { - continue; + if (!line.empty()) { + testNames.insert(line); } - - testNames.insert(trimmed); } result = std::move(testNames); } else { diff --git a/Tests/RunCMake/CTestCommandLine/TestsFromFile-TestList.txt b/Tests/RunCMake/CTestCommandLine/TestsFromFile-TestList.txt index 91f6504..975a21c 100644 --- a/Tests/RunCMake/CTestCommandLine/TestsFromFile-TestList.txt +++ b/Tests/RunCMake/CTestCommandLine/TestsFromFile-TestList.txt @@ -1,2 +1,5 @@ Test1 + +est + Test11 # Test11 diff --git a/Tests/RunCMake/ctest_test/TestsFromFile-TestList.txt b/Tests/RunCMake/ctest_test/TestsFromFile-TestList.txt index 91f6504..975a21c 100644 --- a/Tests/RunCMake/ctest_test/TestsFromFile-TestList.txt +++ b/Tests/RunCMake/ctest_test/TestsFromFile-TestList.txt @@ -1,2 +1,5 @@ Test1 + +est + Test11 # Test11 -- cgit v0.12