Remove sysout call, fix travis again, add coverage.

zchunk · May 19, 2019 · a4d47b9 · a4d47b9
1 parent 4f67b49
commit a4d47b9
Show file tree

Hide file tree

Showing 3 changed files with 57 additions and 15 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -20,8 +20,8 @@ before_install: test -z "$TRAVIS_JDK" || curl -Ls https://git.io/jabba | bash &&
 
 install:
   # Only install and use JDK if it is set. If empty, use system JDK.
-  - test -z "$TRAVIS_JDK" || jabba install "$TRAVIS_JDK");
-  - test -z "$TRAVIS_JDK" || jabba use "$TRAVIS_JDK");
+  - test -z "$TRAVIS_JDK" || jabba install "$TRAVIS_JDK";
+  - test -z "$TRAVIS_JDK" || jabba use "$TRAVIS_JDK";
   # Print the JDK we are going to use.
   - java -Xmx32m -version
   # due to a bug, dependency resolve will try to resolve reactor projects from subfolders, although
@@ -48,4 +48,5 @@ cache:
     - "$HOME/.m2/repository"
     - "$HOME/.jabba/jdk"
 
-
+after_success:
+  - bash <(curl -s https://codecov.io/bash)
diff --git a/compressedint/src/test/java/io/github/zchunk/compressedint/CompressedIntTest.java b/compressedint/src/test/java/io/github/zchunk/compressedint/CompressedIntTest.java
@@ -59,7 +59,6 @@ public void testToCompressedInt_ulong_max() {
     final CompressedInt bytes = CompressedIntFactory.valueOf(unsignedLongValue);
 
     final String binaryString = byteArrayToBinaryString(bytes.getCompressedBytes());
-    System.out.println("" + new BigInteger(1, bytes.getCompressedBytes()).toString(16));
 
     Assertions.assertEquals("1111111011111110111111101111111011111110111111101111111011111110111111110000001", binaryString);
   }

diff --git a/pom.xml b/pom.xml
@@ -105,17 +105,6 @@
           </configuration>
         </plugin>
 
-        <plugin>
-          <artifactId>maven-surefire-plugin</artifactId>
-          <version>3.0.0-M3</version>
-          <configuration>
-            <systemPropertyVariables>
-              <java.util.logging.config.file>${project.build.testOutputDirectory}/logging.properties</java.util.logging.config.file>
-              <user.language>EN</user.language>
-            </systemPropertyVariables>
-          </configuration>
-        </plugin>
-
         <plugin>
           <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-enforcer-plugin</artifactId>
@@ -128,6 +117,27 @@
           <version>3.1.1</version>
         </plugin>
 
+        <!-- test agent for coverage -->
+        <plugin>
+          <groupId>org.jacoco</groupId>
+          <artifactId>jacoco-maven-plugin</artifactId>
+          <version>0.8.4</version>
+        </plugin>
+
+        <plugin>
+          <artifactId>maven-surefire-plugin</artifactId>
+          <version>3.0.0-M3</version>
+          <configuration>
+            <systemPropertyVariables>
+              <java.util.logging.config.file>
+                ${project.build.testOutputDirectory}/logging.properties
+              </java.util.logging.config.file>
+              <user.language>EN</user.language>
+            </systemPropertyVariables>
+            <argLine>@{surefireArgLine}</argLine>
+          </configuration>
+        </plugin>
+
         <!-- maven checkstyle -->
         <plugin>
           <groupId>org.apache.maven.plugins</groupId>
@@ -180,6 +190,38 @@
           </rules>
         </configuration>
       </plugin>
+
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-surefire-plugin</artifactId>
+      </plugin>
+
+      <plugin>
+        <groupId>org.jacoco</groupId>
+        <artifactId>jacoco-maven-plugin</artifactId>
+        <executions>
+          <!-- set arg line property for surefire -->
+          <execution>
+            <id>prepare-agent</id>
+            <goals>
+              <goal>prepare-agent</goal>
+            </goals>
+            <configuration>
+              <propertyName>surefireArgLine</propertyName>
+            </configuration>
+          </execution>
+          <!-- set arg line property for failsafe -->
+          <execution>
+            <id>prepare-agent-integration</id>
+            <goals>
+              <goal>prepare-agent-integration</goal>
+            </goals>
+            <configuration>
+              <propertyName>failsafeArgLine</propertyName>
+            </configuration>
+          </execution>
+        </executions>
+      </plugin>
     </plugins>
   </build>