diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/xml/builder/StaxEventItemReaderBuilder.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/xml/builder/StaxEventItemReaderBuilder.java index 374f893f6..ebaee4163 100644 --- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/xml/builder/StaxEventItemReaderBuilder.java +++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/xml/builder/StaxEventItemReaderBuilder.java @@ -215,12 +215,10 @@ public class StaxEventItemReaderBuilder { if (this.saveState) { Assert.state(StringUtils.hasText(this.name), "A name is required when saveState is set to true."); } - else { - reader.setName(this.name); - } Assert.notEmpty(this.fragmentRootElements, "At least one fragment root element is required"); + reader.setName(this.name); reader.setSaveState(this.saveState); reader.setResource(this.resource); reader.setFragmentRootElementNames( diff --git a/spring-batch-infrastructure/src/test/java/org/springframework/batch/item/xml/builder/StaxEventItemReaderBuilderTests.java b/spring-batch-infrastructure/src/test/java/org/springframework/batch/item/xml/builder/StaxEventItemReaderBuilderTests.java index b5bf35022..478c621d2 100644 --- a/spring-batch-infrastructure/src/test/java/org/springframework/batch/item/xml/builder/StaxEventItemReaderBuilderTests.java +++ b/spring-batch-infrastructure/src/test/java/org/springframework/batch/item/xml/builder/StaxEventItemReaderBuilderTests.java @@ -34,6 +34,7 @@ import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertNull; import static org.junit.Assert.assertNotNull; import static org.junit.Assert.fail; +import static org.springframework.test.util.ReflectionTestUtils.getField; /** * @author Michael Minella @@ -119,6 +120,9 @@ public class StaxEventItemReaderBuilderTests { assertEquals("five", item.getSecond()); assertEquals("six", item.getThird()); assertEquals(2, executionContext.size()); + + Object executionContextUserSupport = getField(reader, "executionContextUserSupport"); + assertEquals("fooReader", getField(executionContextUserSupport, "name")); } @Test(expected = ItemStreamException.class)