Refactor rename from BeanContext to BeanScope

Author: rbygrave

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

@@ -20,7 +20,7 @@ class Constants {
   static final String META_INF_FACTORY = "META-INF/services/io.avaje.inject.spi.BeanContextFactory";
 
   static final String REQUESTSCOPE = "io.avaje.inject.RequestScope";
-  static final String BEANCONTEXT = "io.avaje.inject.BeanContext";
+  static final String BEANCONTEXT = "io.avaje.inject.BeanScope";
   static final String CONTEXTMODULE = "io.avaje.inject.ContextModule";
 
   static final String GENERATED = "io.avaje.inject.spi.Generated";

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

@@ -14,7 +14,7 @@ class SimpleFactoryWriter {
 
   private static final String CODE_COMMENT_FACTORY =
     "/**\n" +
-      " * Generated source - Creates the BeanContext for the %s module.\n" +
+      " * Generated source - Creates the BeanScope for the %s module.\n" +
       " * \n" +
       " * With JPMS Java module system this generated class should be explicitly\n" +
       " * registered in module-info via a <code>provides</code> clause like:\n" +
@@ -33,16 +33,12 @@ class SimpleFactoryWriter {
 
   private static final String CODE_COMMENT_CREATE_CONTEXT =
     "  /**\n" +
-      "   * Create and return the BeanContext.\n" +
+      "   * Create the beans.\n" +
       "   * <p>\n" +
       "   * Creates all the beans in order based on constuctor dependencies.\n" +
       "   * The beans are registered into the builder along with callbacks for\n" +
       "   * field injection, method injection and lifecycle support.\n" +
       "   * <p>\n" +
-      "   * Ultimately the builder returns the BeanContext containing the beans.\n" +
-      "   *\n" +
-      "   * @param parent The parent context for multi-module wiring\n" +
-      "   * @return The BeanContext containing the beans\n" +
       "   */";
 
   private final MetaDataOrdering ordering;

inject-test/src/test/java/io/avaje/inject/BeanContextBuilderTest.java renamed to inject-test/src/test/java/io/avaje/inject/BeanScopeBuilderTest.java

@@ -10,12 +10,12 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-public class BeanContextBuilderTest {
+public class BeanScopeBuilderTest {
 
   @Test
   public void noDepends() {
 
-    DBeanContextBuilder.FactoryOrder factoryOrder = new DBeanContextBuilder.FactoryOrder(Collections.emptySet(), true, true);
+    DBeanScopeBuilder.FactoryOrder factoryOrder = new DBeanScopeBuilder.FactoryOrder(Collections.emptySet(), true, true);
     factoryOrder.add(bc("1", null, null));
     factoryOrder.add(bc("2", null, null));
     factoryOrder.add(bc("3", null, null));
@@ -27,7 +27,7 @@ public void noDepends() {
   @Test
   public void name_depends() {
 
-    DBeanContextBuilder.FactoryOrder factoryOrder = new DBeanContextBuilder.FactoryOrder(Collections.emptySet(), true, true);
+    DBeanScopeBuilder.FactoryOrder factoryOrder = new DBeanScopeBuilder.FactoryOrder(Collections.emptySet(), true, true);
     factoryOrder.add(bc("two", null, "one"));
     factoryOrder.add(bc("one", null, null));
     factoryOrder.orderFactories();
@@ -38,7 +38,7 @@ public void name_depends() {
   @Test
   public void name_depends4() {
 
-    DBeanContextBuilder.FactoryOrder factoryOrder = new DBeanContextBuilder.FactoryOrder(Collections.emptySet(), true, true);
+    DBeanScopeBuilder.FactoryOrder factoryOrder = new DBeanScopeBuilder.FactoryOrder(Collections.emptySet(), true, true);
     factoryOrder.add(bc("1", null, "3"));
     factoryOrder.add(bc("2", null, "4"));
     factoryOrder.add(bc("3", null, "4"));
@@ -52,7 +52,7 @@ public void name_depends4() {
   @Test
   public void nameFeature_depends() {
 
-    DBeanContextBuilder.FactoryOrder factoryOrder = new DBeanContextBuilder.FactoryOrder(Collections.emptySet(), true, true);
+    DBeanScopeBuilder.FactoryOrder factoryOrder = new DBeanScopeBuilder.FactoryOrder(Collections.emptySet(), true, true);
     factoryOrder.add(bc("1", "a", "3"));
     factoryOrder.add(bc("2", null, "4,a"));
     factoryOrder.add(bc("3", null, "4"));
@@ -66,7 +66,7 @@ public void nameFeature_depends() {
   @Test
   public void feature_depends() {
 
-    DBeanContextBuilder.FactoryOrder factoryOrder = new DBeanContextBuilder.FactoryOrder(Collections.emptySet(), true, true);
+    DBeanScopeBuilder.FactoryOrder factoryOrder = new DBeanScopeBuilder.FactoryOrder(Collections.emptySet(), true, true);
     factoryOrder.add(bc("two", null, "myfeature"));
     factoryOrder.add(bc("one", "myfeature", null));
     factoryOrder.orderFactories();
@@ -77,7 +77,7 @@ public void feature_depends() {
   @Test
   public void feature_depends2() {
 
-    DBeanContextBuilder.FactoryOrder factoryOrder = new DBeanContextBuilder.FactoryOrder(Collections.emptySet(), true, true);
+    DBeanScopeBuilder.FactoryOrder factoryOrder = new DBeanScopeBuilder.FactoryOrder(Collections.emptySet(), true, true);
     factoryOrder.add(bc("two", null, "myfeature"));
     factoryOrder.add(bc("one", "myfeature", null));
     factoryOrder.add(bc("three", "myfeature", null));

inject-test/src/test/java/org/example/coffee/BeanContextBuilderAddTest.java renamed to inject-test/src/test/java/org/example/coffee/BeanScopeBuilderAddTest.java

@@ -1,21 +1,21 @@
 package org.example.coffee;
 
-import io.avaje.inject.BeanContext;
+import io.avaje.inject.BeanScope;
 import org.junit.jupiter.api.Test;
 import org.mockito.Mockito;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.mockito.Mockito.verify;
 
-public class BeanContextBuilderAddTest {
+public class BeanScopeBuilderAddTest {
 
   @Test
   public void withModules_exludingThisOne() {
     assertThrows(IllegalStateException.class, () -> {
       TDPump testDoublePump = new TDPump();
 
-      try (BeanContext context = BeanContext.newBuilder()
+      try (BeanScope context = BeanScope.newBuilder()
         .withBeans(testDoublePump)
         // our module is "org.example.coffee"
         // so this effectively includes no modules
@@ -35,7 +35,7 @@ public void withModules_includeThisOne() {
 
     TDPump testDoublePump = new TDPump();
 
-    try (BeanContext context = BeanContext.newBuilder()
+    try (BeanScope context = BeanScope.newBuilder()
       .withBeans(testDoublePump)
       .withModules("org.example")
       .build()) {
@@ -53,7 +53,7 @@ public void withBean_expect_testDoublePumpUsed() {
 
     TDPump testDoublePump = new TDPump();
 
-    try (BeanContext context = BeanContext.newBuilder()
+    try (BeanScope context = BeanScope.newBuilder()
       .withBeans(testDoublePump)
       .build()) {
 
@@ -70,7 +70,7 @@ public void withMockitoMock_expect_mockUsed() {
 
     Pump mock = Mockito.mock(Pump.class);
 
-    try (BeanContext context = BeanContext.newBuilder()
+    try (BeanScope context = BeanScope.newBuilder()
       .withBean(Pump.class, mock)
       .build()) {
 

inject-test/src/test/java/org/example/coffee/BeanContext_Builder_mockitoSpyTest.java renamed to inject-test/src/test/java/org/example/coffee/BeanScope_Builder_mockitoSpyTest.java

@@ -1,6 +1,6 @@
 package org.example.coffee;
 
-import io.avaje.inject.BeanContext;
+import io.avaje.inject.BeanScope;
 import org.example.coffee.factory.SomeImpl;
 import org.example.coffee.factory.SomeImplBean;
 import org.example.coffee.factory.Unused;
@@ -18,15 +18,15 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.mockito.Mockito.*;
 
-public class BeanContext_Builder_mockitoSpyTest {
+public class BeanScope_Builder_mockitoSpyTest {
 
   @Test
   public void withBeans_asMocks() {
 
     Pump pump = mock(Pump.class);
     Grinder grinder = mock(Grinder.class);
 
-    try (BeanContext context = BeanContext.newBuilder()
+    try (BeanScope context = BeanScope.newBuilder()
       .withBeans(pump, grinder)
       .build()) {
 
@@ -47,7 +47,7 @@ public void withBeans_asMocks() {
   @Test
   public void withMockitoSpy_noSetup_expect_spyUsed() {
 
-    try (BeanContext context = BeanContext.newBuilder()
+    try (BeanScope context = BeanScope.newBuilder()
       .withSpy(Pump.class)
       .build()) {
 
@@ -63,7 +63,7 @@ public void withMockitoSpy_noSetup_expect_spyUsed() {
   @Test
   public void withMockitoSpy_postLoadSetup_expect_spyUsed() {
 
-    try (BeanContext context = BeanContext.newBuilder()
+    try (BeanScope context = BeanScope.newBuilder()
       .withSpy(Pump.class)
       .withSpy(Grinder.class)
       .build()) {
@@ -86,7 +86,7 @@ public void withMockitoSpy_postLoadSetup_expect_spyUsed() {
   @Test
   public void withMockitoSpy_expect_spyUsed() {
 
-    try (BeanContext context = BeanContext.newBuilder()
+    try (BeanScope context = BeanScope.newBuilder()
       .withSpy(Pump.class, pump -> {
         // setup the spy
         doNothing().when(pump).pumpWater();
@@ -109,7 +109,7 @@ public void withMockitoSpy_expect_spyUsed() {
   @Test
   public void withMockitoSpy_whenPrimary_expect_spyUsed() {
 
-    try (BeanContext context = BeanContext.newBuilder()
+    try (BeanScope context = BeanScope.newBuilder()
       .withSpy(PEmailer.class) // has a primary
       .build()) {
 
@@ -124,7 +124,7 @@ public void withMockitoSpy_whenPrimary_expect_spyUsed() {
   @Test
   public void withMockitoSpy_whenOnlySecondary_expect_spyUsed() {
 
-    try (BeanContext context = BeanContext.newBuilder()
+    try (BeanScope context = BeanScope.newBuilder()
       .withSpy(Widget.class) // only secondary
       .build()) {
 
@@ -144,7 +144,7 @@ public void withMockitoSpy_whenOnlySecondary_expect_spyUsed() {
   @Test
   public void withMockitoSpy_whenSecondary_expect_spyUsed() {
 
-    try (BeanContext context = BeanContext.newBuilder()
+    try (BeanScope context = BeanScope.newBuilder()
       .withSpy(Something.class) // has a secondary and a normal
       .build()) {
 
@@ -168,7 +168,7 @@ public void withMockitoMock_expect_mockUsed() {
 
     AtomicReference<Grinder> mock = new AtomicReference<>();
 
-    try (BeanContext context = BeanContext.newBuilder()
+    try (BeanScope context = BeanScope.newBuilder()
       .withMock(Pump.class)
       .withMock(Grinder.class, grinder -> {
         // setup the mock

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

@@ -1,7 +1,7 @@
 package org.example.coffee;
 
 import io.avaje.inject.ApplicationScope;
-import io.avaje.inject.BeanContext;
+import io.avaje.inject.BeanScope;
 import io.avaje.inject.SystemContext;
 import org.example.coffee.core.DuperPump;
 import org.junit.jupiter.api.Test;
@@ -26,7 +26,7 @@ public void makeIt_via_SystemContext() {
   @Test
   public void makeIt_via_BootContext_withNoShutdownHook() {
 
-    try (BeanContext context = BeanContext.newBuilder()
+    try (BeanScope context = BeanScope.newBuilder()
       .withNoShutdownHook()
       .build()) {
 
@@ -38,7 +38,7 @@ public void makeIt_via_BootContext_withNoShutdownHook() {
   @Test
   public void makeIt_via_BootContext() {
 
-    try (BeanContext context = BeanContext.newBuilder().build()) {
+    try (BeanScope context = BeanScope.newBuilder().build()) {
       String makeIt = context.getBean(CoffeeMaker.class).makeIt();
       assertThat(makeIt).isEqualTo("done");
     }

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

@@ -1,7 +1,7 @@
 package org.example.coffee;
 
-import io.avaje.inject.BeanContext;
-import io.avaje.inject.BeanContextBuilder;
+import io.avaje.inject.BeanScope;
+import io.avaje.inject.BeanScopeBuilder;
 
 import java.util.List;
 
@@ -15,8 +15,8 @@ class ExtensionExample {
     this.withSpies = withSpies;
   }
 
-  BeanContext build() {
-    BeanContextBuilder bootContext = BeanContext.newBuilder();
+  BeanScope build() {
+    BeanScopeBuilder bootContext = BeanScope.newBuilder();
     withMocks.forEach(bootContext::withMock);
     withSpies.forEach(bootContext::withSpy);
     return bootContext.build();

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

@@ -1,7 +1,7 @@
 package org.example.coffee;
 
-import io.avaje.inject.BeanContext;
-import io.avaje.inject.BeanContextBuilder;
+import io.avaje.inject.BeanScope;
+import io.avaje.inject.BeanScopeBuilder;
 import org.example.coffee.qualifier.SomeStore;
 import org.example.coffee.secondary.SEmailer;
 import org.example.coffee.secondary.Widget;
@@ -16,12 +16,12 @@ public class ExtensionExampleTest {
   public void checkForCompilerWarningsOnly_notATestThatRuns() {
 
     ExtensionExample extensionExample = new ExtensionExample(asList(Widget.class, SEmailer.class), asList(SomeStore.class));
-    BeanContext context = extensionExample.build();
+    BeanScope context = extensionExample.build();
 
     Class cls0 = Widget.class;
     Class<?> cls1 = SEmailer.class;
 
-    BeanContextBuilder bootContext = BeanContext.newBuilder()
+    BeanScopeBuilder bootContext = BeanScope.newBuilder()
       .withSpy(cls0)
       .withSpy(cls1)
       .withMock(cls0)

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

@@ -1,6 +1,6 @@
 package org.example.coffee;
 
-import io.avaje.inject.BeanContext;
+import io.avaje.inject.BeanScope;
 import org.example.coffee.factory.BFact;
 import org.junit.jupiter.api.Test;
 
@@ -11,7 +11,7 @@ public class FactoryTest {
   @Test
   public void test() {
 
-    try (BeanContext context = BeanContext.newBuilder().build()) {
+    try (BeanScope context = BeanScope.newBuilder().build()) {
       BFact bean = context.getBean(BFact.class);
       String b = bean.b();
       assertThat(b).isNotNull();

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

@@ -1,6 +1,6 @@
 package org.example.coffee;
 
-import io.avaje.inject.BeanContext;
+import io.avaje.inject.BeanScope;
 import org.example.coffee.list.CombinedSetSomei;
 import org.example.coffee.list.CombinedSomei;
 import org.junit.jupiter.api.Test;
@@ -13,7 +13,7 @@ public class InjectListTest {
 
   @Test
   public void test() {
-    try (BeanContext context = BeanContext.newBuilder().build()) {
+    try (BeanScope context = BeanScope.newBuilder().build()) {
       CombinedSomei bean = context.getBean(CombinedSomei.class);
       List<String> somes = bean.lotsOfSomes();
       assertThat(somes).containsOnly("a", "b", "a2");
@@ -22,7 +22,7 @@ public void test() {
 
   @Test
   public void test_set() {
-    try (BeanContext context = BeanContext.newBuilder().build()) {
+    try (BeanScope context = BeanScope.newBuilder().build()) {
       CombinedSetSomei bean = context.getBean(CombinedSetSomei.class);
       List<String> somes = bean.lotsOfSomes();
       assertThat(somes).containsOnly("a", "b", "a2");

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

@@ -1,6 +1,6 @@
 package org.example.coffee;
 
-import io.avaje.inject.BeanContext;
+import io.avaje.inject.BeanScope;
 import org.example.coffee.provider.ProvOther;
 import org.example.coffee.provider.ProvOther2;
 import org.junit.jupiter.api.Test;
@@ -12,7 +12,7 @@ public class ProviderTest {
   @Test
   public void test() {
 
-    try (BeanContext context = BeanContext.newBuilder().build()) {
+    try (BeanScope context = BeanScope.newBuilder().build()) {
 
       ProvOther bean = context.getBean(ProvOther.class);
       String other = bean.other();

inject-test/src/test/java/org/example/coffee/circular/CircularDependencyTest.java

@@ -1,6 +1,6 @@
 package org.example.coffee.circular;
 
-import io.avaje.inject.BeanContext;
+import io.avaje.inject.BeanScope;
 import org.junit.jupiter.api.Test;
 
 import static org.assertj.core.api.Assertions.assertThat;
@@ -9,7 +9,7 @@ class CircularDependencyTest {
 
   @Test
   void wire() {
-    try (BeanContext context = BeanContext.newBuilder().build()) {
+    try (BeanScope context = BeanScope.newBuilder().build()) {
       assertThat(context.getBean(CircA.class)).isNotNull();
       assertThat(context.getBean(CircB.class)).isNotNull();
       assertThat(context.getBean(CircC.class)).isNotNull();