From 9495bf24fdb607859f887ab22bf1a49e7f08e35a Mon Sep 17 00:00:00 2001
From: dsyer <dsyer@2e8e966d-6415-0410-94b9-dc4e65435d03>
Date: Wed, 24 Oct 2007 17:35:15 +0000
Subject: [PATCH] IN PROGRESS - issue BATCH-159: JobExecutor should return a
 JobExecution (which itself contains the ExitStatus)
 http://opensource.atlassian.com/projects/spring/browse/BATCH-159

Merged JobLauncher implementations together and separated out thread interruption and JMX notification responsibilities to separate collaborators.
---
 .../src/main/resources/jobs/infiniteLoopJob.xml   | 15 ++++++++++++++-
 .../batch/sample/AbstractBatchLauncherTests.java  |  4 ++--
 2 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/samples/src/main/resources/jobs/infiniteLoopJob.xml b/samples/src/main/resources/jobs/infiniteLoopJob.xml
index f066e14ad..f08ce06d5 100644
--- a/samples/src/main/resources/jobs/infiniteLoopJob.xml
+++ b/samples/src/main/resources/jobs/infiniteLoopJob.xml
@@ -11,7 +11,20 @@
 
 	<bean class="org.springframework.batch.execution.configuration.JobConfigurationRegistryBeanPostProcessor">
 		<property name="jobConfigurationRegistry" ref="jobConfigurationRegistry"/>
-	</bean>
+	</bean>
+	
+	<!-- Override the default jobExecutorFacade to provide a listener that can interrupt the job -->
+	<bean id="jobExecutorFacade" class="org.springframework.batch.execution.facade.SimpleJobExecutorFacade">
+		<property name="jobRepository" ref="simpleJobRepository" />
+		<property name="jobConfigurationLocator" ref="jobConfigurationRegistry"/>
+		<property name="jobExecutor" ref="jobExecutor" />
+		<property name="jobExecutionListeners">
+			<list>
+				<bean class="org.springframework.batch.execution.bootstrap.support.ThreadInterruptJobExecutionListener"/>
+			</list>
+		</property>
+	</bean>
+	
 
 	<bean id="jobConfiguration" parent="simpleJob">
 		<property name="steps">
diff --git a/samples/src/test/java/org/springframework/batch/sample/AbstractBatchLauncherTests.java b/samples/src/test/java/org/springframework/batch/sample/AbstractBatchLauncherTests.java
index 1a3957545..2ed136e3f 100644
--- a/samples/src/test/java/org/springframework/batch/sample/AbstractBatchLauncherTests.java
+++ b/samples/src/test/java/org/springframework/batch/sample/AbstractBatchLauncherTests.java
@@ -32,8 +32,8 @@ public abstract class AbstractBatchLauncherTests extends AbstractDependencyInjec
 	protected ConfigurableApplicationContext createApplicationContext(
 			String[] locations) {
 		String[] allLocations = new String[locations.length+1];
-		System.arraycopy(locations, 0, allLocations, 0, locations.length);
-		allLocations[locations.length] = "simple-container-definition.xml";
+		System.arraycopy(locations, 0, allLocations, 1, locations.length);
+		allLocations[0] = "simple-container-definition.xml";
 		return super.createApplicationContext(allLocations);
 	}