Added androidTest coverage support
#16
Conversation
gradle/libs.versions.toml
Outdated
| @@ -1,6 +1,6 @@ | |||
| [versions] | |||
| kotlin = "1.8.21" | |||
| agp = "8.0.0" | |||
| agp = "8.2.0-alpha02" | |||
There was a problem hiding this comment.
There is an issue with 8.0.1 producing empty results for library modules
| import kotlin.reflect.jvm.isAccessible | ||
| import kotlin.reflect.typeOf | ||
|
|
||
| internal fun <FILE_TYPE : FileSystemLocation> Artifacts.get( |
There was a problem hiding this comment.
This is a blocker to me, I don't want to release code doing all this dark magic to access the artifacts
| } | ||
| } | ||
|
|
||
| if (variant.unitTest != null && buildType.enableUnitTestCoverage) { | ||
| aggregate(variant.unitTest!!.artifacts.get(UNIT_TEST_CODE_COVERAGE)) |
There was a problem hiding this comment.
This internal artifact points to the "build/jacoco/testXXXUnitTest.exec" files
| files(this).asFileTree.forEach { file -> | ||
| outgoing.artifact(file) { | ||
| type = ArtifactTypeDefinition.BINARY_DATA_TYPE | ||
| builtBy(this@file) | ||
| } | ||
| } |
There was a problem hiding this comment.
And this is not working well, as for AndroidTest we know in advance the target output directory, but not all the .ec files getting generated by each emulator.
| into(provider { temporaryDir }) | ||
| duplicatesStrategy = DuplicatesStrategy.INCLUDE // in case of duplicated classes | ||
| include("**/*.class") |
There was a problem hiding this comment.
Now required, as the PRE_JACOCO_TRANSFORMED_CLASSES also includes R.jar from its dependencies
| aggregatedVariants.all task@{ | ||
| from(artifacts | ||
| .forScope(ScopedArtifacts.Scope.PROJECT) | ||
| .get(PRE_JACOCO_TRANSFORMED_CLASSES)) |
There was a problem hiding this comment.
I've been forced to change from ScopedArtifact.CLASSES (public API) to InternalScopedArtifact .PRE_JACOCO_TRANSFORMED_CLASSES because ScopedArtifact.CLASSES gets replaced with JaCoCo instrumented classes when AndroidTest coverage is enabled. JaCoCo report task won't accept instrumented classes
gmazzo
force-pushed
the
android-test-coverage-support
branch
2 times, most recently
from
81fbcfe
to
f690d06
Compare
gmazzo
force-pushed
the
android-test-coverage-support
branch
from
f690d06
to
6945d09
Compare
Draft WIP implementing #15: support for
AndrodiTestvariant.The aggregation functionally was extended to also allow including
AndroidTestvariants.The current implementation is hacky, accessing AGP's internal APIs by reflection. More details here .