Make it possible to inject context to beans

Author: ogesaku

src/integrationTest/groovy/annotated/InjectMultipleDependenciesSpec.groovy

@@ -47,7 +47,8 @@ class InjectMultipleDependenciesSpec extends Specification {
         then:
             ContextException e = thrown(ContextException)
             e.message == "Could not create bean: Baz"
-            e.cause.message == "Beans not found for type: Foo"
+            e.cause.message == "Could not create bean from constructor: public annotated.samples.optional_multiple_deps.Baz(java.util.List)"
+            e.cause.cause.message == "Beans not found for type: Foo"
     }
 
     def "should fail injecting named list of beans"() {
@@ -59,6 +60,7 @@ class InjectMultipleDependenciesSpec extends Specification {
         then:
             ContextException e = thrown(ContextException)
             e.message == "Could not create bean: Bar"
-            e.cause.message.startsWith("Detected named @Dependency for a list of dependencies")
+            e.cause.message == "Could not create bean from constructor: public annotated.samples.named_multiple_deps.Bar(java.util.List)"
+            e.cause.cause.message.startsWith("Detected named @Dependency for a list of dependencies")
     }
 }

src/main/java/com/coditory/quark/context/BeanFinalizer.java

@@ -18,6 +18,9 @@ private BeanFinalizer() {
     }
 
     static void closeBean(Object bean, BeanDescriptor<?> descriptor, ResolutionContext context) {
+        if (bean instanceof Context) {
+            return;
+        }
         if (bean instanceof Closeable) {
             closeBean((Closeable) bean, descriptor);
         }

src/main/java/com/coditory/quark/context/BeanHolder.java

@@ -11,7 +11,6 @@
 import static com.coditory.quark.context.BeanFinalizer.closeBean;
 import static com.coditory.quark.context.BeanInitializer.initializeBean;
 import static com.coditory.quark.context.BeanPostInitializer.postInitializeBean;
-import static com.coditory.quark.context.ResolutionPath.emptyResolutionPath;
 import static java.util.Objects.requireNonNull;
 import static java.util.stream.Collectors.toSet;
 
@@ -145,3 +144,4 @@ public String toString() {
         return "BeanHolder{" + descriptor.toShortString() + '}';
     }
 }
+

src/main/java/com/coditory/quark/context/ConstructorBasedBeanCreator.java

@@ -24,7 +24,7 @@ static <T> ConstructorBasedBeanCreator<T> fromConstructor(Class<T> type) {
                     .map(c -> (Constructor<T>) c)
                     .toList();
             if (annotated.size() == 1) {
-                constructor = annotated.get(0);
+                constructor = annotated.getFirst();
             } else if (annotated.size() > 1) {
                 throw new ContextException("Expected single constructor annotated with @Inject in class: "
                         + type.getCanonicalName() + ". Got: " + annotated.size());
@@ -48,11 +48,11 @@ static <T> ConstructorBasedBeanCreator<T> fromConstructor(Class<T> type) {
     @NotNull
     @Override
     public T create(@NotNull ResolutionContext context) {
-        Object[] args = resolveArguments(constructor, context);
         try {
+            Object[] args = resolveArguments(constructor, context);
             return constructor.newInstance(args);
         } catch (Exception e) {
-            throw new ContextException("Could no create bean from constructor: " + constructor, e);
+            throw new ContextException("Could not create bean from constructor: " + constructor, e);
         }
     }
 

src/main/java/com/coditory/quark/context/Context.java

@@ -98,7 +98,7 @@ private Context(String name, Map<BeanDescriptor<?>, List<BeanHolder<?>>> beanHol
         // register self
         BeanDescriptor<Context> descriptor = descriptor(Context.class);
         BeanHolder<Context> holder = holder(descriptor, (ResolutionContext r) -> this, true);
-        holder.setEventEmitter(eventBus);
+        holder.setEventEmitter(new DummyEventEmitter());
         beanHolders.put(descriptor, List.of(holder));
         // index beans
         this.beanHolders = beanHolders;

src/main/java/com/coditory/quark/context/DummyEventEmitter.java

@@ -0,0 +1,11 @@
+package com.coditory.quark.context;
+
+import com.coditory.quark.eventbus.EventEmitter;
+import org.jetbrains.annotations.NotNull;
+
+class DummyEventEmitter implements EventEmitter {
+    @Override
+    public void emit(@NotNull Object event) {
+        // deliberately empty
+    }
+}

src/main/java/com/coditory/quark/context/MethodBasedBeanCreator.java

@@ -29,9 +29,9 @@ public MethodBasedBeanCreator(BeanHolder<?> holder, Method method) {
     @NotNull
     @Override
     public T create(@NotNull ResolutionContext context) {
-        Object object = holder.get(context);
-        Object[] args = resolveArguments(method, context);
         try {
+            Object object = holder.get(context);
+            Object[] args = resolveArguments(method, context);
             return (T) method.invoke(object, args);
         } catch (Exception e) {
             throw new ContextException("Could not create bean from method: " + simplifyMethodName(method), e);

src/test/groovy/com/coditory/quark/context/BeanDependenciesSpec.groovy renamed to src/test/groovy/com/coditory/quark/context/ContextDependencySpec.groovy

@@ -2,7 +2,7 @@ package com.coditory.quark.context
 
 import spock.lang.Specification
 
-class BeanDependenciesSpec extends Specification {
+class ContextDependencySpec extends Specification {
     def "should register bean with dependency on the dependency context"() {
         given:
             Context context = Context.builder()