Repackaging to remove remaining cycles and increase cohesion

org.springframework.config.java/src/test/java/test/basic/AbstractJavaConfigTests.java

@@ -0,0 +1,14 @@
+package test.basic;
+
+import org.junit.Before;
+
+public abstract class AbstractJavaConfigTests {
+
+	@Before
+	public void setUp() {
+		
+	}
+	
+	//protected 
+	
+}

org.springframework.config.java/src/test/java/test/basic/AutowiredConfigurationTests.java

@@ -0,0 +1,39 @@
+package test.basic;
+
+import static org.hamcrest.CoreMatchers.*;
+import static org.junit.Assert.*;
+
+import org.junit.Test;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.config.java.annotation.Configuration;
+import org.springframework.config.java.ext.Bean;
+import org.springframework.context.support.ClassPathXmlApplicationContext;
+
+import test.beans.Colour;
+import test.beans.TestBean;
+
+public class AutowiredConfigurationTests {
+	public @Test void test() {
+		ClassPathXmlApplicationContext factory = new ClassPathXmlApplicationContext(
+				AutowiredConfigurationTests.class.getSimpleName() + ".xml",
+				AutowiredConfigurationTests.class);
+		
+		assertThat(factory.getBean("colour", Colour.class), equalTo(Colour.RED));
+		assertThat(factory.getBean("testBean", TestBean.class).getName(), equalTo(Colour.RED.toString()));
+	}
+	
+	@Configuration
+	static class AutowiredConfig {
+		private @Autowired Colour colour;
+		
+		public @Bean TestBean testBean() {
+			return new TestBean(colour.toString());
+		}
+	}
+	
+	@Configuration
+	static class ColorConfig {
+		public @Bean Colour colour() { return Colour.RED; }
+	}
+	
+}

org.springframework.config.java/src/test/java/test/basic/AutowiredConfigurationTests.xml

@@ -0,0 +1,14 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<beans xmlns="http://www.springframework.org/schema/beans"
+	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+	xsi:schemaLocation="http://www.springframework.org/schema/beans http://www.springframework.org/schema/beans/spring-beans.xsd
+		http://www.springframework.org/schema/context http://www.springframework.org/schema/context/spring-context.xsd"
+	xmlns:context="http://www.springframework.org/schema/context">
+
+	<context:annotation-config/>
+	
+	<bean class="org.springframework.config.java.process.ConfigurationPostProcessor"/>
+	
+	<bean class="test.basic.AutowiredConfigurationTests$AutowiredConfig"/>
+	<bean class="test.basic.AutowiredConfigurationTests$ColorConfig"/>
+</beans>

org.springframework.config.java/src/test/java/test/basic/BasicTests.java

@@ -5,13 +5,12 @@ import static org.junit.Assert.*;
 import static org.springframework.beans.factory.support.BeanDefinitionBuilder.*;
 
 import org.junit.Test;
-import org.springframework.beans.BeansException;
+import org.springframework.beans.factory.BeanFactory;
+import org.springframework.beans.factory.annotation.AutowiredAnnotationBeanPostProcessor;
 import org.springframework.beans.factory.config.BeanDefinition;
-import org.springframework.beans.factory.config.BeanFactoryPostProcessor;
-import org.springframework.beans.factory.config.ConfigurableListableBeanFactory;
 import org.springframework.beans.factory.support.DefaultListableBeanFactory;
-import org.springframework.config.java.annotation.Bean;
 import org.springframework.config.java.annotation.Configuration;
+import org.springframework.config.java.ext.Bean;
 import org.springframework.config.java.process.ConfigurationPostProcessor;
 import org.springframework.config.java.util.DefaultScopes;
 
