From c5b58b0842925b7de0e36dfb30cfc47383f4d5c6 Mon Sep 17 00:00:00 2001 From: Brad King Date: Sat, 27 Jan 2024 11:32:52 -0500 Subject: Tests: Improve RunCMake.add_test error formatting --- Tests/RunCMake/add_test/TestLauncher-check.cmake | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/Tests/RunCMake/add_test/TestLauncher-check.cmake b/Tests/RunCMake/add_test/TestLauncher-check.cmake index 78020a6..d6e795c 100644 --- a/Tests/RunCMake/add_test/TestLauncher-check.cmake +++ b/Tests/RunCMake/add_test/TestLauncher-check.cmake @@ -2,27 +2,33 @@ set(testfile "${RunCMake_TEST_BINARY_DIR}/CTestTestfile.cmake") if(EXISTS "${testfile}") file(READ "${testfile}" testfile_contents) else() - message(FATAL_ERROR "Could not find expected CTestTestfile.cmake.") + set(RunCMake_TEST_FAILED "Could not find expected CTestTestfile.cmake.") + return() endif() -set(error_details "There is a problem with generated test file: ${testfile}") +set(error_details "There is a problem with generated test file:\n ${testfile}") if(testfile_contents MATCHES "add_test[(]DoesNotUseTestLauncher [^\n]+pseudo_test_launcher[^\n]+\n") - message(SEND_ERROR "Used test launcher when it should not be used. ${error_details}") + set(RunCMake_TEST_FAILED "Used test launcher when it should not be used. ${error_details}") + return() endif() if(NOT testfile_contents MATCHES "add_test[(]UsesTestLauncher [^\n]+pseudo_test_launcher[^\n]+\n") - message(SEND_ERROR "Did not use test launcher when it should be used. ${error_details}") + set(RunCMake_TEST_FAILED "Did not use test launcher when it should be used. ${error_details}") + return() endif() if(testfile_contents MATCHES "add_test[(]DoesNotUseTestLauncherWithGenex [^\n]+pseudo_test_launcher[^\n]+\n") - message(SEND_ERROR "Used test launcher when it should not be used. ${error_details}") + set(RunCMake_TEST_FAILED "Used test launcher when it should not be used. ${error_details}") + return() endif() if(NOT testfile_contents MATCHES "add_test[(]UsesTestLauncherWithExecTargetFromSubdirAddedWithoutGenex [^\n]+pseudo_test_launcher[^\n]+\n") - message(SEND_ERROR "Did not use test launcher when it should be used. ${error_details}") + set(RunCMake_TEST_FAILED "Did not use test launcher when it should be used. ${error_details}") + return() endif() if(testfile_contents MATCHES "add_test[(]DoesNotUseTestLauncherWithExecTargetFromSubdirAddedWithGenex [^\n]+pseudo_test_launcher[^\n]+\n") - message(SEND_ERROR "Used test launcher when it should not be used. ${error_details}") + set(RunCMake_TEST_FAILED "Used test launcher when it should not be used. ${error_details}") + return() endif() -- cgit v0.12