vanniktech / gradle-android-junit-jacoco-plugin

Gradle plugin that generates JaCoCo reports from an Android Gradle Project
http://vanniktech.com
Apache License 2.0
399 stars 72 forks source link

Don't run tests when creating the merged test coverage report. Instea… #134

Closed vRallev closed 5 years ago

vRallev commented 5 years ago

…d it's required to run specific tests manually before.

codecov[bot] commented 5 years ago

Codecov Report

Merging #134 into master will increase coverage by 19.77%. The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             master     #134       +/-   ##
=============================================
+ Coverage     66.85%   86.62%   +19.77%     
- Complexity       36       37        +1     
=============================================
  Files             1        1               
  Lines           175      172        -3     
  Branches         25       24        -1     
=============================================
+ Hits            117      149       +32     
+ Misses           46       15       -31     
+ Partials         12        8        -4
Impacted Files Coverage Δ Complexity Δ
...ktech/android/junit/jacoco/GenerationPlugin.groovy 86.62% <ø> (+19.77%) 37 <0> (+1) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 79092ae...3f6a02a. Read the comment docs.

vRallev commented 5 years ago

Hm, my Gradle and Groovy voodoo isn't really strong. I'm struggling with setting up a parent with subprojects :-/

vanniktech commented 5 years ago

Sorry. I've got sidetracked and lost this completely. You can have a look here - https://github.com/vanniktech/gradle-dependency-graph-generator-plugin/blob/c3ba1b7634436786bae5ae48c4ae57e203901d29/src/test/java/com/vanniktech/dependency/graph/generator/DependencyGraphGeneratorTest.kt#L48 - for setting up testing with parents.

vRallev commented 5 years ago

@vanniktech Thanks. I finally had time to take a look. Let me know what you think.