CICD: Fix for lcov choking Pybind11 init (#138)

* CICD: Fix for lcov choking Pybind11 init

* Remove third party from exclude pattern as it's empty

Author: RUrlus

cmake/CoverageTarget.cmake

@@ -7,40 +7,44 @@ IF (ENABLE_COVERAGE)
         MESSAGE(FATAL_ERROR "lcov not found, cannot perform coverage.")
     ENDIF ()
 
-    # coveralls.io does not support striped paths
-    #find_program (SED NAMES sed)
-    #if (NOT SED)
-    #    message(FATAL_ERROR "Unable to find sed")
-    #else()
-    #    # message(STATUS "sed found at ${SED}")
-    #endif (NOT SED)
-
-    # Don't forget '' around each pattern
-    SET(LCOV_EXCLUDE_PATTERN "'${PROJECT_SOURCE_DIR}/third_party/*'" "'${PROJECT_SOURCE_DIR}/extern/*'")
+    SET(LCOV_EXCLUDE_PATTERN
+        "'${PROJECT_SOURCE_DIR}/extern/*'"
+        "'${PROJECT_SOURCE_DIR}/tests/src/bindings.cpp'"
+    )
 
     ADD_CUSTOM_TARGET(coverage
-        # Cleanup previously generated profiling data
         COMMAND ${LCOV} --base-directory ${PROJECT_SOURCE_DIR} --directory ${PROJECT_BINARY_DIR} --zerocounters
-        # Initialize profiling data with zero coverage for every instrumented line of the project
-        # This way the percentage of total lines covered will always be correct, even when not all source code files were loaded during the test(s)
-        COMMAND ${LCOV} --base-directory ${PROJECT_SOURCE_DIR} --directory ${PROJECT_BINARY_DIR} --capture --initial --output-file coverage_base.info
-        # Run tests
+
+        # Initial capture
+        COMMAND ${LCOV} --base-directory ${PROJECT_SOURCE_DIR}
+                        --directory ${PROJECT_BINARY_DIR}
+                        --capture
+                        --initial
+                        --ignore-errors mismatch
+                        --output-file coverage_base.info
+
         COMMAND ${CMAKE_CTEST_COMMAND} -j ${PROCESSOR_COUNT}
-        # Collect data from executions
-        COMMAND ${LCOV} --base-directory ${PROJECT_SOURCE_DIR} --directory ${PROJECT_BINARY_DIR} --capture --output-file coverage_ctest.info
-        # Combine base and ctest results
-        COMMAND ${LCOV} --add-tracefile coverage_base.info --add-tracefile coverage_ctest.info --output-file coverage_full.info
-        # Extract only project data (--no-capture or --remove options may be used to select collected data)
+
+        # Post-test capture
+        COMMAND ${LCOV} --base-directory ${PROJECT_SOURCE_DIR}
+                        --directory ${PROJECT_BINARY_DIR}
+                        --capture
+                        --ignore-errors mismatch
+                        --output-file coverage_ctest.info
+
+        COMMAND ${LCOV} --add-tracefile coverage_base.info
+                        --add-tracefile coverage_ctest.info
+                        --output-file coverage_full.info
+
         COMMAND ${LCOV} --remove coverage_full.info ${LCOV_EXCLUDE_PATTERN} --output-file coverage_filtered.info
         COMMAND ${LCOV} --extract coverage_filtered.info '${PROJECT_SOURCE_DIR}/*' --output-file coverage.info
-        # coveralls.io does not support striped paths
-        #COMMAND ${SED} -i.bak 's|SF:${PROJECT_SOURCE_DIR}/|SF:|g' coverage.info
+
         DEPENDS tests
         COMMENT "Running test coverage."
         WORKING_DIRECTORY "${PROJECT_BINARY_DIR}"
     )
     MESSAGE(STATUS "coverage target for code coverage is available")
-    
+
     FIND_PROGRAM(GENHTML genhtml)
     IF (NOT GENHTML)
         MESSAGE(WARNING "genhtml not found, cannot perform report-coverage.")