ENH: Minor readability improvement in CTest output
When ctest --build-and-test runs the --test-command its output did not quote the arguments of the command being tested making it difficult to read. This adds the quotes. This also changes the wording of the failure case to not sound like CTest could not run the executable when in fact it ran and returned failure.
This commit is contained in:
parent
1e2c7a7bb3
commit
4c725d1e0e
|
@ -359,10 +359,10 @@ int cmCTestBuildAndTestHandler::RunCMakeAndTest(std::string* outstring)
|
|||
out << "Run test in directory: " << this->BuildRunDir << "\n";
|
||||
cmSystemTools::ChangeDirectory(this->BuildRunDir.c_str());
|
||||
}
|
||||
out << "Running test executable: " << fullPath << " ";
|
||||
out << "Running test command: \"" << fullPath << "\"";
|
||||
for(k=0; k < this->TestCommandArgs.size(); ++k)
|
||||
{
|
||||
out << this->TestCommandArgs[k] << " ";
|
||||
out << " \"" << this->TestCommandArgs[k] << "\"";
|
||||
}
|
||||
out << "\n";
|
||||
|
||||
|
@ -386,7 +386,7 @@ int cmCTestBuildAndTestHandler::RunCMakeAndTest(std::string* outstring)
|
|||
|
||||
if(runTestRes != cmsysProcess_State_Exited || retval != 0)
|
||||
{
|
||||
out << "Failed to run test command: " << testCommand[0] << "\n";
|
||||
out << "Test command failed: " << testCommand[0] << "\n";
|
||||
retval = 1;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue