From f1858a24061edff49cf48eb2feef2d143e198c64 Mon Sep 17 00:00:00 2001 From: dsyer Date: Sun, 30 Sep 2007 12:02:14 +0000 Subject: [PATCH] OPEN - issue BATCH-90: StepExecution and StepExecutionContext are parallel domains, and StepExecution is by comparison anaemic http://opensource.atlassian.com/projects/spring/browse/BATCH-90 Remove setter for job id in JobExecution --- .../org/springframework/batch/core/domain/JobExecution.java | 4 ---- .../springframework/batch/core/domain/JobExecutionTests.java | 2 +- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/core/src/main/java/org/springframework/batch/core/domain/JobExecution.java b/core/src/main/java/org/springframework/batch/core/domain/JobExecution.java index c70869388..100ae10be 100644 --- a/core/src/main/java/org/springframework/batch/core/domain/JobExecution.java +++ b/core/src/main/java/org/springframework/batch/core/domain/JobExecution.java @@ -78,10 +78,6 @@ public class JobExecution extends Entity { return jobId; } - public void setJobId(Long jobId) { - this.jobId = jobId; - } - /** * @param exitCode */ diff --git a/core/src/test/java/org/springframework/batch/core/domain/JobExecutionTests.java b/core/src/test/java/org/springframework/batch/core/domain/JobExecutionTests.java index 0d8ce603a..8ff233f1c 100644 --- a/core/src/test/java/org/springframework/batch/core/domain/JobExecutionTests.java +++ b/core/src/test/java/org/springframework/batch/core/domain/JobExecutionTests.java @@ -66,7 +66,7 @@ public class JobExecutionTests extends TestCase { */ public void testGetJobId() { assertEquals(11, execution.getJobId().longValue()); - execution.setJobId(new Long(23)); + execution = new JobExecution(new Long(23)); assertEquals(23, execution.getJobId().longValue()); }