Skip to content

Commit

Permalink
Fix jacoco report paths (#117)
Browse files Browse the repository at this point in the history
* chore(workflows): update test report paths in GitHub workflows

- In os-extension-test.yml and pro-extension-test.yml, change the path for jacoco.xml from **/target/jacoco.exec to **/target/site/jacoco/jacoco.xml. This is done to correctly locate the Jacoco code coverage report.
- In sonar-pull-request.yml and sonar-push.yml, update the name of the test report artifact to test-reports-jdk-17-ubuntu-latest. This is done to ensure consistency in the naming of the test report artifacts across workflows.

* fix(sonar-pull-request.yml): update action for downloading unit tests report to use the correct syntax and version
fix(sonar-push.yml): update action for downloading unit tests report to use the correct syntax and version

* fix(workflows): update Maven test command to use 'test' goal and 'coverage' profile for better code coverage reporting

The Maven test command in both os-extension-test.yml and pro-extension-test.yml workflows has been updated to use the 'test' goal and 'coverage' profile. This change will improve code coverage reporting for the tests.

* chore(sonar-pull-request.yml): remove unnecessary path argument in download-artifact step
chore(sonar-push.yml): remove unnecessary path argument in download-artifact step

* fix(sonar-push.yml): update action for downloading test results artifact to use dawidd6/action-download-artifact@v2
feat(sonar-push.yml): add support for downloading test results artifact from a specific workflow and repository
  • Loading branch information
jandroav authored Oct 25, 2023
1 parent fbd6ac2 commit 769df03
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions .github/workflows/sonar-push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,12 @@ jobs:
- name: Download unit tests report
uses: actions/download-artifact@v3
with:
github_token: ${{secrets.BOT_TOKEN}}
workflow: test.yml
name: test-reports-jdk-17-ubuntu-latest
repo: ${{ github.repository }}
if_no_artifact_found: warn
workflow_conclusion: ""

# look for dependencies in maven
- name: maven-settings-xml-action
Expand Down

0 comments on commit 769df03

Please sign in to comment.