diff --git a/spring-batch-samples/src/main/java/org/springframework/batch/sample/tasklet/ConfigurableSystemProcessExitCodeMapper.java b/spring-batch-samples/src/main/java/org/springframework/batch/sample/tasklet/ConfigurableSystemProcessExitCodeMapper.java index 5e4af2ef7..b39d1c377 100644 --- a/spring-batch-samples/src/main/java/org/springframework/batch/sample/tasklet/ConfigurableSystemProcessExitCodeMapper.java +++ b/spring-batch-samples/src/main/java/org/springframework/batch/sample/tasklet/ConfigurableSystemProcessExitCodeMapper.java @@ -17,7 +17,7 @@ public class ConfigurableSystemProcessExitCodeMapper implements SystemProcessExi public static final String ELSE_KEY = "else"; - private Map mappings; + private Map mappings; public ExitStatus getExitStatus(int exitCode) { ExitStatus exitStatus = (ExitStatus) mappings.get(new Integer(exitCode)); @@ -32,7 +32,7 @@ public class ConfigurableSystemProcessExitCodeMapper implements SystemProcessExi * @param mappings Integer exit code keys to * {@link org.springframework.batch.repeat.ExitStatus} values. */ - public void setMappings(Map mappings) { + public void setMappings(Map mappings) { Assert.notNull(mappings.get(ELSE_KEY)); this.mappings = mappings; } diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/AbstractCustomerCreditIncreaseTests.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/AbstractCustomerCreditIncreaseTests.java index ec0be8f4c..912b4acf9 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/AbstractCustomerCreditIncreaseTests.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/AbstractCustomerCreditIncreaseTests.java @@ -42,7 +42,7 @@ public abstract class AbstractCustomerCreditIncreaseTests extends AbstractValida protected static final String ID_COLUMN = "ID"; - private List creditsBeforeUpdate; + private List creditsBeforeUpdate; /** * @param jdbcTemplate @@ -62,10 +62,11 @@ public abstract class AbstractCustomerCreditIncreaseTests extends AbstractValida /** * All customers have the same credit */ + @SuppressWarnings("unchecked") protected void validatePreConditions() throws Exception { super.validatePreConditions(); ensureState(); - creditsBeforeUpdate = (List) new TransactionTemplate(transactionManager).execute(new TransactionCallback() { + creditsBeforeUpdate = (List) new TransactionTemplate(transactionManager).execute(new TransactionCallback() { public Object doInTransaction(TransactionStatus status) { return jdbcTemplate.query(ALL_CUSTOMERS, new RowMapper() { @@ -100,7 +101,7 @@ public abstract class AbstractCustomerCreditIncreaseTests extends AbstractValida */ protected void validatePostConditions() throws Exception { - final List matches = new ArrayList(); + final List matches = new ArrayList(); new TransactionTemplate(transactionManager).execute(new TransactionCallback() { public Object doInTransaction(TransactionStatus status) { @@ -130,7 +131,7 @@ public abstract class AbstractCustomerCreditIncreaseTests extends AbstractValida /** * @param matches */ - protected void checkMatches(List matches) { + protected void checkMatches(List matches) { // no-op... } diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/CompositeItemWriterSampleFunctionalTests.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/CompositeItemWriterSampleFunctionalTests.java index 815fe270f..c524d0a50 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/CompositeItemWriterSampleFunctionalTests.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/CompositeItemWriterSampleFunctionalTests.java @@ -7,7 +7,6 @@ import java.math.BigDecimal; import java.sql.ResultSet; import java.sql.SQLException; import java.util.ArrayList; -import java.util.Iterator; import java.util.List; import org.apache.commons.io.IOUtils; @@ -47,7 +46,7 @@ public class CompositeItemWriterSampleFunctionalTests extends AbstractValidating } private void checkOutputTable() { - final List trades = new ArrayList() {{ + final List trades = new ArrayList() {{ add(new Trade("UK21341EAH41", 211, new BigDecimal("31.11"), "customer1")); add(new Trade("UK21341EAH42", 212, new BigDecimal("32.11"), "customer2")); add(new Trade("UK21341EAH43", 213, new BigDecimal("33.11"), "customer3")); @@ -72,13 +71,13 @@ public class CompositeItemWriterSampleFunctionalTests extends AbstractValidating } + @SuppressWarnings("unchecked") private void checkOutputFile() throws FileNotFoundException, IOException { - List outputLines = IOUtils.readLines( + List outputLines = IOUtils.readLines( new FileInputStream("target/test-outputs/20070122.testStream.ParallelCustomerReportStep.TEMP.txt")); String output = ""; - for (Iterator iterator = outputLines.listIterator(); iterator.hasNext();) { - String line = (String) iterator.next(); + for (String line : outputLines) { output += line; } diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/HibernateFailureJobFunctionalTests.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/HibernateFailureJobFunctionalTests.java index 172079c6a..b5da86825 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/HibernateFailureJobFunctionalTests.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/HibernateFailureJobFunctionalTests.java @@ -73,7 +73,7 @@ public class HibernateFailureJobFunctionalTests extends * * @see org.springframework.batch.sample.AbstractCustomerCreditIncreaseTests#checkMatches(java.util.List) */ - protected void checkMatches(List matches) { + protected void checkMatches(List matches) { assertFalse(matches.contains(new BigDecimal(2))); } diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/TradeJobFunctionalTests.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/TradeJobFunctionalTests.java index da3a1ea09..7e5a2f6ce 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/TradeJobFunctionalTests.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/TradeJobFunctionalTests.java @@ -38,12 +38,12 @@ public class TradeJobFunctionalTests extends AbstractValidatingBatchLauncherTest private static final String GET_TRADES = "select ISIN, QUANTITY, PRICE, CUSTOMER from TRADE order by ISIN"; private static final String GET_CUSTOMERS = "select NAME, CREDIT from CUSTOMER order by NAME"; - private List customers; - private List trades; + private List customers; + private List trades; private int activeRow = 0; private JdbcOperations jdbcTemplate; - private Map credits = new HashMap(); + private Map credits = new HashMap(); /** * @param jdbcTemplate the jdbcTemplate to set @@ -55,13 +55,14 @@ public class TradeJobFunctionalTests extends AbstractValidatingBatchLauncherTest /* (non-Javadoc) * @see org.springframework.test.AbstractSingleSpringContextTests#onSetUp() */ + @SuppressWarnings("unchecked") protected void onSetUp() throws Exception { super.onSetUp(); jdbcTemplate.update("delete from TRADE"); - List list = jdbcTemplate.queryForList("select name, CREDIT from customer"); - for (Iterator iterator = list.iterator(); iterator.hasNext();) { - Map map = (Map) iterator.next(); - credits.put(map.get("NAME"), new Double(((Number)map.get("CREDIT")).doubleValue())); + List> list = jdbcTemplate.queryForList("select name, CREDIT from customer"); + for (Iterator> iterator = list.iterator(); iterator.hasNext();) { + Map map = iterator.next(); + credits.put((String) map.get("NAME"), new Double(((Number)map.get("CREDIT")).doubleValue())); } } @@ -73,12 +74,12 @@ public class TradeJobFunctionalTests extends AbstractValidatingBatchLauncherTest // assertTrue(((Resource)applicationContext.getBean("customerFileLocator")).exists()); - customers = new ArrayList() {{add(new Customer("customer1", (((Double)credits.get("customer1")).doubleValue() - 98.34))); - add(new Customer("customer2", (((Double)credits.get("customer2")).doubleValue() - 18.12 - 12.78))); - add(new Customer("customer3", (((Double)credits.get("customer3")).doubleValue() - 109.25))); - add(new Customer("customer4", (((Double)credits.get("customer4")).doubleValue() - 123.39)));}}; + customers = new ArrayList() {{add(new Customer("customer1", (credits.get("customer1").doubleValue() - 98.34))); + add(new Customer("customer2", (credits.get("customer2").doubleValue() - 18.12 - 12.78))); + add(new Customer("customer3", (credits.get("customer3").doubleValue() - 109.25))); + add(new Customer("customer4", (credits.get("customer4").doubleValue() - 123.39)));}}; - trades = new ArrayList() {{add(new Trade("UK21341EAH45", 978, new BigDecimal("98.34"), "customer1")); + trades = new ArrayList() {{add(new Trade("UK21341EAH45", 978, new BigDecimal("98.34"), "customer1")); add(new Trade("UK21341EAH46", 112, new BigDecimal("18.12"), "customer2")); add(new Trade("UK21341EAH47", 245, new BigDecimal("12.78"), "customer2")); add(new Trade("UK21341EAH48", 108, new BigDecimal("109.25"), "customer3")); diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/advice/JobExecutionNotificationPublisherTests.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/advice/JobExecutionNotificationPublisherTests.java index 6c5eec544..1e077f63f 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/advice/JobExecutionNotificationPublisherTests.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/advice/JobExecutionNotificationPublisherTests.java @@ -34,7 +34,7 @@ public class JobExecutionNotificationPublisherTests extends TestCase { JobExecutionNotificationPublisher publisher = new JobExecutionNotificationPublisher(); public void testRepeatOperationsOpenUsed() throws Exception { - final List list = new ArrayList(); + final List list = new ArrayList(); publisher.setNotificationPublisher(new NotificationPublisher() { public void sendNotification(Notification notification) throws UnableToSendNotificationException { list.add(notification); @@ -42,7 +42,7 @@ public class JobExecutionNotificationPublisherTests extends TestCase { }); publisher.onApplicationEvent(new SimpleMessageApplicationEvent(this, "foo")); assertEquals(1, list.size()); - String message = ((Notification) list.get(0)).getMessage(); + String message = list.get(0).getMessage(); assertTrue("Message does not contain 'foo': ", message.indexOf("foo") > 0); } diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/dao/FlatFileOrderWriterTests.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/dao/FlatFileOrderWriterTests.java index b78907225..1073b8f0f 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/dao/FlatFileOrderWriterTests.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/dao/FlatFileOrderWriterTests.java @@ -21,7 +21,7 @@ import org.springframework.batch.sample.domain.Order; public class FlatFileOrderWriterTests extends TestCase { - List list = new ArrayList(); + List list = new ArrayList(); private ItemWriter output = new AbstractItemWriter() { public void write(Object output) { @@ -47,7 +47,7 @@ public class FlatFileOrderWriterTests extends TestCase { order.setCustomer(new Customer()); order.setBilling(new BillingInfo()); order.setBillingAddress(new Address()); - List lineItems = new ArrayList(); + List lineItems = new ArrayList(); LineItem item = new LineItem(); item.setPrice(BigDecimal.valueOf(0)); lineItems.add(item); @@ -59,7 +59,7 @@ public class FlatFileOrderWriterTests extends TestCase { LineAggregator aggregator = new StubLineAggregator(); //create map of aggregators and set it to writer - Map aggregators = new HashMap(); + Map aggregators = new HashMap(); OrderTransformer converter = new OrderTransformer(); aggregators.put("header", aggregator); @@ -77,7 +77,7 @@ public class FlatFileOrderWriterTests extends TestCase { //verify method calls assertEquals(1, list.size()); assertTrue(list.get(0) instanceof List); - assertEquals("02007/06/01", ((List) list.get(0)).get(0)); + assertEquals("02007/06/01", ((List) list.get(0)).get(0)); } diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/dao/JdbcJobRepositoryTests.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/dao/JdbcJobRepositoryTests.java index 0b9e26e61..8509620de 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/dao/JdbcJobRepositoryTests.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/dao/JdbcJobRepositoryTests.java @@ -1,5 +1,6 @@ package org.springframework.batch.sample.dao; +import java.io.Serializable; import java.sql.Timestamp; import java.util.ArrayList; import java.util.HashSet; @@ -23,11 +24,11 @@ public class JdbcJobRepositoryTests extends AbstractTransactionalDataSourceSprin private JobSupport jobConfiguration; - private Set jobExecutionIds = new HashSet(); + private Set jobExecutionIds = new HashSet(); - private Set jobIds = new HashSet(); + private Set jobIds = new HashSet(); - private List list = new ArrayList(); + private List list = new ArrayList(); public void setRepository(JobRepository repository) { this.repository = repository; @@ -55,18 +56,18 @@ public class JdbcJobRepositoryTests extends AbstractTransactionalDataSourceSprin protected void onTearDownAfterTransaction() throws Exception { startNewTransaction(); - for (Iterator iterator = jobExecutionIds.iterator(); iterator.hasNext();) { - Long id = (Long) iterator.next(); + for (Iterator iterator = jobExecutionIds.iterator(); iterator.hasNext();) { + Long id = iterator.next(); getJdbcTemplate().update("DELETE FROM BATCH_JOB_EXECUTION where JOB_EXECUTION_ID=?", new Object[] { id }); } - for (Iterator iterator = jobIds.iterator(); iterator.hasNext();) { - Long id = (Long) iterator.next(); + for (Iterator iterator = jobIds.iterator(); iterator.hasNext();) { + Long id = iterator.next(); getJdbcTemplate().update("DELETE FROM BATCH_JOB_INSTANCE where JOB_INSTANCE_ID=?", new Object[] { id }); } setComplete(); endTransaction(); - for (Iterator iterator = jobIds.iterator(); iterator.hasNext();) { - Long id = (Long) iterator.next(); + for (Iterator iterator = jobIds.iterator(); iterator.hasNext();) { + Long id = iterator.next(); int count = getJdbcTemplate().queryForInt("SELECT COUNT(*) FROM BATCH_JOB_INSTANCE where JOB_INSTANCE_ID=?", new Object[] { id }); assertEquals(0, count); } diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/dao/OrderTransformerTests.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/dao/OrderTransformerTests.java index 567526c81..07b3fb887 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/dao/OrderTransformerTests.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/dao/OrderTransformerTests.java @@ -16,17 +16,19 @@ package org.springframework.batch.sample.dao; import java.math.BigDecimal; +import java.util.ArrayList; import java.util.Collection; -import java.util.Collections; import java.util.Date; import java.util.HashMap; import junit.framework.TestCase; import org.springframework.batch.item.file.transform.DelimitedLineAggregator; +import org.springframework.batch.item.file.transform.LineAggregator; import org.springframework.batch.sample.domain.Address; import org.springframework.batch.sample.domain.BillingInfo; import org.springframework.batch.sample.domain.Customer; +import org.springframework.batch.sample.domain.LineItem; import org.springframework.batch.sample.domain.Order; /** @@ -38,7 +40,7 @@ public class OrderTransformerTests extends TestCase { private OrderTransformer converter = new OrderTransformer(); public void testConvert() throws Exception { - converter.setAggregators(new HashMap() { + converter.setAggregators(new HashMap() { { put("header", new DelimitedLineAggregator()); put("customer", new DelimitedLineAggregator()); @@ -53,7 +55,7 @@ public class OrderTransformerTests extends TestCase { order.setCustomer(new Customer()); order.setBillingAddress(new Address()); order.setBilling(new BillingInfo()); - order.setLineItems(Collections.EMPTY_LIST); + order.setLineItems(new ArrayList()); order.setTotalPrice(new BigDecimal(10)); Object result = converter.transform(order); assertTrue(result instanceof Collection); diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/item/reader/OrderItemReaderTests.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/item/reader/OrderItemReaderTests.java index f3cae9ba4..1bac0ef67 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/item/reader/OrderItemReaderTests.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/item/reader/OrderItemReaderTests.java @@ -136,7 +136,7 @@ public class OrderItemReaderTests extends TestCase { assertEquals(o.getShipping(), shippingInfo); //there should be 3 line items assertEquals(3, o.getLineItems().size()); - for (Iterator i = o.getLineItems().iterator(); i.hasNext();) { + for (Iterator i = o.getLineItems().iterator(); i.hasNext();) { assertEquals(i.next(),item); } diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/launch/RemoteLauncherTests.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/launch/RemoteLauncherTests.java index 468b7a632..32a3cc592 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/launch/RemoteLauncherTests.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/launch/RemoteLauncherTests.java @@ -39,7 +39,7 @@ public class RemoteLauncherTests extends TestCase { private static Log logger = LogFactory.getLog(RemoteLauncherTests.class); - private static List errors = new ArrayList(); + private static List errors = new ArrayList(); private static Thread thread; @@ -134,7 +134,7 @@ public class RemoteLauncherTests extends TestCase { * @param interfaceType * @throws MalformedObjectNameException */ - private static Object getMBean(MBeanServerConnectionFactoryBean connectionFactory, String objectName, Class interfaceType) + private static Object getMBean(MBeanServerConnectionFactoryBean connectionFactory, String objectName, Class interfaceType) throws MalformedObjectNameException { MBeanProxyFactoryBean factory = new MBeanProxyFactoryBean(); factory.setObjectName(objectName); diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/quartz/JobLauncherDetailsTests.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/quartz/JobLauncherDetailsTests.java index 32c8cb5f8..e426f17bd 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/quartz/JobLauncherDetailsTests.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/quartz/JobLauncherDetailsTests.java @@ -15,6 +15,7 @@ */ package org.springframework.batch.sample.quartz; +import java.io.Serializable; import java.util.ArrayList; import java.util.Date; import java.util.List; @@ -47,7 +48,7 @@ public class JobLauncherDetailsTests extends TestCase { private TriggerFiredBundle firedBundle; - private List list = new ArrayList(); + private List list = new ArrayList(); protected void setUp() throws Exception { details.setJobLauncher(new JobLauncher() { diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/tasklet/ConfigurableSystemProcessExitCodeMapperTests.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/tasklet/ConfigurableSystemProcessExitCodeMapperTests.java index b1f4a7491..f339f6496 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/tasklet/ConfigurableSystemProcessExitCodeMapperTests.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/tasklet/ConfigurableSystemProcessExitCodeMapperTests.java @@ -1,14 +1,11 @@ package org.springframework.batch.sample.tasklet; -import java.util.Collections; import java.util.HashMap; -import java.util.Iterator; import java.util.Map; import junit.framework.TestCase; import org.springframework.batch.repeat.ExitStatus; -import org.springframework.batch.sample.tasklet.ConfigurableSystemProcessExitCodeMapper; /** * Tests for {@link ConfigurableSystemProcessExitCodeMapper} @@ -21,7 +18,7 @@ public class ConfigurableSystemProcessExitCodeMapperTests extends TestCase { * Regular usage scenario - mapping adheres to injected values */ public void testMapping() { - Map mappings = new HashMap() {{ + Map mappings = new HashMap() {{ put(new Integer(0), ExitStatus.FINISHED); put(new Integer(1), ExitStatus.FAILED); put(new Integer(2), ExitStatus.CONTINUABLE); @@ -33,8 +30,7 @@ public class ConfigurableSystemProcessExitCodeMapperTests extends TestCase { mapper.setMappings(mappings); //check explicitly defined values - for (Iterator iterator = mappings.entrySet().iterator(); iterator.hasNext();) { - Map.Entry entry = (Map.Entry) iterator.next(); + for (Map.Entry entry : mappings.entrySet()) { if (entry.getKey().equals(ConfigurableSystemProcessExitCodeMapper.ELSE_KEY)) continue; int exitCode = ((Integer)entry.getKey()).intValue(); @@ -50,7 +46,7 @@ public class ConfigurableSystemProcessExitCodeMapperTests extends TestCase { * Else clause is required in the injected map - setter checks its presence. */ public void testSetMappingsMissingElseClause() { - Map missingElse = Collections.EMPTY_MAP; + Map missingElse = new HashMap(); try { mapper.setMappings(missingElse); fail(); @@ -59,7 +55,7 @@ public class ConfigurableSystemProcessExitCodeMapperTests extends TestCase { // expected } - Map containsElse = new HashMap() {{ + Map containsElse = new HashMap() {{ put(ConfigurableSystemProcessExitCodeMapper.ELSE_KEY, ExitStatus.FAILED); }}; // no error expected now diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/tasklet/ExceptionThrowingItemReaderProxyTests.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/tasklet/ExceptionThrowingItemReaderProxyTests.java index e50c221b4..b4dcabaa9 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/tasklet/ExceptionThrowingItemReaderProxyTests.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/tasklet/ExceptionThrowingItemReaderProxyTests.java @@ -23,7 +23,7 @@ public class ExceptionThrowingItemReaderProxyTests extends TestCase { //create module and set item processor and iteration count ExceptionThrowingItemReaderProxy itemReader = new ExceptionThrowingItemReaderProxy(); - itemReader.setItemReader(new ListItemReader(new ArrayList() {{ + itemReader.setItemReader(new ListItemReader(new ArrayList() {{ add("a"); add("b"); add("c"); diff --git a/spring-batch-samples/src/test/java/org/springframework/batch/sample/tasklet/JobSupport.java b/spring-batch-samples/src/test/java/org/springframework/batch/sample/tasklet/JobSupport.java index 0fa0960ba..fd7e65b4d 100644 --- a/spring-batch-samples/src/test/java/org/springframework/batch/sample/tasklet/JobSupport.java +++ b/spring-batch-samples/src/test/java/org/springframework/batch/sample/tasklet/JobSupport.java @@ -37,7 +37,7 @@ import org.springframework.util.ClassUtils; */ public class JobSupport implements BeanNameAware, Job { - private List steps = new ArrayList(); + private List steps = new ArrayList(); private String name; @@ -98,11 +98,11 @@ public class JobSupport implements BeanNameAware, Job { /* (non-Javadoc) * @see org.springframework.batch.core.domain.IJob#getSteps() */ - public List getSteps() { + public List getSteps() { return steps; } - public void setSteps(List steps) { + public void setSteps(List steps) { this.steps.clear(); this.steps.addAll(steps); }