Remove unnecessary code

Author: Hind-M

CMakeLists.txt

@@ -47,9 +47,15 @@ MESSAGE(STATUS "🔧 Build tests: ${SPARROW_IPC_BUILD_TESTS}")
 OPTION(BUILD_DOCS  "Build sparrow-ipc documentation" OFF)
 MESSAGE(STATUS "🔧 Build docs: ${BUILD_DOCS}")
 
+# Code coverage
+# =============
 OPTION(SPARROW_IPC_ENABLE_COVERAGE "Enable test coverage" OFF)
 MESSAGE(STATUS "🔧 Enable coverage: ${SPARROW_IPC_ENABLE_COVERAGE}")
 
+if(SPARROW_IPC_ENABLE_COVERAGE)
+    include(code_coverage)
+endif()
+
 set(SPARROW_IPC_INCLUDE_DIR ${CMAKE_CURRENT_SOURCE_DIR}/include)
 set(SPARROW_IPC_SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR}/src)
 
@@ -159,10 +165,6 @@ target_link_libraries(sparrow-ipc
 
 add_dependencies(sparrow-ipc generate_flatbuffers_headers)
 
-if(SPARROW_IPC_ENABLE_COVERAGE)
-    include(code_coverage)
-    #enable_coverage(sparrow-ipc)
-endif()
 # Tests
 # =====
 if(SPARROW_IPC_BUILD_TESTS)

cmake/code_coverage.cmake

@@ -7,7 +7,6 @@ set(COBERTURA_REPORT_PATH "${COVERAGE_REPORT_PATH}/cobertura.xml" CACHE PATH "Pa
 set(COVERAGE_TARGETS_FOLDER "Tests utilities/Code Coverage")
 
 if(CMAKE_CXX_COMPILER_ID MATCHES "MSVC")
-    message(STATUS "==============> Using MSVC")
     find_program(OpenCPPCoverage OpenCppCoverage.exe opencppcoverage.exe REQUIRED
         PATHS "C:/Program Files/OpenCppCoverage" "C:/Program Files (x86)/OpenCppCoverage")
 
@@ -33,19 +32,4 @@ if(CMAKE_CXX_COMPILER_ID MATCHES "MSVC")
         COMMENT "Generating coverage report with OpenCppCoverage: ${COVERAGE_REPORT_PATH}"
     )
     set(TARGET_PROPERTIES sparrow_ipc_generate_cobertura PROPERTIES FOLDER "Tests utilities/Code Coverage")
-elseif(CMAKE_CXX_COMPILER_ID MATCHES "Clang" OR CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
-    message(STATUS "==============> Using Clang or GNU")
 endif()
-
-#TODO this may not be useful if coverage only on Windows ci?
-# function(enable_coverage target)
-#     set(CLANG_COVERAGE_FLAGS --coverage -fprofile-instr-generate -fcoverage-mapping -fno-inline -fno-elide-constructors)
-#     set(GCC_COVERAGE_FLAGS --coverage -fno-inline -fno-inline-small-functions -fno-default-inline)
-#
-#     target_compile_options(${target} PRIVATE
-#         $<$<CXX_COMPILER_ID:Clang>:${CLANG_COVERAGE_FLAGS}>
-#         $<$<CXX_COMPILER_ID:GNU>:${GCC_COVERAGE_FLAGS}>)
-#     target_link_options(${target} PRIVATE
-#         $<$<CXX_COMPILER_ID:Clang>:${CLANG_COVERAGE_FLAGS}>
-#         $<$<CXX_COMPILER_ID:GNU>:${GCC_COVERAGE_FLAGS}>)
-# endfunction(enable_coverage target)

tests/CMakeLists.txt

@@ -14,10 +14,6 @@ set(
 
 add_executable(${test_target} ${SPARROW_IPC_TESTS_SRC})
 
-# if(ENABLE_COVERAGE)
-#     enable_coverage(${test_target})
-# endif()
-
 target_link_libraries(${test_target}
     PRIVATE
         sparrow-ipc