Skip to content

Commit

Permalink
CI: merge code coverage with asserting builds (#792)
Browse files Browse the repository at this point in the history
* Update verify_build.yml
Add code coverage publishing
* Delete test_coverage.yml
  • Loading branch information
asmfstatoil authored Jul 31, 2023
1 parent 873d862 commit 2d3d7be
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 33 deletions.
29 changes: 0 additions & 29 deletions .github/workflows/test_coverage.yml

This file was deleted.

14 changes: 10 additions & 4 deletions .github/workflows/verify_build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,6 @@ on:
branches: # Only run for changes in master branch and any releases branch
- master
- 'releases/**'
# paths: # Only run if some java-file is updated
# - '*.java'
pull_request:
branches:
- master
Expand All @@ -27,7 +25,15 @@ jobs:
- name: Generate javadoc
run: mvn javadoc:javadoc
- name: Run tests
run: mvn -B test --file pom.xml
run: mvn -B test --file pom.xml -ntp
- name: Create coverage report from jacoco execution data
run: mvn -B jacoco:report -ntp
- name: Upload jacoco coverage reports to Codecov
uses: codecov/codecov-action@v3
with:
file: jacoco.xml
name: codecov
token: ${{ secrets.CODECOV_TOKEN }}

test_java_8:
name: Assert tests and javadoc with java 8
Expand All @@ -43,5 +49,5 @@ jobs:
java-version: '8'
cache: 'maven'
- name: Run tests
run: mvn -B test --file pomJava8.xml
run: mvn -B test --file pomJava8.xml -ntp

0 comments on commit 2d3d7be

Please sign in to comment.