#303 - Add code coverage support improve (#304)

* #303 - Add code coverage support improve
- Added support for code coverage GitHub action check as part of PR.
- Used Absa fork of JaCoCo solution - with scala method filtering.

Author: miroslavpojer

.github/workflows/build.yml

@@ -35,4 +35,31 @@ jobs:
       - name: Switch scala version
         run: ./mvnw scala-cross-build:change-version --no-transfer-progress -Pscala-${{ matrix.scala }},spark-${{ matrix.spark }}
       - name: Build and run tests
-        run: ./mvnw clean verify --no-transfer-progress -Pscala-${{ matrix.scala }},spark-${{ matrix.spark }},all-tests,no-upgraded-snappy
+        run: ./mvnw clean verify --no-transfer-progress -Pscala-${{ matrix.scala }},spark-${{ matrix.spark }},all-tests,no-upgraded-snappy,code-coverage
+      - name: Add coverage to PR
+        id: jacoco
+        uses: madrapps/[email protected]
+        with:
+          paths: >
+            ${{ github.workspace }}/api/target/site/jacoco/jacoco.xml,
+            ${{ github.workspace }}/compatibility-provider/target/site/jacoco/jacoco.xml,
+            ${{ github.workspace }}/compatibility_spark-${{ matrix.spark }}/target/site/jacoco/jacoco.xml,
+            ${{ github.workspace }}/component-scanner/target/site/jacoco/jacoco.xml,
+            ${{ github.workspace }}/driver/target/site/jacoco/jacoco.xml,
+            ${{ github.workspace }}/ingestor-default/target/site/jacoco/jacoco.xml,
+            ${{ github.workspace }}/shared/target/site/jacoco/jacoco.xml
+          token: ${{ secrets.GITHUB_TOKEN }}
+          min-coverage-overall: 0.0
+          min-coverage-changed-files: 80.0
+          title: JaCoCo code coverage report - Scala ${{ matrix.scala }} & Spark ${{ matrix.spark }}
+          update-comment: true
+      - name: Get the Coverage info
+        run: |
+          echo "Total coverage ${{ steps.jacoco.outputs.coverage-overall }}"
+          echo "Changed Files coverage ${{ steps.jacoco.outputs.coverage-changed-files }}"
+      - name: Fail PR if changed files coverage is less than 80%
+        if: ${{ steps.jacoco.outputs.coverage-changed-files < 80.0 }}
+        uses: actions/github-script@v6
+        with:
+          script: |
+            core.setFailed('Changed files coverage is less than 80%!')

parent-conf/pom.xml

@@ -79,7 +79,7 @@
 
         <!--Other properties-->
         <skip.docker.tests>true</skip.docker.tests>
-        <jacoco.plugin.version>0.8.8</jacoco.plugin.version>
+        <jacoco.plugin.version>0.8.9</jacoco.plugin.version>
         <spark-commons-test>0.4.0</spark-commons-test>
     </properties>
 
@@ -418,12 +418,12 @@
         <profile>
             <id>code-coverage</id>
             <properties>
-                <skip.docker.tests>false</skip.docker.tests>
+                <skip.docker.tests>true</skip.docker.tests>
             </properties>
             <build>
                 <plugins>
                     <plugin>
-                        <groupId>org.jacoco</groupId>
+                        <groupId>za.co.absa.jacoco</groupId>
                         <artifactId>jacoco-maven-plugin</artifactId>
                         <version>${jacoco.plugin.version}</version>
                         <configuration>
@@ -445,6 +445,11 @@
                                 <goals>
                                     <goal>report</goal>
                                 </goals>
+                                <configuration>
+                                    <title>${project.name} - scala:${scala.version}</title>
+                                    <doMethodFiltration>true</doMethodFiltration>
+                                    <doScalaMethodFiltration>true</doScalaMethodFiltration>
+                                </configuration>
                             </execution>
                         </executions>
                     </plugin>