fix coverage report

alibaba · Apr 16, 2024 · 790eb50 · 790eb50
1 parent a1613b5
commit 790eb50
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 14 deletions.
diff --git a/.github/workflows/maven.yml b/.github/workflows/maven.yml
@@ -33,7 +33,7 @@ jobs:
         cache: maven
     - name: Build with coveralls report
       if: ${{ env.cov_token != '' }}
-      run: mvn -B -DjacocoReports=jetcache-test/target/site/jacoco-aggregate/jacoco.xml clean test org.jacoco:jacoco-maven-plugin:0.8.8:report-aggregate org.eluder.coveralls:coveralls-maven-plugin:4.3.0:report -DrepoToken=${{ secrets.COV_TOKEN }} -Dtick=3
+      run: mvn -B -DjacocoReports=jetcache-test/target/site/jacoco-aggregate/jacoco.xml clean install org.eluder.coveralls:coveralls-maven-plugin:4.3.0:report -DrepoToken=${{ secrets.COV_TOKEN }} -Dtick=3
     - name: Build without coveralls report
       if: ${{ env.cov_token == '' }}
       run: mvn -B clean test
diff --git a/jetcache-test/pom.xml b/jetcache-test/pom.xml
@@ -170,6 +170,12 @@
                 <artifactId>jacoco-maven-plugin</artifactId>
                 <version>0.8.8</version>
                 <executions>
+                    <execution>
+                        <id>prepare-agent</id>
+                        <goals>
+                            <goal>prepare-agent</goal>
+                        </goals>
+                    </execution>
                     <execution>
                         <id>report-aggregate</id>
                         <phase>verify</phase>

diff --git a/pom.xml b/pom.xml
@@ -58,19 +58,6 @@
     </properties>
     <build>
         <plugins>
-            <plugin>
-                <groupId>org.jacoco</groupId>
-                <artifactId>jacoco-maven-plugin</artifactId>
-                <version>0.8.8</version>
-                <executions>
-                    <execution>
-                        <id>prepare-agent</id>
-                        <goals>
-                            <goal>prepare-agent</goal>
-                        </goals>
-                    </execution>
-                </executions>
-            </plugin>
             <plugin>
                 <groupId>org.eluder.coveralls</groupId>
                 <artifactId>coveralls-maven-plugin</artifactId>