From c66f98501d235a878efc8ac9c972d49366f4e6e0 Mon Sep 17 00:00:00 2001 From: dsyer Date: Wed, 3 Sep 2008 11:30:50 +0000 Subject: [PATCH] RESOLVED - issue BATCH-807: Serialization bug in JobExecution --- .../java/org/springframework/batch/core/JobExecution.java | 2 +- .../org/springframework/batch/core/JobExecutionTests.java | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/spring-batch-core/src/main/java/org/springframework/batch/core/JobExecution.java b/spring-batch-core/src/main/java/org/springframework/batch/core/JobExecution.java index bc4704586..d596700e3 100644 --- a/spring-batch-core/src/main/java/org/springframework/batch/core/JobExecution.java +++ b/spring-batch-core/src/main/java/org/springframework/batch/core/JobExecution.java @@ -33,7 +33,7 @@ public class JobExecution extends Entity { private JobInstance jobInstance; - private volatile transient Collection stepExecutions = new HashSet(); + private volatile Collection stepExecutions = new HashSet(); private volatile BatchStatus status = BatchStatus.STARTING; diff --git a/spring-batch-core/src/test/java/org/springframework/batch/core/JobExecutionTests.java b/spring-batch-core/src/test/java/org/springframework/batch/core/JobExecutionTests.java index 0db2b9a34..47ba2b425 100644 --- a/spring-batch-core/src/test/java/org/springframework/batch/core/JobExecutionTests.java +++ b/spring-batch-core/src/test/java/org/springframework/batch/core/JobExecutionTests.java @@ -151,7 +151,8 @@ public class JobExecutionTests extends TestCase { public void testSerialization() { byte[] serialized = SerializationUtils.serialize(execution); - - assertEquals(execution, SerializationUtils.deserialize(serialized)); + JobExecution deserialize = (JobExecution) SerializationUtils.deserialize(serialized); + assertEquals(execution, deserialize); + assertNotNull(deserialize.createStepExecution("foo")); } }