Check overridden method for provides annotation

Fixes #269

Author: evant

ast/kapt/src/main/kotlin/me/tatarka/kotlin/ast/ModelAst.kt

@@ -334,9 +334,7 @@ private class ModelAstClass(
             addAll(methods)
             for (superType in superTypes) {
                 for (superMethod in superType.allMethods) {
-                    if (none { it.overrides(superMethod) }) {
-                        add(superMethod)
-                    }
+                    add(superMethod)
                 }
             }
         }

ast/ksp/src/main/kotlin/me/tatarka/kotlin/ast/KSAst.kt

@@ -206,13 +206,9 @@ private class KSAstClass(override val resolver: Resolver, override val declarati
             for (type in declaration.inheritanceChain()) {
                 for (declaration in type.declarations) {
                     if (declaration is KSPropertyDeclaration) {
-                        if (declarations.none { resolver.overrides(it, declaration) }) {
-                            declarations.add(declaration)
-                        }
+                        declarations.add(declaration)
                     } else if (declaration is KSFunctionDeclaration && !declaration.isConstructor()) {
-                        if (declarations.none { resolver.overrides(it, declaration) }) {
-                            declarations.add(declaration)
-                        }
+                        declarations.add(declaration)
                     }
                 }
             }

integration-tests/common/src/test/kotlin/me/tatarka/inject/test/InheritanceTest.kt

@@ -5,7 +5,9 @@ import assertk.assertions.hasClass
 import assertk.assertions.isNotNull
 import assertk.assertions.isSameAs
 import me.tatarka.inject.annotations.Component
+import me.tatarka.inject.annotations.Inject
 import me.tatarka.inject.annotations.Provides
+import me.tatarka.inject.annotations.Scope
 import kotlin.test.Test
 
 interface ComponentInterface {
@@ -59,6 +61,19 @@ abstract class ProvidesScopedInterfaceComponent : ProvidesScopedComponentInterfa
     abstract val foo: IFoo
 }
 
+
+interface AbstractProvidesInterface {
+    @get:Provides val foo: IFoo
+}
+
+@Component
+abstract class AbstractProvidesImplComponent: AbstractProvidesInterface {
+    abstract val bar: BarImpl
+
+    override val foo: IFoo
+        get() = Foo()
+}
+
 class InheritanceTest {
     @Test
     fun generates_a_component_that_provides_a_dep_defined_in_an_implemented_interface() {

kotlin-inject-compiler/core/src/main/kotlin/me/tatarka/inject/compiler/InjectGenerator.kt

@@ -232,7 +232,7 @@ fun AstAnnotated.qualifier(options: Options): AstAnnotation? {
 }
 
 fun AstMember.isProvider(): Boolean =
-    !isProvides() && isAbstract && when (this) {
+    isAbstract && when (this) {
         is AstFunction -> parameters.isEmpty()
         is AstProperty -> true
     } && receiverParameterType == null && returnType.isNotUnit()

kotlin-inject-compiler/core/src/main/kotlin/me/tatarka/inject/compiler/TypeCollector.kt

@@ -267,9 +267,21 @@ class TypeCollector(private val provider: AstProvider, private val options: Opti
                 }
             }
 
-            for (method in astClass.allMethods) {
+            val allMethods = astClass.allMethods
+            // some methods may override others
+            val methods = mutableMapOf<AstMember, AstMember?>()
+            for (method in allMethods) {
+                val overrides = methods.keys.find { it.overrides(method) }
+                if (overrides != null) {
+                    methods[overrides] = method
+                } else {
+                    methods[method] = null
+                }
+            }
+            for (method in methods.keys) {
                 val abstract = method.isAbstract
-                if (method.isProvides()) {
+                val overriden = methods[method]
+                if (method.isProvides() || overriden?.isProvides() == true) {
                     if (method.visibility == AstVisibility.PRIVATE) {
                         provider.error("@Provides method must not be private", method)
                         continue
@@ -296,8 +308,7 @@ class TypeCollector(private val provider: AstProvider, private val options: Opti
                     } else {
                         providesMethods.add(method)
                     }
-                }
-                if (method.isProvider()) {
+                } else if (method.isProvider()) {
                     val scope = method.scopeType(options)
                     if (scope != null) {
                         provider.warn(