Merge pull request syslog-ng#5047 from HofiOne/cmake-test-switching-fixes

cmake: fixed multiple testing related issues

Author: HofiOne

CMakeLists.txt

@@ -379,17 +379,17 @@ endif()
 
 include(add_tests)
 
-# The inclusion of CTest triggers enable_testing()
-# CMake will generate tests only if the enable_testing() command has been invoked.
-# The CTest module invokes the command automatically when the BUILD_TESTING option is ON.
-
 if (BUILD_TESTING)
   if (NOT CRITERION_FOUND)
     message(FATAL_ERROR "BUILD_TESTING enabled without criterion detected!")
   else()
     set(CTEST_ENVIRONMENT
       "G_SLICE=always-malloc,debug-blocks"
       "G_DEBUG=fatal-warnings,fatal-criticals,gc-friendly")
+
+    # The inclusion of CTest triggers enable_testing()
+    # CMake will generate tests only if the enable_testing() command has been invoked.
+    # The CTest module invokes the command automatically when the BUILD_TESTING option is ON.
     include(CTest)
 
     # This flag might be a security issue, do not use in production code, unfortunately we still need it for criterion tests and the current mocking soution

libtest/CMakeLists.txt

@@ -1,3 +1,7 @@
+if (NOT BUILD_TESTING)
+    return ()
+endif ()
+
 set(LIBTEST_HEADERS
     config_parse_lib.h
     fake-time.h

modules/kafka/CMakeLists.txt

@@ -1,23 +1,21 @@
-find_package(rdkafka 1.1.0)
+if (NOT DEFINED ENABLE_KAFKA OR ENABLE_KAFKA)
+  find_package(rdkafka 1.1.0)
+endif ()
 
-if (RDKAFKA_FOUND)
-  option(ENABLE_KAFKA "Enable kafka module" ON)
-else()
-  option(ENABLE_KAFKA "Enable kafka module" OFF)
-endif()
+module_switch (ENABLE_KAFKA "Enable kafka module" RDKAFKA_FOUND)
 
-if (NOT ENABLE_KAFKA)
- return()
-endif()
+if (ENABLE_KAFKA AND NOT RDKAFKA_FOUND)
+  message (FATAL_ERROR "Kafka enabled but no librdkafka found")
+endif ()
 
-if (NOT RDKAFKA_FOUND)
-  message(FATAL_ERROR "Kafka enabled but no librdkafka found")
+if (NOT ENABLE_KAFKA)
+  return()
 endif()
 
+check_symbol_exists (rd_kafka_init_transactions "librdkafka/rdkafka.h" SYSLOG_NG_HAVE_RD_KAFKA_INIT_TRANSACTIONS)
 
 set(CMAKE_REQUIRED_INCLUDES ${RDKAFKA_INCLUDE_DIR})
 set(CMAKE_REQUIRED_LIBRARIES ${RDKAFKA_LIBRARY})
-check_symbol_exists(rd_kafka_init_transactions "librdkafka/rdkafka.h" SYSLOG_NG_HAVE_RD_KAFKA_INIT_TRANSACTIONS)
 
 set(KAFKA_SOURCES
   kafka-parser.c

modules/mqtt/CMakeLists.txt

@@ -3,6 +3,11 @@ if (NOT DEFINED ENABLE_MQTT OR ENABLE_MQTT)
 endif()
 
 module_switch(ENABLE_MQTT "Enable mqtt" eclipse-paho-mqtt-c_FOUND)
+
+if (ENABLE_MQTT AND NOT eclipse-paho-mqtt-c_FOUND)
+  message (FATAL_ERROR "MQTT enabled but no eclipse-paho-mqtt-c library found")
+endif ()
+
 if (NOT ENABLE_MQTT)
   return()
 endif()