Skip to content

Commit

Permalink
Switch coverage plugin from cobertura to Jacoco
Browse files Browse the repository at this point in the history
There is a know bug in cobertura with Java 9
See cobertura/cobertura#271

Update badge in README file

Update pom for jacoco plugin
  • Loading branch information
gkoudjou committed Jul 2, 2019
1 parent 2c49b99 commit 37cf181
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 15 deletions.
2 changes: 0 additions & 2 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@
# This enables the 'defaults' to test java applications:
language: java

script: "mvn cobertura:cobertura"

# We can specify a list of JDKs to be used for testing
# A list of available JDKs in Trusty can be seed in:
# https://docs.travis-ci.com/user/reference/trusty/#jvm-clojure-groovy-java-scala-images
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
[![Build Status](https://api.travis-ci.com/gkoudjou/primesum.svg?branch=master)](https://api.travis-ci.com/gkoudjou/primesum.svg?branch=master)
[![Code Coverage](https://codecov.io/github/gkoudjou/primesum/coverage.svg)](https://codecov.io/github/gkoudjou/primesum/coverage.svg)
[![codecov](https://codecov.io/gh/gkoudjou/primesum/branch/master/graph/badge.svg)](https://codecov.io/gh/gkoudjou/primesum)

# PrimeSum
Approach to solving the problem of the sum of the first N primes. Using stream.
Expand Down
31 changes: 19 additions & 12 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -74,18 +74,6 @@
<artifactId>maven-project-info-reports-plugin</artifactId>
<version>3.0.0</version>
</plugin>
<plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>cobertura-maven-plugin</artifactId>
<version>2.7</version>
<configuration>
<formats>
<format>html</format>
<format>xml</format>
</formats>
<check />
</configuration>
</plugin>
</plugins>
</pluginManagement>
<plugins>
Expand All @@ -97,6 +85,25 @@
<target>8</target>
</configuration>
</plugin>
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
<version>0.8.2</version>
<executions>
<execution>
<goals>
<goal>prepare-agent</goal>
</goals>
</execution>
<execution>
<id>report</id>
<phase>test</phase>
<goals>
<goal>report</goal>
</goals>
</execution>
</executions>
</plugin>
</plugins>
</build>
</project>

0 comments on commit 37cf181

Please sign in to comment.