Skip to content

Commit

Permalink
fix warning in extras/CatchAddTests.cmake
Browse files Browse the repository at this point in the history
  - don't warn on zero return code of --list-reporters
  - previously return code was the number of reporters (catchorg#1410, catchorg#1146)
  - as of 2c06ee9 return code is zero on success
  • Loading branch information
lkeegan committed Oct 8, 2020
1 parent b9853b4 commit c44d9cc
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions extras/CatchAddTests.cmake
Expand Up @@ -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"
Expand Down

0 comments on commit c44d9cc

Please sign in to comment.