code coverage: simplified caching, JaCoCo ver as env var

Motivation:

    To improve maintainability and caching efficiency, the JaCoCo version is now configurable via an environment variable in build.yml,
    eliminating hardcoded values in scripts. The cache configuration was also updated to align paths and simplify version management.

Modifications:

    Added JACOCO_VERSION as an environment variable in build.yml and updated both scripts (get-jacoco.sh and generate-jacoco-report.sh) to use it.
    Simplified the cache path and key in build.yml to depend on the environment variable, ensuring the cache is only invalidated when the version changes.

Results:

    The JaCoCo version can now be updated in a single location, ensuring consistency across scripts and cache keys.
    The cache reliably restores the JaCoCo CLI when the version remains unchanged, reducing CI runtime.

Testing:

    Validated locally and in CI by confirming the correct JaCoCo version is used in scripts and that the cache behaves as expected.

Acked-by:
Target: master
Require-book: no
Require-notes: no
Committed:

Author: chubnastia

.github/workflows/build.yml

@@ -5,6 +5,9 @@ on:
   pull_request:
     types: [opened, reopened]
 
+env:
+  JACOCO_VERSION: "0.8.14"
+
 jobs:
   build:
     runs-on: ubuntu-latest
@@ -22,10 +25,8 @@ jobs:
         id: cache-jacoco
         uses: actions/cache@v4
         with:
-            path: ~/jacoco-0.8.14
-            key: ${{ runner.os }}-jacoco-cli-${{ hashFiles('generate-jacoco-report.sh') }}
-            restore-keys: |
-                ${{ runner.os }}-jacoco-cli-
+            path: ${{ github.workspace }}/jacoco-${{ env.JACOCO_VERSION }}
+            key: ${{ runner.os }}-jacoco-cli-${{ env.JACOCO_VERSION }}
 
       - name: Build with Maven
         run: mvn --batch-mode --update-snapshots package

generate-jacoco-report.sh

@@ -2,7 +2,7 @@
 
 # Define paths
 PROJECT_ROOT="${PROJECT_ROOT:-$(pwd)}"
-JACOCO_VERSION="0.8.14"
+JACOCO_VERSION="${JACOCO_VERSION:-0.8.14}"  #fallback to 0.8.14 if env var is unset
 JACOCO_DIR="${PROJECT_ROOT}/jacoco-$JACOCO_VERSION"
 JACOCO_CLI_JAR="$JACOCO_DIR/lib/jacococli.jar"
 MERGED_EXEC="$PROJECT_ROOT/target/coverage-reports/merged.exec"

get-jacoco.sh

@@ -2,12 +2,10 @@
 
 # Define paths
 PROJECT_ROOT="${PROJECT_ROOT:-$(pwd)}"
-JACOCO_VERSION="0.8.14"
+JACOCO_VERSION="${JACOCO_VERSION:-0.8.14}"  #fallback to 0.8.14 if env var is unset
 JACOCO_DIR="${PROJECT_ROOT}/jacoco-$JACOCO_VERSION"
 JACOCO_CLI_JAR="$JACOCO_DIR/lib/jacococli.jar"
 
-# Debug: Print paths
-echo "DEBUG: JACOCO_CLI_JAR: $JACOCO_CLI_JAR"
 echo "DEBUG: Checking if $JACOCO_CLI_JAR exists..."
 
 # Check if JaCoCo CLI JAR exists in the cache directory