Skip to content

Commit

Permalink
Fix jacoco report paths (#116)
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
  • Loading branch information
jandroav committed Oct 25, 2023
1 parent c0c5e3f commit fbd6ac2
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion .github/workflows/sonar-pull-request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,6 @@ jobs:
uses: actions/download-artifact@v3
with:
name: test-reports-jdk-17-ubuntu-latest
path: ./target

- name: Sonar Scan
env:
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/sonar-push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,6 @@ jobs:
uses: actions/download-artifact@v3
with:
name: test-reports-jdk-17-ubuntu-latest
path: ./target

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

0 comments on commit fbd6ac2

Please sign in to comment.