#176 - Module interface lacks the classes() method

Modify the Module.classes() method to be non-static interface method

Author: rbygrave

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

@@ -43,14 +43,6 @@ class SimpleModuleWriter {
       "   * field injection, method injection and lifecycle support.\n" +
       "   */";
 
-  private static final String CLASSES_COMMENT =
-    "  /**\n" +
-    "   * Return public classes of the beans that would be registered by this module.\n" +
-    "   * <p>\n" +
-    "   * This method allows code to use reflection to inspect the modules classes \n" +
-    "   * before the module is wired. This method is not required for DI wiring.\n" +
-    "   */";
-
   private final ProcessingContext context;
   private final String modulePackage;
   private final String shortName;
@@ -99,8 +91,8 @@ private void writeServicesFile(ScopeInfo.Type scopeType) {
 
   private void writeClassesMethod() {
     Set<String> allClasses = distinctPublicClasses();
-    writer.append(CLASSES_COMMENT).eol();
-    writer.append("  public static Class<?>[] classes() {").eol();
+    writer.append("  @Override").eol();
+    writer.append("  public Class<?>[] classes() {").eol();
     writer.append("    return new Class<?>[]{").eol();
     for (String rawType : allClasses) {
       writer.append("      %s.class,", rawType).eol();

inject-test/src/test/java/org/example/coffee/BeanScopeBuilderAddTest.java

@@ -40,6 +40,11 @@ public Class<?>[] provides() {
       return new Class[0];
     }
 
+    @Override
+    public Class<?>[] classes() {
+      return new Class[0];
+    }
+
     @Override
     public void build(Builder builder) {
       // do nothing

inject-test/src/test/java/org/example/custom3/ParentScopeTest.java

@@ -50,7 +50,8 @@ void parentScope() {
 
   @Test
   void module_classes() {
-    Class<?>[] classes = MyThreeModule.classes();
+    MyThreeModule module = new MyThreeModule();
+    Class<?>[] classes = module.classes();
     Set<Class<?>> asSet = new HashSet<>(Arrays.asList(classes));
 
     assertThat(asSet).contains(

inject/src/main/java/io/avaje/inject/spi/Module.java

@@ -15,6 +15,14 @@ public interface Module {
    */
   Class<?>[] provides();
 
+  /**
+   * Return public classes of the beans that would be registered by this module.
+   * <p>
+   * This method allows code to use reflection to inspect the modules classes
+   * before the module is wired. This method is not required for DI wiring.
+   */
+  Class<?>[] classes();
+
   /**
    * Build all the beans.
    */

inject/src/test/java/io/avaje/inject/BeanScopeBuilderTest.java

@@ -117,6 +117,11 @@ public Class<?>[] provides() {
       return provides;
     }
 
+    @Override
+    public Class<?>[] classes() {
+      return new Class[0];
+    }
+
     @Override
     public Class<?>[] requires() {
       return requires;