Fix: BuildConfig file not generated/disappearing (#57)

Apparently using `variant.sources.java.all` in AGP 7.2 causes BuildConfig files to disappear or not generate at all, this is very likely a bug in AGP 7.2 because it is working in AGP 7.3.0-beta01. For now this is a workaround that reintroduces code that utilises old non-public AGP code, but at least this code is working.

Fixes #54

Author: Rolf-Smit

plugin/build.gradle

@@ -67,7 +67,7 @@ test {
 
 dependencies {
     compileOnly gradleApi()
-    implementation libs.androidGradlePluginApi
+    implementation libs.androidGradlePlugin
 
     // Without depending on the full Android Gradle Plugin, the test fixture is unable to locate the Android Gradle Plugin.
     // Not sure why this is, but for now this is a quick fx to add it to the TestKit classpath.

plugin/src/main/kotlin/org/neotech/plugin/rootcoverage/RootCoveragePlugin.kt

@@ -5,6 +5,9 @@ import com.android.build.api.artifact.MultipleArtifact
 import com.android.build.api.dsl.BuildType
 import com.android.build.api.variant.AndroidComponentsExtension
 import com.android.build.api.variant.Variant
+import com.android.build.gradle.AppExtension
+import com.android.build.gradle.LibraryExtension
+import com.android.build.gradle.api.SourceKind
 import org.gradle.api.GradleException
 import org.gradle.api.NamedDomainObjectContainer
 import org.gradle.api.Plugin
@@ -172,7 +175,32 @@ class RootCoveragePlugin : Plugin<Project> {
             dependsOn("$path:connected${name}AndroidTest")
         }
 
-        sourceDirectories.from(variant.sources.java.all)
+
+        // Start temporary fix for AGP 7.2
+        // For some reason `variant.sources.java.all` causes BuildConfig files to go missing
+        // See: https://github.com/NeoTech-Software/Android-Root-Coverage-Plugin/issues/54
+        val baseVariant = when(val androidExtension = subProject.extensions.findByName("android")) {
+            is LibraryExtension -> androidExtension.libraryVariants
+            is AppExtension -> androidExtension.applicationVariants
+            else -> {
+                subProject.logger.warn(
+                    "Note: Skipping code coverage for module '${subProject.name}', currently the" +
+                            " RootCoveragePlugin only supports Android Library and App Modules.")
+                    return
+            }
+        }
+        baseVariant.forEach {
+            subProject.logger.warn("Name: $name, BaseName: ${it.baseName}")
+            if(name.contentEquals(it.baseName, ignoreCase = true)) {
+                val sourceFiles = it.getSourceFolders(SourceKind.JAVA).map { file -> file.dir }
+                sourceDirectories.from(subProject.files(sourceFiles))
+            }
+        }
+        // End temporary fix for AGP 7.2
+
+        // Working code in AGP 7.3-beta01:
+        // sourceDirectories.from(variant.sources.java?.all)
+
         classDirectories.from(variant.artifacts.getAll(MultipleArtifact.ALL_CLASSES_DIRS).map {
             it.map { directory ->
                 subProject.fileTree(directory.asFile, excludes = rootProjectExtension.getFileFilterPatterns())

plugin/src/test/test-fixtures/multi-module/app/build.gradle

@@ -1,5 +1,5 @@
 plugins {
-    alias(libs.plugins.androidApp)
+    id "com.android.application"
     alias(libs.plugins.kotlinAndroid)
 }
 

plugin/src/test/test-fixtures/multi-module/app/src/main/java/org/neotech/app/AppJava.java

@@ -12,7 +12,9 @@ public static synchronized AppJava getInstance() {
     }
 
     private AppJava() {
-
+        // Check if the BuildConfig file is available, this is here to prevent regression on:
+        // https://github.com/NeoTech-Software/Android-Root-Coverage-Plugin/issues/54
+        System.out.println(org.neotech.app.BuildConfig.DEBUG);
     }
 
     public String touchedByJavaInstrumentedTestInApp() {

plugin/src/test/test-fixtures/multi-module/library_android/build.gradle

@@ -1,5 +1,5 @@
 plugins {
-    alias(libs.plugins.androidLibrary)
+    id "com.android.library"
     alias(libs.plugins.kotlinAndroid)
 }
 

plugin/src/test/test-fixtures/multi-module/library_android/src/main/java/org/neotech/library/android/LibraryAndroidJava.java

@@ -12,7 +12,9 @@ public static synchronized LibraryAndroidJava getInstance() {
     }
 
     private LibraryAndroidJava() {
-
+        // Check if the BuildConfig file is available, this is here to prevent regression on:
+        // https://github.com/NeoTech-Software/Android-Root-Coverage-Plugin/issues/54
+        System.out.println(org.neotech.library.a.BuildConfig.DEBUG);
     }
 
     public String touchedByJavaInstrumentedTestInAndroidLibrary() {