Fixed

Bug

Status Update

No update yet.

Description

au...@google.com

created issue #1

Jun 23, 2022 12:41AM

DESCRIBE THE ISSUE IN DETAIL: Stop creating androidJacocoAnt configuration if coverage is not enabled. We are wasting resources to create it.

STEPS TO REPRODUCE:

Have a project that does not have coverage enabled
./gradlew app:dependencies

Expected:

There is no jacoco listed in dependencies

Actual

There is:

androidJacocoAnt - The Jacoco agent to use to get coverage data.
\--- org.jacoco:org.jacoco.ant:0.8.8

https://ge.androidx.dev/s/jw2l6ljceb4qq/console-log#L89

Studio Build: EE Canary 5
Version of Gradle Plugin: AGP 7.4.0-alpha05
Version of Gradle: 7.5-rc-2
Version of Java: 11
OS: linux

Comments

sp...@google.com <sp...@google.com> Jun 23, 2022 12:46AM

Assigned to an...@google.com.

je...@google.com <je...@google.com> Jun 26, 2022 12:07PM

Reassigned to cm...@google.com.

lu...@google.com <lu...@google.com> #2Sep 28, 2022 02:10PM

Accepted by lu...@google.com.

As I understand it, we need to migrate namespace definitions both from AndroidManifest.xml and from implicit definitions from applicationId / testApplicationId? Is that right? So an 8.0 project would have a

namespace <n>
testNamespace <tn>

where <n> is taken from AndroidManifest.xml package definitions if present, and applicationId if not, and <tn> is testApplicationId if present and <n>.test if not?

lu...@google.com <lu...@google.com> Sep 29, 2022 07:55AM

Marked as fixed.