Make ComponentMustBeAbstractDetector aware of Anvil Component annotat…

…ions (#260)

## Description

<!--- Describe your changes in detail -->

## Related Issue

<!--- This project only accepts pull requests related to open issues -->

<!--- If suggesting a new feature or change, please discuss it in an
issue first -->

<!--- If fixing a bug, there should be an issue describing it with steps
to reproduce -->

<!--- Please link to the issue here: -->

## Checklist

<!--- Go over all the following points, and put an `x` in all the boxes
that apply. -->

<!--- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->

- [ ] I have made corresponding changes to the documentation
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] I have mentioned changes in [CHANGELOG.md](../CHANGELOG.md).
- [ ] I have read the [**CONTRIBUTING**](CONTRIBUTING.md) document.

## PR Stack

<!-- branch-stack -->

- `main`
  - \#260 :point\_left:

lint/anvil/src/test/java/dev/whosnickdoglio/anvil/detectors/Stubs.kt → lint/annotation-constants/src/testFixtures/java/dev/whosnickdoglio/stubs/Anvil.kt

@@ -2,7 +2,7 @@
  * Copyright (C) 2024 Nicholas Doglio
  * SPDX-License-Identifier: MIT
  */
-package dev.whosnickdoglio.anvil.detectors
+package dev.whosnickdoglio.stubs
 
 import com.android.tools.lint.checks.infrastructure.TestFile
 import com.android.tools.lint.checks.infrastructure.TestFiles

lint/anvil/src/test/java/dev/whosnickdoglio/anvil/detectors/ContributesBindingMustHaveSuperDetectorTest.kt

@@ -11,6 +11,7 @@ import com.google.testing.junit.testparameterinjector.TestParameterInjector
 import dev.whosnickdoglio.lint.annotations.anvil.CONTRIBUTES_BINDING
 import dev.whosnickdoglio.lint.annotations.anvil.CONTRIBUTES_MULTI_BINDING
 import dev.whosnickdoglio.lint.annotations.anvil.CONTRIBUTES_TO
+import dev.whosnickdoglio.stubs.anvilAnnotations
 import dev.whosnickdoglio.stubs.daggerAnnotations
 import dev.whosnickdoglio.stubs.javaxAnnotations
 import org.junit.Test

lint/anvil/src/test/java/dev/whosnickdoglio/anvil/detectors/MissingContributesBindingDetectorTest.kt

@@ -10,6 +10,7 @@ import com.google.testing.junit.testparameterinjector.TestParameter
 import com.google.testing.junit.testparameterinjector.TestParameterInjector
 import dev.whosnickdoglio.lint.annotations.anvil.CONTRIBUTES_BINDING
 import dev.whosnickdoglio.lint.annotations.anvil.CONTRIBUTES_MULTI_BINDING
+import dev.whosnickdoglio.stubs.anvilAnnotations
 import dev.whosnickdoglio.stubs.javaxAnnotations
 import org.junit.Test
 import org.junit.runner.RunWith

lint/anvil/src/test/java/dev/whosnickdoglio/anvil/detectors/MissingContributesToDetectorTest.kt

@@ -6,6 +6,7 @@ package dev.whosnickdoglio.anvil.detectors
 
 import com.android.tools.lint.checks.infrastructure.TestFiles
 import com.android.tools.lint.checks.infrastructure.TestLintTask
+import dev.whosnickdoglio.stubs.anvilAnnotations
 import dev.whosnickdoglio.stubs.daggerAnnotations
 import org.junit.Test
 

lint/anvil/src/test/java/dev/whosnickdoglio/anvil/detectors/NoAnvilInJavaDetectorTest.kt

@@ -15,6 +15,7 @@ import dev.whosnickdoglio.lint.annotations.anvil.CONTRIBUTES_SUBCOMPONENT_FACTOR
 import dev.whosnickdoglio.lint.annotations.anvil.CONTRIBUTES_TO
 import dev.whosnickdoglio.lint.annotations.anvil.MERGE_COMPONENT
 import dev.whosnickdoglio.lint.annotations.anvil.MERGE_SUBCOMPONENT
+import dev.whosnickdoglio.stubs.anvilAnnotations
 import org.junit.Test
 import org.junit.runner.RunWith
 

lint/dagger/src/main/java/dev/whosnickdoglio/dagger/detectors/ComponentMustBeAbstractDetector.kt

@@ -15,6 +15,9 @@ import com.android.tools.lint.detector.api.Scope
 import com.android.tools.lint.detector.api.Severity
 import com.android.tools.lint.detector.api.SourceCodeScanner
 import com.android.tools.lint.detector.api.TextFormat
+import dev.whosnickdoglio.lint.annotations.anvil.CONTRIBUTES_SUBCOMPONENT
+import dev.whosnickdoglio.lint.annotations.anvil.MERGE_COMPONENT
+import dev.whosnickdoglio.lint.annotations.anvil.MERGE_SUBCOMPONENT
 import dev.whosnickdoglio.lint.annotations.dagger.COMPONENT
 import dev.whosnickdoglio.lint.annotations.dagger.SUBCOMPONENT
 import org.jetbrains.uast.UAnnotation
