fix aggregate report collecting invalid configurations

Author: wakingrufus

nebula-archrules-gradle-plugin/src/main/kotlin/com/netflix/nebula/archrules/gradle/ArchrulesAggregateConsoleReportPlugin.kt

@@ -3,6 +3,7 @@ package com.netflix.nebula.archrules.gradle
 import com.tngtech.archunit.lang.Priority
 import org.gradle.api.Plugin
 import org.gradle.api.Project
+import org.gradle.api.artifacts.result.ResolvedArtifactResult
 import org.gradle.api.attributes.Category
 import org.gradle.api.attributes.VerificationType
 import org.gradle.kotlin.dsl.named
@@ -11,7 +12,9 @@ import org.gradle.kotlin.dsl.register
 
 class ArchrulesAggregateConsoleReportPlugin : Plugin<Project> {
     override fun apply(project: Project) {
-        val ext  = project.extensions.create("archRulesAggregate", ArchrulesAggregateExtension::class.java)
+        val verification: Category = project.objects.named(Category.VERIFICATION)
+        val archRulesVerificationType: VerificationType = project.objects.named("arch-rules")
+        val ext = project.extensions.create("archRulesAggregate", ArchrulesAggregateExtension::class.java)
         ext.skipPassingSummaries.set(false)
         ext.consoleDetailsThreshold(Priority.MEDIUM)
         val archRulesAggregateDependencies = project.configurations.dependencyScope("archRulesAggregateDependencies") {
@@ -20,8 +23,8 @@ class ArchrulesAggregateConsoleReportPlugin : Plugin<Project> {
         val archRulesDataFiles = project.configurations.resolvable("archRulesDataFiles") {
             extendsFrom(archRulesAggregateDependencies.get())
             attributes {
-                attribute(Category.CATEGORY_ATTRIBUTE, project.objects.named(Category.VERIFICATION))
-                attribute(VerificationType.VERIFICATION_TYPE_ATTRIBUTE, project.objects.named("arch-rules"))
+                attribute(Category.CATEGORY_ATTRIBUTE, verification)
+                attribute(VerificationType.VERIFICATION_TYPE_ATTRIBUTE, archRulesVerificationType)
             }
         }
         project.subprojects {
@@ -30,11 +33,19 @@ class ArchrulesAggregateConsoleReportPlugin : Plugin<Project> {
         project.tasks.register<PrintConsoleReportTask>("archRulesAggregateConsoleReport") {
             dataFiles.from(archRulesDataFiles.map {
                 it.incoming.artifactView {
-                   lenient(true) // to handle the case where a subproject doesn't have archrules runner
-                }.files
+                    lenient(true) // to handle the case where a subproject doesn't have archrules runner
+                }.artifacts.filter {
+                    it.hasCategory(verification) && it.hasVerificationType(archRulesVerificationType)
+                }.map { it.file }
             })
             summaryForPassingDisabled.set(ext.skipPassingSummaries)
             detailsThreshold.set(ext.consoleDetailsThreshold)
         }
     }
+
+    fun ResolvedArtifactResult.hasCategory(category: Category): Boolean =
+        variant.attributes.getAttribute(Category.CATEGORY_ATTRIBUTE) == category
+
+    fun ResolvedArtifactResult.hasVerificationType(type: VerificationType): Boolean =
+        variant.attributes.getAttribute(VerificationType.VERIFICATION_TYPE_ATTRIBUTE) == type
 }

nebula-archrules-gradle-plugin/src/test/kotlin/com/netflix/nebula/archrules/gradle/ArchrulesAggregateConsoleReportPluginTest.kt

@@ -14,6 +14,7 @@ import nebula.test.dsl.subProject
 import nebula.test.dsl.testProject
 import org.gradle.kotlin.dsl.findByType
 import org.gradle.testfixtures.ProjectBuilder
+import org.gradle.testkit.runner.TaskOutcome
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.io.TempDir
 import java.io.File
@@ -22,9 +23,11 @@ class ArchrulesAggregateConsoleReportPluginTest {
     @TempDir
     lateinit var projectDir: File
 
-    private fun TestProjectBuilder.setup(withFailures : Boolean = true) {
+    private fun TestProjectBuilder.setup(withFailures: Boolean = true) {
         properties {
             buildCache(true)
+            configurationCache(true)
+            property("org.gradle.unsafe.isolated-projects","true")
         }
         rootProject {
             plugins {
@@ -55,7 +58,7 @@ class ArchrulesAggregateConsoleReportPluginTest {
             )
             src {
                 main {
-                    if(withFailures) {
+                    if (withFailures) {
                         exampleDeprecatedUsage("FailingCode1")
                     }
                 }
@@ -66,7 +69,7 @@ class ArchrulesAggregateConsoleReportPluginTest {
                 id("java")
                 id("com.netflix.nebula.archrules.runner")
             }
-            repositories{
+            repositories {
                 mavenCentral()
             }
             dependencies(
@@ -75,7 +78,7 @@ class ArchrulesAggregateConsoleReportPluginTest {
             )
             src {
                 main {
-                    if(withFailures) {
+                    if (withFailures) {
                         exampleDeprecatedUsage("FailingCode2")
                     }
                 }
@@ -96,7 +99,7 @@ class ArchrulesAggregateConsoleReportPluginTest {
     @Test
     fun test() {
         val runner = testProject(projectDir) {
-         setup()
+            setup()
         }
         val result = runner.run("archRulesAggregateConsoleReport") {
             forwardOutput()
@@ -110,8 +113,9 @@ class ArchrulesAggregateConsoleReportPluginTest {
     fun `test passing skip`() {
         val runner = testProject(projectDir) {
             setup(withFailures = false)
-            rootProject{
-                rawBuildScript("""
+            rootProject {
+                rawBuildScript(
+                    """
 archRulesAggregate {
     skipPassingSummaries = true
 }
@@ -132,8 +136,9 @@ archRulesAggregate {
     fun `test details threshold`() {
         val runner = testProject(projectDir) {
             setup()
-            rootProject{
-                rawBuildScript("""
+            rootProject {
+                rawBuildScript(
+                    """
 archRulesAggregate {
     consoleDetailsThreshold("LOW")
 }
@@ -147,4 +152,29 @@ archRulesAggregate {
         assertThat(result.output)
             .contains("Method <com.example.consumer.FailingCode1.aMethod()> calls method <com.example.library.LibraryClass.deprecatedApi()>")
     }
+
+    @Test
+    fun `test empty subproject`() {
+        val runner = testProject(projectDir) {
+            setup()
+            subProject("empty") {
+                plugins {
+                    id("base")
+                }
+                rawBuildScript(
+                    // language=kotlin
+                    """
+val jarTask = tasks.register<Jar>("someJar")
+artifacts {
+    add("default", jarTask)
+}
+"""
+                )
+            }
+        }
+        val result = runner.run("archRulesAggregateConsoleReport")
+        assertThat(result.task(":archRulesAggregateConsoleReport"))
+            .hasOutcome(TaskOutcome.SUCCESS)
+        assertThat(result.output).doesNotContain("Archrules data read failed")
+    }
 }