Support Builder using name qualifier with List and Set (#944)

This supports injecting a List or Set of beans that have a
common name qualifier.

This seems like a rare use case, for example:
```
// A List of String that all have the same qualifier name of "args"
@nAmed("args") List<String>
```

Author: rbygrave

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

@@ -11,6 +11,12 @@
 @Factory
 public class ListFactory {
 
+  @Bean
+  @Named("args")
+  List<String> someNamedStrings() {
+    return List.of("arg0", "arg1");
+  }
+
   @Bean
   List<String> test(List<Cloneable> emptyList) {
     return List.of("test1", "test2");

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

@@ -1,20 +1,39 @@
 package org.example.myapp;
 
 import java.util.List;
+import java.util.Set;
 
+import jakarta.inject.Named;
 import jakarta.inject.Singleton;
 
 @Singleton
 public class ListService {
 
   private final List<String> strings;
+  private final List<String> argStrings;
+  private final Set<String> argsAsSet;
   private final List<ListFactory.Some> somes;
 
-  public ListService(List<String> strings, List<ListFactory.Some> somes) {
-    this.strings = strings;
+  public ListService(
+      List<String> allStrings,
+      @Named("args") List<String> argStrings,
+      @Named("args") Set<String> argsAsSet,
+      List<ListFactory.Some> somes) {
+
+    this.strings = allStrings;
+    this.argStrings = argStrings;
+    this.argsAsSet = argsAsSet;
     this.somes = somes;
   }
 
+  public List<String> args() {
+    return argStrings;
+  }
+
+  public Set<String> argsAsSet() {
+    return argsAsSet;
+  }
+
   public List<String> strings() {
     return strings;
   }

blackbox-test-inject/src/test/java/org/example/myapp/ListFactoryTest.java

@@ -18,7 +18,9 @@ class ListFactoryTest {
   @Test
   void test() {
 
-    assertThat(list.strings()).contains("test1", "test1", "test3");
+    assertThat(list.strings()).contains("test1", "test1", "test3", "arg0", "arg1");
+    assertThat(list.args()).containsOnly("arg0", "arg1");
+    assertThat(list.argsAsSet()).containsOnly("arg0", "arg1");
 
     List<ListFactory.Some> somes = list.somes();
     assertThat(somes).hasSize(4);

inject/src/main/java/io/avaje/inject/BeanScope.java

@@ -199,7 +199,14 @@ default <T> T get(Type type) {
    */
   <T> List<T> list(Type type);
 
-  /** Return the list of beans that implement the class sorting by priority. */
+  /**
+   * Return the list of beans for the given type and name.
+   */
+  <T> List<T> list(Type type, @Nullable String name);
+
+  /**
+   * Return the list of beans that implement the class sorting by priority.
+   */
   default <T> List<T> listByPriority(Class<T> type) {
     return listByPriority((Type) type);
   }

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

@@ -237,6 +237,11 @@ default <T> void registerLazy(Provider<T> provider, Function<Provider<T>, T> pro
    */
   <T> List<T> list(Type type);
 
+  /**
+   * Get a list of dependencies for the type and name.
+   */
+  <T> List<T> list(Type type, String name);
+
   /**
    * Get a set of dependencies for the type.
    */
@@ -247,6 +252,11 @@ default <T> void registerLazy(Provider<T> provider, Function<Provider<T>, T> pro
    */
   <T> Set<T> set(Type type);
 
+  /**
+   * Get a set of dependencies for the type and name.
+   */
+  <T> Set<T> set(Type type, String name);
+
   /**
    * Return a map of dependencies for the type keyed by qualifier name.
    */

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

@@ -157,11 +157,11 @@ <T> Provider<T> provider(Type type, String name) {
   }
 
   /**
-   * Return all bean instances matching the given type.
+   * Return all bean instances matching the given type and name.
    */
-  List<Object> all(Type type) {
+  List<Object> all(Type type, @Nullable String name) {
     DContextEntry entry = beans.get(type.getTypeName());
-    return entry != null ? entry.all() : List.of();
+    return entry != null ? entry.all(name) : List.of();
   }
 
   /**

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

@@ -158,21 +158,26 @@ public <T> Map<String, T> map(Type type) {
 
   @Override
   public <T> List<T> list(Class<T> type) {
-    return listOf(type);
+    return listOf(type, null);
   }
 
   @Override
   public <T> List<T> list(Type type) {
-    return listOf(type);
+    return listOf(type, null);
+  }
+
+  @Override
+  public <T> List<T> list(Type type, @Nullable String name) {
+    return listOf(type, name);
   }
 
   @SuppressWarnings("unchecked")
-  private <T> List<T> listOf(Type type) {
-    List<T> values = (List<T>) beans.all(type);
+  private <T> List<T> listOf(Type type, @Nullable String name) {
+    List<T> values = (List<T>) beans.all(type, name);
     if (parent == null) {
       return values;
     }
-    return combine(values, parent.list(type));
+    return combine(values, parent.list(type, name));
   }
 
   static <T> List<T> combine(List<T> values, List<T> parentValues) {

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

@@ -8,6 +8,7 @@
 
 import io.avaje.inject.BeanEntry;
 import io.avaje.inject.BeanScope;
+import org.jspecify.annotations.Nullable;
 
 /** Proxy used when injecting the BeanScope. */
 final class DBeanScopeProxy implements BeanScope {
@@ -88,7 +89,6 @@ public List<Object> listByAnnotation(Class<? extends Annotation> annotation) {
 
   @Override
   public <T> List<T> list(Class<T> type) {
-
     if (delegate != null) {
       return delegate.list(type);
     } else {
@@ -105,6 +105,15 @@ public <T> List<T> list(Type type) {
     }
   }
 
+  @Override
+  public <T> List<T> list(Type type, @Nullable String name) {
+    if (delegate != null) {
+      return delegate.list(type, name);
+    } else {
+      return builder.list(type, name);
+    }
+  }
+
   @Override
   public <T> List<T> listByPriority(Type type) {
     if (delegate != null) {

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

@@ -120,31 +120,41 @@ private Type firstOf(Type[] types) {
 
   @Override
   public final <T> Set<T> set(Class<T> type) {
-    return new LinkedHashSet<>(listOf(type));
+    return new LinkedHashSet<>(listOf(type, null));
   }
 
   @Override
   public final <T> List<T> list(Class<T> type) {
-    return listOf(type);
+    return listOf(type, null);
   }
 
   @Override
   public final <T> Set<T> set(Type type) {
-    return new LinkedHashSet<>(listOf(type));
+    return set(type, null);
+  }
+
+  @Override
+  public final <T> Set<T> set(Type type, String name) {
+    return new LinkedHashSet<>(listOf(type, name));
   }
 
   @Override
   public final <T> List<T> list(Type type) {
-    return listOf(type);
+    return listOf(type, null);
+  }
+
+  @Override
+  public <T> List<T> list(Type type, String name) {
+    return listOf(type, name);
   }
 
   @SuppressWarnings({"unchecked"})
-  private <T> List<T> listOf(Type type) {
-    final List<T> values = (List<T>) beanMap.all(type);
+  private <T> List<T> listOf(Type type, @Nullable String name) {
+    final List<T> values = (List<T>) beanMap.all(type, name);
     if (parent == null) {
       return values;
     }
-    return combine(values, parent.list(type));
+    return combine(values, parent.list(type, name));
   }
 
   @Override

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

@@ -7,6 +7,7 @@
 
 import io.avaje.inject.BeanEntry;
 import jakarta.inject.Provider;
+import org.jspecify.annotations.Nullable;
 
 /**
  * Entry for a given key (bean class, interface class or annotation class).
@@ -55,12 +56,14 @@ Object get(String name, Class<? extends AvajeModule> currentModule) {
   }
 
   /**
-   * Return all the beans.
+   * Return all the beans matching the name qualifier (if passed).
    */
-  List<Object> all() {
+  List<Object> all(@Nullable String name) {
     List<Object> list = new ArrayList<>(entries.size());
     for (DContextEntryBean entry : entries) {
-      list.add(entry.bean());
+      if (entry.isNameMatch(name)) {
+        list.add(entry.bean());
+      }
     }
     return list;
   }