#1: enable coverage

Author: tlamonthezie

.env

@@ -12,10 +12,10 @@ FOO_BUILD_DIR=/opt/foo/build
 FOO_OUTPUT_DIR=/opt/foo/output
 FOO_TESTS_ENABLED=ON
 FOO_RUN_TESTS=ON
-
-# Temporary: Turn off coverage reporting (requires lcov but not yet in DARMA/workflows setup requirements)
-FOO_COVERAGE_ENABLED=OFF
+FOO_COVERAGE_ENABLED=ON
 FOO_COVERAGE_REPORT=/opt/foo/output/cov
 FOO_WERROR_ENABLED=0
 
-# TODO: in CI this should be configurable per test environment
+# ENHANCEMENT: in CI we might add specific environment files per test environment
+# We might envisage in ci directory to put environment files like
+# - current .env and specific per setup like .env.{test_environment_name} (e.g. .env.wf-amd64-ubuntu-18.04-gcc-8-cpp)

.github/workflows/build-and-test.yml

@@ -141,6 +141,7 @@ jobs:
             ${{ runner.os }}-${{ env.cache_name }}-${{ env.TS_YEAR }}-
             ${{ runner.os }}-${{ env.cache_name }}-
 
+      # ENHANCEMENT: remove lcov from there and add to base images in the workflows repository
       - name: PR tests
         run: |
           WORKSPACE=${{ github.workspace }} 
@@ -149,6 +150,8 @@ jobs:
           fi
           CMD='
             cd '${WORKSPACE}'; \
+            wget https://raw.githubusercontent.com/${{ env.CI_REPO }}/refs/heads/${{ env.CI_BRANCH }}/ci/shared/scripts/deps/packages.sh && chmod +x packages.sh
+            ./packages.sh lcov
             ls -l;
             chmod +x ./build.sh; \
             \
@@ -210,4 +213,4 @@ jobs:
         if: always()
         with:
           name: foo-output-${{ matrix.runner.name }}
-          path: /opt/foo/output
+          path: ${{ env.OUTPUT_DIR }}