build(CodeCov): Enable CodeCov. #626
Conversation
googlebot
added
the
cla: yes
|
One of the Travis builds timed out, so I just restarted it (the rest passed). |
Codecov Report
@@ Coverage Diff @@
## master #626 +/- ##
=========================================
Coverage ? 18.59%
=========================================
Files ? 71
Lines ? 3980
Branches ? 585
=========================================
Hits ? 740
Misses ? 3206
Partials ? 34Continue to review full report at Codecov.
|
| @@ -96,7 +96,7 @@ install: | |||
| - source ~/.install-jdk-travis.sh | |||
|
|
|||
| script: | |||
| - if [ "$API" != "NONE" ]; then ./gradlew test check connectedCheck -x lint --stacktrace; fi | |||
| - if [ "$API" != "NONE" ]; then ./gradlew test check jacocoTestReport connectedCheck -x lint --stacktrace; fi | |||
There was a problem hiding this comment.
test executes the JUnit tests we have in the library/src/test, and connectedCheck executes the instrumented tests we have in library/src/androidTest. Do you know if jacocoTestReport covers the output of both of these?
There was a problem hiding this comment.
jacocoTestReport will only evaluate the existence of a test (unit or instrumented test) for a particular set of classes (i.e. classes within the :library module). The actual result of the unit and instrumented tests are currently handled by Travis and not CodeCov
Example CodeCov result:
|
I misread the CodeCov report and thought that reports weren't being uploaded which is true for |
Enabling CodeCov on Travis so that code coverage reports can be generated and evaluated on a per-PR basis.