Rename ENABLE_COVERAGE

Author: Hind-M

.github/workflows/windows.yml

@@ -65,7 +65,7 @@ jobs:
         if: matrix.build_type == 'Debug'
         run: |
           choco install opencppcoverage
-          echo "TEST_COVERAGE_ACTIVATION=' -DENABLE_COVERAGE=ON'" >> $GITHUB_ENV
+          echo "TEST_COVERAGE_ACTIVATION=' -DSPARROW_IPC_ENABLE_COVERAGE=ON'" >> $GITHUB_ENV
 
       - name: Configure using cmake
         run: |

CMakeLists.txt

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

cmake/code_coverage.cmake

@@ -38,14 +38,14 @@ elseif(CMAKE_CXX_COMPILER_ID MATCHES "Clang" OR CMAKE_CXX_COMPILER_ID STREQUAL "
 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)
+# 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,9 +14,9 @@ set(
 
 add_executable(${test_target} ${SPARROW_IPC_TESTS_SRC})
 
-if(ENABLE_COVERAGE)
-    enable_coverage(${test_target})
-endif()
+# if(ENABLE_COVERAGE)
+#     enable_coverage(${test_target})
+# endif()
 
 target_link_libraries(${test_target}
     PRIVATE