Skip to content

Commit

Permalink
CIF-1777 - Jacoco build issue with CIF components library and CloudMa…
Browse files Browse the repository at this point in the history 
…nager (#455)

- switch jacoco to on-the-fly instrumentation to fix build issue in CloudManager
- remove the no-longer needed runtime dependency for offline instrumentation
- increase minimum coverage
  • Loading branch information
@cjelger
cjelger authored Dec 15, 2020
1 parent 099f870 commit 13c2807
Showing 1 changed file with 4 additions and 18 deletions.
22 changes: 4 additions & 18 deletions bundles/core/pom.xml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -177,15 +177,10 @@
<artifactId>jacoco-maven-plugin</artifactId>
<executions>
<execution>
<id>default-instrument</id>
<id>prepare-agent</id>
<phase>test-compile</phase>
<goals>
<goal>instrument</goal>
</goals>
</execution>
<execution>
<id>default-restore-instrumented-classes</id>
<goals>
<goal>restore-instrumented-classes</goal>
<goal>prepare-agent</goal>
</goals>
</execution>
<execution>
Expand All @@ -203,7 +198,7 @@
<limit>
<counter>BRANCH</counter>
<value>COVEREDRATIO</value>
<minimum>0.55</minimum>
<minimum>0.75</minimum>
</limit>
</limits>
</rule>
Expand Down Expand Up @@ -250,15 +245,6 @@
</plugin>
</plugins>
</build>
<dependencies>
<dependency>
<groupId>org.jacoco</groupId>
<artifactId>org.jacoco.agent</artifactId>
<version>${jacoco.version}</version>
<classifier>runtime</classifier>
<scope>test</scope>
</dependency>
</dependencies>
</profile>

<profile>
Expand Down

0 comments on commit 13c2807

Please sign in to comment.