Add codecov (#3842)

chingor13 · web-flow · commit 69e2cdab8cff · 2018-10-23T19:05:49.000-07:00
* Setup Kokoro to run codecov script after tests

* Enable jacoco plugin to capture code coverage in clients
diff --git a/.kokoro/build.sh b/.kokoro/build.sh
@@ -31,6 +31,7 @@ fi
 case $JOB_TYPE in
 test)
     mvn test -B
+    bash $KOKORO_GFILE_DIR/codecov.sh
     ;;
 javadoc)
     mvn javadoc:javadoc javadoc:test-javadoc
diff --git a/.kokoro/continuous/common.cfg b/.kokoro/continuous/common.cfg
@@ -28,8 +28,6 @@ before_action {
     keystore_resource {
       keystore_config_id: 73713
       keyname: "java_it_service_account"
-      # TODO(chingor): remove this after secrets have globally propagated
-      backend_type: FASTCONFIGPUSH
     }
   }
 }
@@ -39,8 +37,16 @@ before_action {
     keystore_resource {
       keystore_config_id: 73713
       keyname: "java_firestore_it_service_account"
-      # TODO(chingor): remove this after secrets have globally propagated
-      backend_type: FASTCONFIGPUSH
+    }
+  }
+}
+
+# Request the codecov master token
+before_action {
+  fetch_keystore {
+    keystore_resource {
+      keystore_config_id: 73713
+      keyname: "dpebot_codecov_token"
     }
   }
 }
diff --git a/.kokoro/presubmit/common.cfg b/.kokoro/presubmit/common.cfg
@@ -28,8 +28,6 @@ before_action {
     keystore_resource {
       keystore_config_id: 73713
       keyname: "java_it_service_account"
-      # TODO(chingor): remove this after secrets have globally propagated
-      backend_type: FASTCONFIGPUSH
     }
   }
 }
@@ -39,8 +37,17 @@ before_action {
     keystore_resource {
       keystore_config_id: 73713
       keyname: "java_firestore_it_service_account"
-      # TODO(chingor): remove this after secrets have globally propagated
-      backend_type: FASTCONFIGPUSH
     }
   }
 }
+
+# Request the codecov master token
+before_action {
+  fetch_keystore {
+    keystore_resource {
+      keystore_config_id: 73713
+      keyname: "dpebot_codecov_token"
+    }
+  }
+}
+
diff --git a/codecov.yaml b/codecov.yaml
@@ -0,0 +1,4 @@
+---
+codecov:
+  ci:
+    - source.cloud.google.com
diff --git a/google-api-grpc/pom.xml b/google-api-grpc/pom.xml
@@ -788,6 +788,25 @@
           </reportPlugins>
         </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>
   <reporting>
diff --git a/google-cloud-bom/pom.xml b/google-cloud-bom/pom.xml
@@ -1127,6 +1127,25 @@
           </reportPlugins>
         </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>
 
diff --git a/google-cloud-clients/pom.xml b/google-cloud-clients/pom.xml
@@ -765,6 +765,25 @@
           </reportPlugins>
         </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>
 

Original file line number	Diff line number	Diff line change
`@@ -28,8 +28,6 @@ before_action {`
`28`	`28`	`keystore_resource {`
`29`	`29`	`keystore_config_id: 73713`
`30`	`30`	`keyname: "java_it_service_account"`
`31`		`- # TODO(chingor): remove this after secrets have globally propagated`
`32`		`- backend_type: FASTCONFIGPUSH`
`33`	`31`	`}`
`34`	`32`	`}`
`35`	`33`	`}`
`@@ -39,8 +37,16 @@ before_action {`
`39`	`37`	`keystore_resource {`
`40`	`38`	`keystore_config_id: 73713`
`41`	`39`	`keyname: "java_firestore_it_service_account"`
`42`		`- # TODO(chingor): remove this after secrets have globally propagated`
`43`		`- backend_type: FASTCONFIGPUSH`
	`40`	`+ }`
	`41`	`+ }`
	`42`	`+}`
	`43`	`+`
	`44`	`+# Request the codecov master token`
	`45`	`+before_action {`
	`46`	`+ fetch_keystore {`
	`47`	`+ keystore_resource {`
	`48`	`+ keystore_config_id: 73713`
	`49`	`+ keyname: "dpebot_codecov_token"`
`44`	`50`	`}`
`45`	`51`	`}`
`46`	`52`	`}`
-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +---
 +codecov:
 +  ci:
 +    - source.cloud.google.com