From 64f0e8e0340dc94754de7200035a3c947a3f7f7a Mon Sep 17 00:00:00 2001 From: Dominik Holland Date: Wed, 6 Mar 2024 19:49:12 +0100 Subject: Improve the code-coverage setup Instead of making a standalone_test build and referencing the build folder from there, we do the test build together with the module build and run it like on a developer machine. This makes the whole report more reliable and fixes some areas which were marked as not tested at all. Change-Id: Ifa646ebc613e35bb8aa2dc6e1c1df5c57191c180 Reviewed-by: Robert Griebl --- coin/instructions/cmake_module_lcov_code_coverage.yaml | 4 ++-- coin/module_config.yaml | 10 ---------- 2 files changed, 2 insertions(+), 12 deletions(-) diff --git a/coin/instructions/cmake_module_lcov_code_coverage.yaml b/coin/instructions/cmake_module_lcov_code_coverage.yaml index 1b86e123..9b6d75df 100644 --- a/coin/instructions/cmake_module_lcov_code_coverage.yaml +++ b/coin/instructions/cmake_module_lcov_code_coverage.yaml @@ -2,7 +2,7 @@ type: Group instructions: - !include "{{qt/qtbase}}/coin_module_test_template_common.yaml" - type: ChangeDirectory - directory: "{{.SourceDir}}_standalone_tests" + directory: "{{.SourceDir}}_build" - type: EnvironmentVariable variableName: CMAKE_BUILD_TIMEOUT variableValue: "6000" @@ -42,4 +42,4 @@ instructions: - type: ScheduleUploadTestResults - type: EnvironmentVariable variableName: COIN_CTEST_RESULTSDIR - variableValue: "{{.SourceDir}}_standalone_tests/upload" + variableValue: "{{.SourceDir}}_build/upload" diff --git a/coin/module_config.yaml b/coin/module_config.yaml index f2aed118..8fef75a6 100644 --- a/coin/module_config.yaml +++ b/coin/module_config.yaml @@ -35,13 +35,6 @@ instructions: condition: property property: features contains_value: lcov - - type: AppendToEnvironmentVariable - variableName: COMMON_TEST_CMAKE_ARGS - variableValue: " -DAM_COVERAGE=ON" - enable_if: - condition: property - property: features - contains_value: lcov - type: Rename sourcePath: "{{.SourceDir}}/coin/axivion/ci_config_common.json" targetPath: "{{.Env.HOME}}/axivion/ci_config_common.json" @@ -80,6 +73,3 @@ instructions: - condition: property property: target.os in_values: ["IOS", "Android", "WebAssembly", "QNX"] - - condition: property - property: features - contains_value: lcov -- cgit v1.2.3