[Improve][SeaTunnel-Web] Change JobStatus to enum type to avoid hard coding

Author: wuchunfu

seatunnel-web-it/src/test/java/org/apache/seatunnel/app/test/JobControllerTest.java

@@ -29,6 +29,7 @@
 import org.apache.seatunnel.app.domain.response.job.JobRes;
 import org.apache.seatunnel.app.domain.response.metrics.JobPipelineDetailMetricsRes;
 import org.apache.seatunnel.app.utils.JobTestingUtils;
+import org.apache.seatunnel.engine.core.job.JobStatus;
 import org.apache.seatunnel.server.common.SeatunnelErrorEnum;
 
 import org.junit.jupiter.api.AfterAll;
@@ -73,7 +74,7 @@ public void createJobWithSingleAPI_shouldExecuteSuccessfully() {
         Result<List<JobPipelineDetailMetricsRes>> listResult =
                 JobTestingUtils.waitForJobCompletion(result.getData());
         assertEquals(1, listResult.getData().size());
-        assertEquals("FINISHED", listResult.getData().get(0).getStatus());
+        assertEquals(JobStatus.FINISHED, listResult.getData().get(0).getStatus());
         assertEquals(5, listResult.getData().get(0).getReadRowCount());
         assertEquals(5, listResult.getData().get(0).getWriteRowCount());
     }

seatunnel-web-it/src/test/java/org/apache/seatunnel/app/test/JobExecutorControllerTest.java

@@ -73,7 +73,7 @@ public void executeJob_shouldReturnSuccess_whenValidRequest() {
         Result<List<JobPipelineDetailMetricsRes>> listResult =
                 JobTestingUtils.waitForJobCompletion(result.getData());
         assertEquals(1, listResult.getData().size());
-        assertEquals("FINISHED", listResult.getData().get(0).getStatus());
+        assertEquals(JobStatus.FINISHED, listResult.getData().get(0).getStatus());
         assertEquals(5, listResult.getData().get(0).getReadRowCount());
         assertEquals(5, listResult.getData().get(0).getWriteRowCount());
     }
@@ -88,7 +88,7 @@ public void executeJobWithParameters() {
         Result<List<JobPipelineDetailMetricsRes>> listResult =
                 JobTestingUtils.waitForJobCompletion(result.getData());
         assertEquals(1, listResult.getData().size());
-        assertEquals("FINISHED", listResult.getData().get(0).getStatus());
+        assertEquals(JobStatus.FINISHED, listResult.getData().get(0).getStatus());
         assertEquals(5, listResult.getData().get(0).getReadRowCount());
         assertEquals(5, listResult.getData().get(0).getWriteRowCount());
         String generatedJobFile = getGenerateJobFile(String.valueOf(jobVersionId));
@@ -122,7 +122,7 @@ public void executeJobWithParameters() {
         assertTrue(result.getData() > 0);
         listResult = JobTestingUtils.waitForJobCompletion(result.getData());
         assertEquals(1, listResult.getData().size());
-        assertEquals("FINISHED", listResult.getData().get(0).getStatus());
+        assertEquals(JobStatus.FINISHED, listResult.getData().get(0).getStatus());
         assertEquals(numberOfRecords, listResult.getData().get(0).getReadRowCount());
         assertEquals(numberOfRecords, listResult.getData().get(0).getWriteRowCount());
 
@@ -281,7 +281,7 @@ public void executeJob_JobStatusUpdate_WhenSubmissionFailed() {
         Result<JobExecutionStatus> jobExecutionStatusResult =
                 jobExecutorControllerWrapper.getJobExecutionStatus(jobInstanceId);
         assertTrue(jobExecutionStatusResult.isSuccess());
-        assertEquals(JobStatus.FAILED.name(), jobExecutionStatusResult.getData().getJobStatus());
+        assertEquals(JobStatus.FAILED, jobExecutionStatusResult.getData().getJobStatus());
         assertNotNull(jobExecutionStatusResult.getData().getErrorMessage());
 
         // Invalid jobInstanceId
@@ -306,7 +306,7 @@ public void storeErrorMessageWhenJobFailed() throws InterruptedException {
         Result<SeaTunnelJobInstanceDto> jobExecutionDetailResult =
                 jobExecutorControllerWrapper.getJobExecutionDetail(jobInstanceId);
         assertTrue(jobExecutionDetailResult.isSuccess());
-        assertEquals(JobStatus.FAILED.name(), jobExecutionDetailResult.getData().getJobStatus());
+        assertEquals(JobStatus.FAILED, jobExecutionDetailResult.getData().getJobStatus());
         assertNotNull(jobExecutionDetailResult.getData().getErrorMessage());
         assertNotNull(jobExecutionDetailResult.getData().getJobDefineName());
 

seatunnel-web-it/src/test/java/org/apache/seatunnel/app/test/TaskInstanceControllerTest.java

@@ -25,6 +25,7 @@
 import org.apache.seatunnel.app.domain.dto.job.SeaTunnelJobInstanceDto;
 import org.apache.seatunnel.app.domain.response.metrics.JobPipelineDetailMetricsRes;
 import org.apache.seatunnel.app.utils.JobTestingUtils;
+import org.apache.seatunnel.engine.core.job.JobStatus;
 
 import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.BeforeAll;
@@ -75,7 +76,7 @@ private static void extractedJob1(String jobName) {
         Result<List<JobPipelineDetailMetricsRes>> listResult =
                 JobTestingUtils.waitForJobCompletion(execuitonResult.getData());
         assertEquals(1, listResult.getData().size());
-        assertEquals("FINISHED", listResult.getData().get(0).getStatus());
+        assertEquals(JobStatus.FINISHED, listResult.getData().get(0).getStatus());
     }
 
     @AfterAll