From 4be9de6c549af6e1e4c5fda1e3adf9c9ec4ae515 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Ho=C5=99e=C5=88ovsk=C3=BD?= Date: Sun, 28 Sep 2025 12:46:32 +0200 Subject: [PATCH] Disable test for deprecated CATCH_CONFIG_BAZEL_SUPPORT option Because it requires full rebuild of the base library, it adds about 20% to the build time of the test suite. Between the fact that the option is deprecated, and that Bazel has added the `BAZEL_TEST` env var _years_ ago, nobody should be using it, and the chance of breakage is tiny, the test is not worth its compile-time cost. --- tests/ExtraTests/CMakeLists.txt | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/tests/ExtraTests/CMakeLists.txt b/tests/ExtraTests/CMakeLists.txt index d7ae548b..fbcc3465 100644 --- a/tests/ExtraTests/CMakeLists.txt +++ b/tests/ExtraTests/CMakeLists.txt @@ -109,18 +109,6 @@ set_tests_properties(CATCH_CONFIG_DISABLE_EXCEPTIONS-1 FAIL_REGULAR_EXPRESSION "abort;terminate;fatal" ) -add_executable(BazelReporter ${TESTS_DIR}/X30-BazelReporter.cpp) -target_compile_definitions(BazelReporter PRIVATE CATCH_CONFIG_BAZEL_SUPPORT) -target_link_libraries(BazelReporter Catch2_buildall_interface) -add_test(NAME CATCH_CONFIG_BAZEL_REPORTER-1 - COMMAND - Python3::Interpreter "${CATCH_DIR}/tests/TestScripts/testBazelReporter.py" $ "${CMAKE_CURRENT_BINARY_DIR}" -) -set_tests_properties(CATCH_CONFIG_BAZEL_REPORTER-1 - PROPERTIES - LABELS "uses-python" -) - # We must now test this works without the build flag. add_executable(BazelReporterNoCatchConfig ${TESTS_DIR}/X30-BazelReporter.cpp) target_link_libraries(BazelReporterNoCatchConfig Catch2WithMain)