Add code coverage requirements for new code

.github/workflows/publish-snapshot.yml → .github/workflows/build.yml

@@ -1,9 +1,12 @@
-name: Publish Snapshot
+name: Build
 on:
   push:
     branches:
       - master
 
+permissions:
+  contents: write
+
 jobs:
   gradle:
     runs-on: ubuntu-latest
@@ -13,7 +16,6 @@ jobs:
       with:
         distribution: temurin
         java-version: 11
-
     - name: Setup and execute Gradle 'publish' task
       uses: gradle/gradle-build-action@v2
       with:
@@ -22,3 +24,12 @@ jobs:
       env:
         OSSRH_USERNAME: ${{ vars.OSSRH_USERNAME }}
         OSSRH_TOKEN: ${{ secrets.OSSRH_TOKEN }}
+    - name: Publish Test Report
+      uses: mikepenz/action-junit-report@v3
+      if: success() || failure() # always run even if the previous step fails
+      with:
+       report_paths: '**/build/test-results/test/TEST-*.xml'
+    - name: Upload coverage reports to Codecov
+      uses: codecov/codecov-action@v3
+      env:
+        CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

.github/workflows/check-pr.yml

@@ -9,15 +9,17 @@ jobs:
       with:
         distribution: temurin
         java-version: 11
-
     - name: Setup and execute Gradle 'check' task
       uses: gradle/gradle-build-action@v2
       with:
         arguments: check -i
         build-root-directory: Src/java
-
     - name: Publish Test Report
       uses: mikepenz/action-junit-report@v3
       if: success() || failure() # always run even if the previous step fails
       with:
-        report_paths: '**/build/test-results/test/TEST-*.xml'
+        report_paths: '**/build/test-results/test/TEST-*.xml'
+    - name: Upload coverage reports to Codecov
+      uses: codecov/codecov-action@v3
+      env:
+        CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

Src/java/buildSrc/src/main/groovy/cql.java-conventions.gradle

@@ -47,10 +47,19 @@ jar {
 }
 
 jacoco {
-    toolVersion = "0.8.8"
+    toolVersion = "0.8.9"
 }
 
-check.dependsOn(jacocoTestReport)
+test {
+    finalizedBy jacocoTestReport // report is always generated after tests run
+}
+
+jacocoTestReport {
+    reports {
+        xml.required = true
+    }
+    dependsOn test // tests are required to run before generating the report
+}
 
 tasks.withType(Javadoc) {
     options.addStringOption('Xdoclint:none', '-quiet')

codecov.yaml

@@ -0,0 +1,20 @@
+coverage:
+  status:
+    project:
+      default:
+        target: auto
+        paths:
+          - "Src/java"
+    patch:
+      default:
+        # basic
+        target: 80
+        threshold: 0%
+        base: auto
+        # advanced
+        branches:
+          - master
+        if_ci_failed: error #success, failure, error, ignore
+        only_pulls: false
+        paths:
+          - "Src/java"