Enhance job status aggregation logic

- Updated the condition for returning `StatusFailure` to ensure it only occurs when all jobs are done.

Author: nienjiuntai

models/actions/run_job.go

@@ -168,10 +168,12 @@ func UpdateRunJob(ctx context.Context, job *ActionRunJob, cond builder.Cond, col
 func AggregateJobStatus(jobs []*ActionRunJob) Status {
 	allSuccessOrSkipped := len(jobs) != 0
 	allSkipped := len(jobs) != 0
+	allDone := len(jobs) != 0
 	var hasFailure, hasCancelled, hasWaiting, hasRunning, hasBlocked bool
 	for _, job := range jobs {
 		allSuccessOrSkipped = allSuccessOrSkipped && (job.Status == StatusSuccess || job.Status == StatusSkipped)
 		allSkipped = allSkipped && job.Status == StatusSkipped
+		allDone = allDone && job.Status.IsDone()
 		hasFailure = hasFailure || job.Status == StatusFailure
 		hasCancelled = hasCancelled || job.Status == StatusCancelled
 		hasWaiting = hasWaiting || job.Status == StatusWaiting
@@ -185,7 +187,7 @@ func AggregateJobStatus(jobs []*ActionRunJob) Status {
 		return StatusSuccess
 	case hasCancelled:
 		return StatusCancelled
-	case hasFailure:
+	case hasFailure && allDone:
 		return StatusFailure
 	case hasRunning:
 		return StatusRunning