@@ -19,46 +18,54 @@ import test.beans.ITestBean;
 import test.beans.TestBean;
 
 public class BasicTests {
-	@Test
-	public void test() {
-		DefaultListableBeanFactory factory = new DefaultListableBeanFactory();
-		factory.registerBeanDefinition("config", rootBeanDefinition(Config.class).getBeanDefinition());
-		
-		Config config = factory.getBean("config", Config.class);
-		assertThat(config, notNullValue());
-	}
 	
-	@Test
-	public void test2() {
+	/**
+	 * Creates a new {@link BeanFactory}, populates it with a {@link BeanDefinition} for
+	 * each of the given {@link Configuration} <var>configClasses</var>, and then post-processes
+	 * the factory using JavaConfig's {@link ConfigurationPostProcessor}.  When complete,
+	 * the factory is ready to service requests for any {@link Bean} methods declared by
+	 * <var>configClasses</var>.
+	 * 
+	 * @param configClasses the {@link Configuration} classes under test. may be an empty list.
+	 * 
+	 * @return fully initialized and post-processed {@link BeanFactory}
+	 */
+	private static BeanFactory initBeanFactory(Class<?>... configClasses) {
 		DefaultListableBeanFactory factory = new DefaultListableBeanFactory();
-		factory.registerBeanDefinition("config", rootBeanDefinition(Config.class).getBeanDefinition());
 		
-		BeanFactoryPostProcessor bfpp = new MyPostProcessor();
-		
-		bfpp.postProcessBeanFactory(factory);
-		
-		OtherConfig config = factory.getBean("config", OtherConfig.class);
-		assertThat(config, notNullValue());
-	}
-	
-	@Test
-	public void test3() {
-		DefaultListableBeanFactory factory = new DefaultListableBeanFactory();
-		factory.registerBeanDefinition("config", rootBeanDefinition(Config.class).getBeanDefinition());
+		for(Class<?> configClass : configClasses) {
+			String configBeanName = configClass.getName();
+			factory.registerBeanDefinition(configBeanName, rootBeanDefinition(configClass).getBeanDefinition());
+		}
 		
 		new ConfigurationPostProcessor().postProcessBeanFactory(factory);
 		
+		factory.addBeanPostProcessor(new AutowiredAnnotationBeanPostProcessor());
+		
+		return factory;
+	}
+	
+	
+	@Test
+	public void simplestPossibleConfiguration() {
+		BeanFactory factory = initBeanFactory(SimplestPossibleConfig.class);
+		
 		String stringBean = factory.getBean("stringBean", String.class);
 		
 		assertThat(stringBean, equalTo("foo"));
 	}
 	
+	@Configuration
+	static class SimplestPossibleConfig {
+		public @Bean String stringBean() {
+			return "foo";
+		}
+	}
+	
+	
 	@Test
-	public void test4() {
-		DefaultListableBeanFactory factory = new DefaultListableBeanFactory();
-		factory.registerBeanDefinition("config", rootBeanDefinition(Config2.class).getBeanDefinition());
-		
-		new ConfigurationPostProcessor().postProcessBeanFactory(factory);
+	public void configurationWithPrototypeScopedBeans() {
+		BeanFactory factory = initBeanFactory(ConfigWithPrototypeBean.class);
 		
 		TestBean foo = factory.getBean("foo", TestBean.class);
 		ITestBean bar = factory.getBean("bar", ITestBean.class);
@@ -67,44 +74,25 @@ public class BasicTests {
 		assertThat(foo.getSpouse(), sameInstance(bar));
 		assertThat(bar.getSpouse(), not(sameInstance(baz)));
 	}
-}
-
-class MyPostProcessor implements BeanFactoryPostProcessor {
-
-	public void postProcessBeanFactory(
-			ConfigurableListableBeanFactory beanFactory) throws BeansException {
-		BeanDefinition beanDefinition = beanFactory.getBeanDefinition("config");
-		beanDefinition.setBeanClassName(OtherConfig.class.getName());
-	}
 	
-}
-
-@Configuration
-class Config {
-	public @Bean String stringBean() {
-		return "foo";
+	@Configuration
+	static class ConfigWithPrototypeBean {
+		public @Bean TestBean foo() {
+			TestBean foo = new TestBean("foo");
+			foo.setSpouse(bar());
+			return foo;
+		}
+		
+		public @Bean TestBean bar() {
+			TestBean bar = new TestBean("bar");
+			bar.setSpouse(baz());
+			return bar;
+		}
+		
+		@Bean(scope=DefaultScopes.PROTOTYPE) 
+		public TestBean baz() {
+			return new TestBean("bar");
+		}
 	}
 }
 
-@Configuration
-class Config2 {
-	public @Bean TestBean foo() {
-		TestBean foo = new TestBean("foo");
-		foo.setSpouse(bar());
-		return foo;
-	}
-	
-	public @Bean TestBean bar() {
-		TestBean bar = new TestBean("bar");
-		bar.setSpouse(baz());
-		return bar;
-	}
-	
-	@Bean(scope=DefaultScopes.PROTOTYPE) 
-	public TestBean baz() {
-		return new TestBean("bar");
-	}
-}
-
-class OtherConfig {
-}