@@ -34,7 +37,7 @@ internal class ComponentMustBeAbstractDetector :
     override fun createUastHandler(context: JavaContext): UElementHandler =
         object : UElementHandler() {
             override fun visitAnnotation(node: UAnnotation) {
-                if (node.qualifiedName == COMPONENT || node.qualifiedName == SUBCOMPONENT) {
+                if (node.qualifiedName in componentAnnotations) {
                     val component = node.uastParent as? UClass ?: return
 
                     if (!context.evaluator.isAbstract(component)) {
@@ -62,6 +65,16 @@ internal class ComponentMustBeAbstractDetector :
         private val implementation =
             Implementation(ComponentMustBeAbstractDetector::class.java, Scope.JAVA_FILE_SCOPE)
 
+        internal val componentAnnotations = setOf(
+            // Vanilla Dagger
+            COMPONENT,
+            SUBCOMPONENT,
+            // Anvil
+            MERGE_COMPONENT,
+            MERGE_SUBCOMPONENT,
+            CONTRIBUTES_SUBCOMPONENT,
+        )
+
         internal val ISSUE =
             Issue.create(
                 id = "ComponentMustBeAbstract",

lint/dagger/src/test/java/dev/whosnickdoglio/dagger/detectors/ComponentMustBeAbstractDetectorTest.kt

@@ -8,22 +8,28 @@ import com.android.tools.lint.checks.infrastructure.TestFiles
 import com.android.tools.lint.checks.infrastructure.TestLintTask
 import com.google.testing.junit.testparameterinjector.TestParameter
 import com.google.testing.junit.testparameterinjector.TestParameterInjector
-import dev.whosnickdoglio.lint.annotations.dagger.COMPONENT
-import dev.whosnickdoglio.lint.annotations.dagger.SUBCOMPONENT
+import com.google.testing.junit.testparameterinjector.TestParameterValuesProvider
+import dev.whosnickdoglio.stubs.anvilAnnotations
 import dev.whosnickdoglio.stubs.daggerAnnotations
 import org.junit.Test
 import org.junit.runner.RunWith
 
 @RunWith(TestParameterInjector::class)
 class ComponentMustBeAbstractDetectorTest {
-    @TestParameter(value = [COMPONENT, SUBCOMPONENT])
+    private class ComponentMustBeAbstractTestParameterValueProvider : TestParameterValuesProvider() {
+        override fun provideValues(context: Context?): List<*> =
+            ComponentMustBeAbstractDetector.componentAnnotations.toList()
+    }
+
+    @TestParameter(valuesProvider = ComponentMustBeAbstractTestParameterValueProvider::class)
     lateinit var componentAnnotation: String
 
     @Test
     fun `kotlin abstract class component does not show error`() {
         TestLintTask.lint()
             .files(
                 daggerAnnotations,
+                anvilAnnotations,
                 TestFiles.kotlin(
                     """
                     import $componentAnnotation
@@ -40,10 +46,11 @@ class ComponentMustBeAbstractDetectorTest {
     }
 
     @Test
-    fun `java kotlin abstract class component does not show error`() {
+    fun `java abstract class component does not show error`() {
         TestLintTask.lint()
             .files(
                 daggerAnnotations,
+                anvilAnnotations,
                 TestFiles.java(
                     """
                     import $componentAnnotation;
@@ -64,6 +71,7 @@ class ComponentMustBeAbstractDetectorTest {
         TestLintTask.lint()
             .files(
                 daggerAnnotations,
+                anvilAnnotations,
                 TestFiles.kotlin(
                     """
                     import $componentAnnotation
@@ -84,6 +92,7 @@ class ComponentMustBeAbstractDetectorTest {
         TestLintTask.lint()
             .files(
                 daggerAnnotations,
+                anvilAnnotations,
                 TestFiles.java(
                     """
                     import $componentAnnotation;
@@ -104,6 +113,7 @@ class ComponentMustBeAbstractDetectorTest {
         TestLintTask.lint()
             .files(
                 daggerAnnotations,
+                anvilAnnotations,
                 TestFiles.java(
                     """
                     import $componentAnnotation;
@@ -134,7 +144,8 @@ class ComponentMustBeAbstractDetectorTest {
                 @@ -4 +4
                 - class MyComponent {}
                 + interface MyComponent {}
-                """.trimIndent(),
+                """
+                    .trimIndent(),
             )
     }
 
@@ -143,6 +154,7 @@ class ComponentMustBeAbstractDetectorTest {
         TestLintTask.lint()
             .files(
                 daggerAnnotations,
+                anvilAnnotations,
                 TestFiles.kotlin(
                     """
                     import $componentAnnotation
@@ -173,7 +185,8 @@ class ComponentMustBeAbstractDetectorTest {
                 @@ -4 +4
                 - class MyComponent
                 + interface MyComponent
-                """.trimIndent(),
+                """
+                    .trimIndent(),
             )
     }
 }