Merge pull request #2044 from lkeegan/fix_cmake_catch_add_tests_warning

fix warning in extras/CatchAddTests.cmake
2025-01-16 07:08:01 +00:00 · 2020-10-08 17:15:05 +02:00 · 2020-10-08 16:44:20 +02:00
1 changed files with 1 additions and 5 deletions
--- a/extras/CatchAddTests.cmake
+++ b/extras/CatchAddTests.cmake
@ -55,11 +55,7 @@ execute_process(
  RESULT_VARIABLE reporters_result
  WORKING_DIRECTORY "${TEST_WORKING_DIR}"
 )
-if(${reporters_result} EQUAL 0)
-  message(WARNING
-    "Test executable '${TEST_EXECUTABLE}' contains no reporters!\n"
-  )
-elseif(${reporters_result} LESS 0)
+if(NOT ${reporters_result} EQUAL 0)
  message(FATAL_ERROR
    "Error running test executable '${TEST_EXECUTABLE}':\n"
    "  Result: ${reporters_result}\n"
Author	SHA1	Message	Date
Martin Hořeňovský	4ea18d6d17	Merge pull request #2044 from lkeegan/fix_cmake_catch_add_tests_warning fix warning in extras/CatchAddTests.cmake	2020-10-08 17:15:05 +02:00
Liam Keegan	c44d9cc718	fix warning in extras/CatchAddTests.cmake - don't warn on zero return code of --list-reporters - previously return code was the number of reporters (#1410, #1146) - as of 2c06ee9 return code is zero on success	2020-10-08 16:44:20 +02:00