#256 - Support the case of @factory @bean that needs to use fully qualified class name (short name clashes)

Author: rbygrave

blackbox-test-inject/src/main/java/org/example/myapp/config/DupShortNameFactory.java

@@ -0,0 +1,25 @@
+package org.example.myapp.config;
+
+import io.avaje.inject.Bean;
+import io.avaje.inject.Factory;
+
+@Factory
+public class DupShortNameFactory {
+
+  @Bean
+  MyDup one() {
+    return new MyDup();
+  }
+
+  /**
+   * Requires fully qualified name as short name of MyDup clashes.
+   */
+  @Bean
+  org.example.myapp.config.MyDup two() {
+    return new org.example.myapp.config.MyDup();
+  }
+
+  public static class MyDup {
+
+  }
+}

blackbox-test-inject/src/main/java/org/example/myapp/config/MyDup.java

@@ -0,0 +1,4 @@
+package org.example.myapp.config;
+
+public class MyDup {
+}

blackbox-test-inject/src/test/java/org/example/myapp/config/DupShortNameFactoryTest.java

@@ -0,0 +1,23 @@
+package org.example.myapp.config;
+
+import io.avaje.inject.BeanScope;
+import org.junit.jupiter.api.Test;
+
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.junit.jupiter.api.Assertions.*;
+
+class DupShortNameFactoryTest {
+
+  @Test
+  void factoryBeanRequiringFullyQualifiedName() {
+    try (BeanScope testScope = BeanScope.builder().build()) {
+      // both have short name of MyDup
+      var one = testScope.get(org.example.myapp.config.MyDup.class);
+      var two = testScope.get(DupShortNameFactory.MyDup.class);
+
+      assertNotNull(one);
+      assertNotNull(two);
+      assertThat(one).isNotSameAs(two);
+    }
+  }
+}