#167 - No dependency provided for _ on _ - The underlying issue, provides meta data not including non-interface inherited types

Author: rbygrave

inject-generator/src/main/java/io/avaje/inject/generator/BeanReader.java

@@ -92,8 +92,8 @@ List<MethodReader> getFactoryMethods() {
     return factoryMethods;
   }
 
-  List<String> getInterfaces() {
-    return typeReader.getInterfaces();
+  List<String> getProvides() {
+    return typeReader.getProvides();
   }
 
   Set<GenericType> getGenericTypes() {

inject-generator/src/main/java/io/avaje/inject/generator/MetaData.java

@@ -100,7 +100,7 @@ private List<String> asList(String[] content) {
   }
 
   void update(BeanReader beanReader) {
-    this.provides = beanReader.getInterfaces();
+    this.provides = beanReader.getProvides();
     this.dependsOn = beanReader.getDependsOn();
   }
 

inject-generator/src/main/java/io/avaje/inject/generator/TypeExtendsReader.java

@@ -72,6 +72,14 @@ List<String> getInterfaceTypes() {
     return interfaceTypes;
   }
 
+  List<String> getProvides() {
+    List<String> all = new ArrayList<>(extendsTypes.size() + interfaceTypes.size());
+    all.addAll(extendsTypes);
+    all.addAll(interfaceTypes);
+    all.remove(baseTypeRaw);
+    return all;
+  }
+
   boolean isCloseable() {
     return closeable;
   }

inject-generator/src/main/java/io/avaje/inject/generator/TypeReader.java

@@ -38,8 +38,8 @@ String getTypesRegister() {
     return typesRegister;
   }
 
-  List<String> getInterfaces() {
-    return extendsReader.getInterfaceTypes();
+  List<String> getProvides() {
+    return extendsReader.getProvides();
   }
 
   boolean isClosable() {

inject-test/src/test/java/org/example/inheritprovides/Application.java

@@ -0,0 +1,21 @@
+package org.example.inheritprovides;
+
+import jakarta.inject.Inject;
+import jakarta.inject.Singleton;
+import java.util.List;
+
+@Singleton
+public class Application {
+
+    private final List<Controller> controllers;
+
+    @Inject
+    public Application(List<Controller> controllers) {
+        this.controllers = controllers;
+    }
+
+    public List<Controller> getControllers() {
+        return controllers;
+    }
+
+}

inject-test/src/test/java/org/example/inheritprovides/ApplicationTest.java

@@ -0,0 +1,22 @@
+package org.example.inheritprovides;
+
+import io.avaje.inject.BeanScope;
+import org.junit.jupiter.api.Test;
+
+import java.util.List;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+class ApplicationTest {
+
+  @Test
+  void asd() {
+    try (BeanScope beanScope =  BeanScope.newBuilder().build()) {
+      Application application = beanScope.get(Application.class);
+      List<? extends Controller> controllers = application.getControllers();
+
+      assertThat(application).isNotNull();
+      assertThat(controllers).hasSize(2);
+    }
+  }
+}

inject-test/src/test/java/org/example/inheritprovides/Controller.java

@@ -0,0 +1,12 @@
+package org.example.inheritprovides;
+
+import java.util.HashMap;
+import java.util.Map;
+
+public class Controller {
+
+    Map<String, String> getContext() {
+        return new HashMap<>();
+    }
+
+}

inject-test/src/test/java/org/example/inheritprovides/Controller1.java

@@ -0,0 +1,16 @@
+package org.example.inheritprovides;
+
+import jakarta.inject.Singleton;
+import java.util.Map;
+
+@Singleton
+public class Controller1 extends Controller {
+
+  @Override
+  Map<String, String> getContext() {
+    Map<String, String> context = super.getContext();
+    context.put("something", "1");
+    return context;
+  }
+
+}

inject-test/src/test/java/org/example/inheritprovides/Controller2.java

@@ -0,0 +1,17 @@
+package org.example.inheritprovides;
+
+import jakarta.inject.Singleton;
+
+import java.util.Map;
+
+@Singleton
+public class Controller2 extends Controller {
+
+  @Override
+  Map<String, String> getContext() {
+    Map<String, String> context = super.getContext();
+    context.put("something", "2");
+    return context;
+  }
+
+}