feat: add code coverage support and expose parameter registration in tests

Author: daantimmer

.vscode/settings.json

@@ -24,5 +24,9 @@
     "sonarlint.connectedMode.project": {
         "connectionId": "philips-software",
         "projectKey": "philips-software_amp-cucumber-cpp-runner"
-    }
+    },
+    "cmake.postRunCoverageTarget": "generate-coverage-report",
+    "cmake.coverageInfoFiles": [
+        "${workspaceFolder}/.build/${command:cmake.activeConfigurePresetName}/lcov.info"
+    ]
 }

CMakeLists.txt

@@ -31,6 +31,19 @@ option(CCR_BUILD_TESTS "Enable building the tests" ${CCR_DEFAULTOPT})
 
 if (CCR_BUILD_TESTS)
     ccr_enable_testing()
+
+    find_program(GCOVR_PATH gcovr)
+
+    if (NOT GCOVR_PATH)
+      message(FATAL_ERROR "Could not find gcovr, which is required for code coverage IDE integration")
+    endif()
+
+    add_custom_target(
+        generate-coverage-report
+        COMMAND ${GCOVR_PATH} --delete --lcov ${CMAKE_CURRENT_BINARY_DIR}/lcov.info
+        WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
+        COMMENT "Generating code coverage report in LCOV format"
+      )
 endif()
 
 set(CMAKE_CXX_STANDARD 20)

cucumber_cpp/library/cucumber_expression/ParameterRegistry.cpp

@@ -69,20 +69,20 @@ namespace cucumber_cpp::library::cucumber_expression
         const static std::string stringSingleRegex{ R"__('([^'\\]*(\\.[^'\\]*)*)')__" };
         const static std::string wordRegex{ R"__([^\s]+)__" };
 
-        Add("int", { integerNegativeRegex, integerPositiveRegex }, CreateStreamConverter<std::int32_t>());
-        Add("float", { floatRegex }, CreateStreamConverter<float>());
-        Add("word", { wordRegex }, CreateStreamConverter<std::string>());
-        Add("string", { stringDoubleRegex, stringSingleRegex }, CreateStringConverter());
-        Add("", { ".*" }, CreateStreamConverter<std::string>());
-        Add("bigdecimal", { floatRegex }, CreateStreamConverter<double>());
-        Add("biginteger", { { integerNegativeRegex, integerPositiveRegex } }, CreateStreamConverter<std::int64_t>());
-        Add("byte", { { integerNegativeRegex, integerPositiveRegex } }, CreateStreamConverter<std::int32_t>());
-        Add("short", { { integerNegativeRegex, integerPositiveRegex } }, CreateStreamConverter<std::int32_t>());
-        Add("long", { { integerNegativeRegex, integerPositiveRegex } }, CreateStreamConverter<std::int64_t>());
-        Add("double", { floatRegex }, CreateStreamConverter<double>());
+        ParameterRegistry::Add("int", { integerNegativeRegex, integerPositiveRegex }, CreateStreamConverter<std::int32_t>());
+        ParameterRegistry::Add("float", { floatRegex }, CreateStreamConverter<float>());
+        ParameterRegistry::Add("word", { wordRegex }, CreateStreamConverter<std::string>());
+        ParameterRegistry::Add("string", { stringDoubleRegex, stringSingleRegex }, CreateStringConverter());
+        ParameterRegistry::Add("", { ".*" }, CreateStreamConverter<std::string>());
+        ParameterRegistry::Add("bigdecimal", { floatRegex }, CreateStreamConverter<double>());
+        ParameterRegistry::Add("biginteger", { { integerNegativeRegex, integerPositiveRegex } }, CreateStreamConverter<std::int64_t>());
+        ParameterRegistry::Add("byte", { { integerNegativeRegex, integerPositiveRegex } }, CreateStreamConverter<std::int32_t>());
+        ParameterRegistry::Add("short", { { integerNegativeRegex, integerPositiveRegex } }, CreateStreamConverter<std::int32_t>());
+        ParameterRegistry::Add("long", { { integerNegativeRegex, integerPositiveRegex } }, CreateStreamConverter<std::int64_t>());
+        ParameterRegistry::Add("double", { floatRegex }, CreateStreamConverter<double>());
 
         // extension
-        Add("bool", { wordRegex }, CreateStreamConverter<bool>());
+        ParameterRegistry::Add("bool", { wordRegex }, CreateStreamConverter<bool>());
     }
 
     Parameter ParameterRegistry::Lookup(const std::string& name) const

cucumber_cpp/library/test/TestApplication.cpp

@@ -124,4 +124,9 @@ namespace cucumber_cpp::library
 
         EXPECT_THAT(stdoutString, testing::HasSubstr("1/2 passed"));
     }
+
+    TEST_F(TestApplication, ExposeParameterRegistration)
+    {
+        EXPECT_THAT(&Application{}.ParameterRegistration(), testing::NotNull());
+    }
 }