Improve coverage report format in GitHub pull requests.

Calculate and upload coverage reports to our own Metrics Service instead
of Codecov.

Author: yifanyang

build.gradle

@@ -57,7 +57,7 @@ apply plugin: com.google.firebase.gradle.plugins.publish.PublishingPlugin
 apply plugin: com.google.firebase.gradle.plugins.ci.ContinuousIntegrationPlugin
 apply plugin: com.google.firebase.gradle.plugins.ci.SmokeTestsPlugin
 apply plugin: com.google.firebase.gradle.plugins.ci.metrics.MetricsPlugin
-apply plugin: com.google.firebase.gradle.plugins.ci.CheckCoveragePlugin
+apply plugin: com.google.firebase.gradle.plugins.measurement.coverage.CheckCoveragePlugin
 
 firebaseContinuousIntegration {
     ignorePaths = [

buildSrc/src/main/groovy/com/google/firebase/gradle/plugins/ci/CheckCoveragePlugin.groovy

@@ -34,13 +34,14 @@ public static void upload(Project project, String report) {
 
     String owner = System.getenv("REPO_OWNER");
     String repo = System.getenv("REPO_NAME");
+    String branch = System.getenv("PULL_BASE_REF");
     String baseCommit = System.getenv("PULL_BASE_SHA");
     String headCommit = System.getenv("PULL_PULL_SHA");
     String pullRequest = System.getenv("PULL_NUMBER");
 
-    String commit = headCommit != null && !headCommit.isEmpty() ? headCommit : headCommit;
+    String commit = headCommit != null && !headCommit.isEmpty() ? headCommit : baseCommit;
 
-    post(project, report, owner, repo, commit, baseCommit, pullRequest);
+    post(project, report, owner, repo, commit, branch, baseCommit, pullRequest);
   }
 
   private static void post(
@@ -49,17 +50,18 @@ private static void post(
       String owner,
       String repo,
       String commit,
+      String branch,
       String baseCommit,
       String pullRequest) {
     String post = "-X POST";
     String headerAuth = "-H \"Authorization: Bearer $(gcloud auth print-identity-token)\"";
     String headerContentType = "-H \"Content-Type: application/json\"";
     String body = String.format("-d @%s", report);
 
-    String template = "%s/repos/%s/%s/commits/%s/reports";
-    String endpoint = String.format(template, METRICS_SERVICE_URL, owner, repo, commit);
+    String template = "%s/repos/%s/%s/commits/%s/reports/?branch=%s";
+    String endpoint = String.format(template, METRICS_SERVICE_URL, owner, repo, commit, branch);
     if (pullRequest != null && !pullRequest.isEmpty()) {
-      endpoint += String.format("?base_commit=%s&pull_request=%s", baseCommit, pullRequest);
+      endpoint += String.format("&base_commit=%s&pull_request=%s", baseCommit, pullRequest);
     }
 
     String request =

buildSrc/src/main/groovy/com/google/firebase/gradle/plugins/measurement/MetricsReportUploader.java

@@ -0,0 +1,80 @@
+// Copyright 2020 Google LLC
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//      http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package com.google.firebase.gradle.plugins.measurement.coverage
+
+import com.google.firebase.gradle.plugins.FirebaseLibraryExtension
+import com.google.firebase.gradle.plugins.FirebaseLibraryPlugin
+import org.gradle.api.Plugin
+import org.gradle.api.Project
+import org.gradle.api.tasks.testing.Test
+import org.gradle.testing.jacoco.tasks.JacocoReport
+
+class CheckCoveragePlugin implements Plugin<Project> {
+
+    @Override
+    void apply(Project project) {
+        project.configure(project.subprojects) { sub ->
+            apply plugin: 'jacoco'
+
+            def reportsDir = "${buildDir}/reports/jacoco"
+            jacoco {
+                toolVersion = '0.8.5'
+                reportsDir = reportsDir
+            }
+
+            plugins.withType(FirebaseLibraryPlugin.class) {
+
+                sub.tasks.withType(Test) {
+                    jacoco {
+                        excludes = ['jdk.internal.*']
+                        includeNoLocationClasses = true
+                    }
+                }
+
+                sub.task('checkCoverage', type: JacocoReport) {
+                    dependsOn 'check'
+                    description 'Generates JaCoCo check coverage report.'
+                    group 'verification'
+
+                    def excludes = [
+                            '**/R.class',
+                            '**/R$*.class',
+                            '**/BuildConfig.*',
+                            '**/proto/**',
+                            '**Manifest*.*'
+                    ]
+                    classDirectories = files([
+                            fileTree(dir: "$buildDir/intermediates/javac/release", excludes: excludes),
+                            fileTree(dir: "$buildDir/tmp/kotlin-classes/release", excludes: excludes),
+                    ])
+                    sourceDirectories = files(['src/main/java', 'src/main/kotlin'])
+                    executionData = fileTree(dir: "$buildDir", includes: ['jacoco/*.exec'])
+
+                    def firebaseLibrary = sub.extensions.getByType(FirebaseLibraryExtension.class)
+                    def artifactId = firebaseLibrary.artifactId.get()
+                    reports {
+                        html.destination file("${reportsDir}/${artifactId}/html")
+                        xml {
+                            enabled true
+                            destination file("${reportsDir}/${artifactId}.xml")
+                        }
+                    }
+
+                    outputs.upToDateWhen { false }
+                }
+            }
+        }
+    }
+}