Repro project for SPR-11027

SPR-11027/pom.xml

@@ -0,0 +1,75 @@
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+	xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
+	<modelVersion>4.0.0</modelVersion>
+	<groupId>org.springframework.issues</groupId>
+	<artifactId>SPR-11027</artifactId>
+	<version>1.0-SNAPSHOT</version>
+	<packaging>jar</packaging>
+	<dependencies>
+		<dependency>
+			<groupId>org.springframework</groupId>
+			<artifactId>spring-context</artifactId>
+			<version>3.1.2.RELEASE</version>
+		</dependency>
+		<dependency>
+			<groupId>log4j</groupId>
+			<artifactId>log4j</artifactId>
+			<version>1.2.16</version>
+		</dependency>
+		<dependency>
+			<groupId>junit</groupId>
+			<artifactId>junit</artifactId>
+			<version>4.8</version>
+			<scope>test</scope>
+		</dependency>
+	</dependencies>
+	<repositories>
+		<!--
+		<repository>
+			<id>s2-release</id>
+			<url>http://repo.springsource.org/release</url>
+		</repository>
+		<repository>
+			<id>s2-milestone</id>
+			<url>http://repo.springsource.org/milestone</url>
+		</repository>
+		<repository>
+			<id>s2-staging</id>
+			<url>http://repo.springsource.org/libs-staging-local</url>
+		</repository>
+		-->
+		<repository>
+			<id>s2-snapshot</id>
+			<url>http://repo.springsource.org/snapshot</url>
+			<snapshots><enabled>true</enabled></snapshots>
+		</repository>
+	</repositories>
+	<properties>
+		<project.build.sourceEncoding>UTF8</project.build.sourceEncoding>
+	</properties>
+	<build>
+		<plugins>
+			<plugin>
+				<artifactId>maven-compiler-plugin</artifactId>
+				<version>2.3.2</version>
+				<configuration>
+					<source>1.6</source>
+					<target>1.6</target>
+				</configuration>
+			</plugin>
+			<plugin>
+				<artifactId>maven-surefire-plugin</artifactId>
+				<version>2.7.2</version>
+				<configuration>
+					<includes>
+						<include>**/*Tests.java</include>
+					</includes>
+					<excludes>
+						<exclude>**/*Abstract*.java</exclude>
+					</excludes>
+				</configuration>
+			</plugin>
+		</plugins>
+	</build>
+</project>
+

SPR-11027/src/main/java/org/springframework/issues/Bar.java

@@ -0,0 +1,9 @@
+package org.springframework.issues;
+
+/**
+ * @author piofin <piotr.findeisen@syncron.com>
+ * @since Oct 17, 2013
+ */
+public class Bar {
+
+}

SPR-11027/src/main/java/org/springframework/issues/Foo.java

@@ -0,0 +1,20 @@
+package org.springframework.issues;
+
+import java.util.ArrayList;
+import java.util.List;
+
+import org.springframework.beans.factory.annotation.Autowired;
+
+public class Foo {
+
+	private List<Bar> bars = new ArrayList<Bar>();
+
+	public List<Bar> getBars() {
+		return bars;
+	}
+
+	@Autowired
+	public void setBars(List<Bar> bars) {
+		this.bars.addAll(bars);
+	}
+}

SPR-11027/src/test/java/org/springframework/issues/ReproTests.java

@@ -0,0 +1,58 @@
+package org.springframework.issues;
+
+import static java.lang.Boolean.TRUE;
+import static org.junit.Assert.assertThat;
+
+import org.hamcrest.CoreMatchers;
+import org.junit.Test;
+import org.springframework.context.support.GenericXmlApplicationContext;
+
+/**
+ * Unit test that reproduces an issue reported against SPR JIRA. @Test methods within need not pass with the green bar!
+ * Rather they should fail in such a way that demonstrates the reported issue.
+ */
+public class ReproTests {
+
+	@Test
+	public void shouldAutowire() {
+		GenericXmlApplicationContext ctx = getContext();
+
+		Foo foo = (Foo) ctx.getBean("justFoo");
+		assertThat(foo.getBars().size(), CoreMatchers.equalTo(1));
+	}
+
+	@Test
+	public void shouldOverrideAutowiredWithExplicitXml() {
+		GenericXmlApplicationContext ctx = getContext();
+
+		Foo foo = (Foo) ctx.getBean("foo");
+		assertThat(foo.getBars().isEmpty(), CoreMatchers.equalTo(TRUE));
+	}
+
+	@Test
+	public void shouldOverrideAutowiredWithExplicitXmlEvenAfterAutowiredUsed() {
+		GenericXmlApplicationContext ctx = getContext();
+
+		ctx.getBean("justFoo");
+
+		Foo foo = (Foo) ctx.getBean("foo");
+		assertThat(foo.getBars().isEmpty(), CoreMatchers.equalTo(TRUE));
+	}
+
+	@Test
+	public void shouldAutowireEvenAfterOtherMoreExplicitBeanIsInstantiated() {
+		GenericXmlApplicationContext ctx = getContext();
+
+		ctx.getBean("foo");
+
+		Foo foo = (Foo) ctx.getBean("justFoo");
+		assertThat(foo.getBars().size(), CoreMatchers.equalTo(1));
+	}
+
+	private GenericXmlApplicationContext getContext() {
+		GenericXmlApplicationContext ctx = new GenericXmlApplicationContext();
+		ctx.load("classpath:org/springframework/issues/ReproTests-context.xml");
+		ctx.refresh();
+		return ctx;
+	}
+}

SPR-11027/src/test/resources/log4j.properties

@@ -0,0 +1,7 @@
+log4j.rootCategory=ERROR, stdout
+
+log4j.appender.stdout=org.apache.log4j.ConsoleAppender
+log4j.appender.stdout.layout=org.apache.log4j.PatternLayout
+log4j.appender.stdout.layout.ConversionPattern=%d %p [%c] - <%m>%n
+
+log4j.category.org.springframework=WARN

SPR-11027/src/test/resources/org/springframework/issues/ReproTests-context.xml

@@ -0,0 +1,22 @@
+<?xml version="1.0" encoding="ISO-8859-1"?>
+<beans xmlns="http://www.springframework.org/schema/beans" xmlns:context="http://www.springframework.org/schema/context"
+	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-3.1.xsd
+	 	http://www.springframework.org/schema/context http://www.springframework.org/schema/context/spring-context.xsd
+	">
+
+	<context:annotation-config />
+
+	<bean id="bar" class="org.springframework.issues.Bar" lazy-init="true" />
+
+	<bean id="justFoo" class="org.springframework.issues.Foo" lazy-init="true">
+	</bean>
+
+	<bean id="foo" class="org.springframework.issues.Foo" lazy-init="true">
+		<property name="bars">
+			<list />
+		</property>
+	</bean>
+
+</beans>