diff --git a/spring-batch-core/src/test/java/org/springframework/batch/core/step/item/FaultTolerantStepFactoryBeanRollbackTests.java b/spring-batch-core/src/test/java/org/springframework/batch/core/step/item/FaultTolerantStepFactoryBeanRollbackTests.java
index 700adab37..716e256ce 100644
--- a/spring-batch-core/src/test/java/org/springframework/batch/core/step/item/FaultTolerantStepFactoryBeanRollbackTests.java
+++ b/spring-batch-core/src/test/java/org/springframework/batch/core/step/item/FaultTolerantStepFactoryBeanRollbackTests.java
@@ -339,7 +339,7 @@ public class FaultTolerantStepFactoryBeanRollbackTests {
 		assertEquals(BatchStatus.COMPLETED, stepExecution.getStatus());
 
 		assertEquals("[1, 2, 3, 4, 5]", processor.getProcessed().toString());
-		assertEquals("[1, 2, 3, 5]", processor.getCommitted().toString());
+		assertEquals("[1, 2, 3, 4, 5]", processor.getCommitted().toString());
 		assertEquals("[1, 2, 3, 5]", writer.getWritten().toString());
 		assertEquals("[1, 2, 3, 5]", writer.getCommitted().toString());
 	}
diff --git a/spring-batch-core/src/test/java/org/springframework/batch/core/step/item/SkipReaderStub.java b/spring-batch-core/src/test/java/org/springframework/batch/core/step/item/SkipReaderStub.java
index dd7ea45d3..9c3bf9257 100644
--- a/spring-batch-core/src/test/java/org/springframework/batch/core/step/item/SkipReaderStub.java
+++ b/spring-batch-core/src/test/java/org/springframework/batch/core/step/item/SkipReaderStub.java
@@ -64,8 +64,8 @@ public class SkipReaderStub<T> extends AbstractExceptionThrowingItemHandlerStub<
 			return null;
 		}
 		T item = items[counter];
-		read.add(item);
 		checkFailure(item);
+		read.add(item);
 		return item;
 	}
 }