diff options
| author | Eric Anderson <ejona@google.com> | 2016-02-13 14:43:38 -0800 |
|---|---|---|
| committer | Eric Anderson <ejona@google.com> | 2016-02-17 12:24:17 -0800 |
| commit | 6715f1202c1c73f556e668aed5b5dd1863ac3539 (patch) | |
| tree | 6c19fb3fe516b5a1aa194020a2aa129d8cb43c74 /all | |
| parent | daf920d4b4f3788b753d8eec470d763afcffd459 (diff) | |
| download | grpc-grpc-java-6715f1202c1c73f556e668aed5b5dd1863ac3539.tar.gz | |
Add support for codecov.io
Codecov.io provides patch-based code coverage, so you can easily know
how many of the added lines are covered. It also has a more useful UI.
Unfortunately, the percentage it reports does not include partially-
covered lines--those with uncovered conditions. Thus the reported
percentage is about 6% lower than the coverage we've been looking at
previously. Because of this alone, I don't expect to remove coveralls
support soon.
Use the bash script instead of python module since pip isn't available
by default on Travis OS X.
jacocoTestReport uses mustRunAfter (contrary to the docs; see
https://issues.gradle.org/browse/GRADLE-2960) to make sure it runs after
all tests, only if testing is taking place. We would like
:grpc-all:jacocoTestReport to behave the same way. Without it, we would
need two separate invocations of gradle (adding ~1m to Travis run) in
order to prevent getting random results depending on what tests just so
happened to have been run.
Diffstat (limited to 'all')
| -rw-r--r-- | all/build.gradle | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/all/build.gradle b/all/build.gradle index e7a235a62..b5c0b084d 100644 --- a/all/build.gradle +++ b/all/build.gradle @@ -59,6 +59,8 @@ javadoc { } task jacocoMerge(type: JacocoMerge) { + dependsOn(subprojects.jacocoTestReport.dependsOn) + mustRunAfter(subprojects.jacocoTestReport.mustRunAfter) destinationFile = file("${buildDir}/jacoco/test.exec") executionData = files(subprojects.jacocoTestReport.executionData) .plus(project(':grpc-interop-testing').jacocoTestReport.executionData) |