From c45dd11d1133c24c2beae947138b4e3e71c9bfe9 Mon Sep 17 00:00:00 2001 From: Stian Sandvold Date: Sun, 31 May 2026 08:45:12 +0200 Subject: [PATCH] chore(ci): remove the temporary jacoco coverage-debug artifacts The unit-test and integration-h2-test jobs each tarred up the jacoco-aggregate report and uploaded it as a build artifact, behind a 'delete the next 2 steps once we are confident in the coverage setup' comment. Those debug steps were added back in 2022 and have been running ever since. Coverage reporting does not depend on them: codecov is fed directly by the codecov-action step, which reads the jacoco reports itself. The tar/upload only produced a downloadable artifact for manual inspection, which has gone unused. Remove the four steps (two per job). --- .github/workflows/run-tests.yml | 18 ------------------ 1 file changed, 18 deletions(-) diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml index 49d41611e4d7..8607b9e75ec9 100644 --- a/.github/workflows/run-tests.yml +++ b/.github/workflows/run-tests.yml @@ -52,15 +52,6 @@ jobs: flags: unit fail_ci_if_error: false verbose: true - # delete the next 2 steps once we are confident in the coverage setup - - name: Tar jacoco coverage report to debug - run: tar -cvf coverage.tar dhis-2/dhis-test-coverage/target/site/jacoco-aggregate - - uses: actions/upload-artifact@v7 - name: Upload jacoco coverage report to debug - with: - name: unit-test-coverage - path: coverage.tar - retention-days: 5 - name: Generate surefire aggregate report run: mvn surefire-report:report-only -Daggregate=true --batch-mode --no-transfer-progress --file ./dhis-2/pom.xml --projects -dhis-test-coverage # tar due to https://github.com/actions/upload-artifact/blob/3cea5372237819ed00197afe530f5a7ea3e805c8/README.md?plain=1#L254 @@ -223,15 +214,6 @@ jobs: flags: integration-h2 fail_ci_if_error: false verbose: true - # delete the next 2 steps once we are confident in the coverage setup - - name: Tar jacoco coverage report to debug - run: tar -cvf coverage.tar dhis-2/dhis-test-coverage/target/site/jacoco-aggregate - - uses: actions/upload-artifact@v7 - name: Upload jacoco coverage report to debug - with: - name: integration-h2-test-coverage - path: coverage.tar - retention-days: 5 - name: Generate surefire aggregate report run: mvn surefire-report:report-only -Daggregate=true --batch-mode --no-transfer-progress --file ./dhis-2/pom.xml --projects -dhis-test-coverage # tar due to https://github.com/actions/upload-artifact/blob/3cea5372237819ed00197afe530f5a7ea3e805c8/README.md?plain=1#L254