Fix: Don't log "skipping module" message for Gradle (sub)projects without a build.gradle file (#50)

Fixes #45

Author: Rolf-Smit

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

@@ -15,6 +15,7 @@ import org.gradle.testing.jacoco.tasks.JacocoReport
 import org.neotech.plugin.rootcoverage.utilities.afterAndroidPluginApplied
 import org.neotech.plugin.rootcoverage.utilities.fileTree
 import org.neotech.plugin.rootcoverage.utilities.onVariant
+import java.io.File
 
 class RootCoveragePlugin : Plugin<Project> {
 
@@ -119,7 +120,11 @@ class RootCoveragePlugin : Plugin<Project> {
     private fun JacocoReport.addSubProject(subProject: Project) {
         subProject.afterAndroidPluginApplied(
             notFoundAction = {
-                subProject.logger.info("Note: Skipping code coverage for module '${subProject.name}', reason: not an Android module.")
+                // Only log if a build.gradle file was found in the project directory, if not it could just be an empty project that holds
+                // child-projects
+                if (File(subProject.projectDir, Project.DEFAULT_BUILD_FILE).exists()) {
+                    subProject.logger.info("Note: Skipping code coverage for module '${subProject.name}', reason: not an Android module.")
+                }
             },
             action = {
                 subProject.applyJacocoPluginIfRequired()

plugin/src/test/test-fixtures/multi-module/library_nested/java/.gitignore

@@ -0,0 +1 @@
+/build

plugin/src/test/test-fixtures/multi-module/library_nested/java/build.gradle

@@ -0,0 +1,10 @@
+plugins {
+    id "java-library"
+}
+
+dependencies {
+    implementation fileTree(dir: 'libs', include: ['*.jar'])
+}
+
+sourceCompatibility = JavaVersion.VERSION_1_8
+targetCompatibility = JavaVersion.VERSION_1_8

plugin/src/test/test-fixtures/multi-module/library_nested/java/src/main/java/org/neotech/library/nested/java/Stub.java

@@ -0,0 +1,4 @@
+package org.neotech.library.nested.java;
+
+public class Stub {
+}

plugin/src/test/test-fixtures/multi-module/settings.gradle

@@ -19,4 +19,4 @@ dependencyResolutionManagement {
     }
 }
 
-include ':app', ':library_android', ':library_java'
+include ':app', ':library_android', ':library_java', ':library_nested:java'