diff --git a/src/utils/executionStatus.test.ts b/src/utils/executionStatus.test.ts index d7d0f8053..3ab1f51cb 100644 --- a/src/utils/executionStatus.test.ts +++ b/src/utils/executionStatus.test.ts @@ -130,6 +130,19 @@ describe("getOverallExecutionStatusFromStats()", () => { ).toBe("RUNNING"); }); + test("returns succeeded when completed tasks include intentional skips", () => { + expect( + getOverallExecutionStatusFromStats({ + SUCCEEDED: 2, + SKIPPED: 1, + }), + ).toBe("SUCCEEDED"); + }); + + test("returns skipped when every task was skipped", () => { + expect(getOverallExecutionStatusFromStats({ SKIPPED: 3 })).toBe("SKIPPED"); + }); + test("returns raw status values (use getExecutionStatusLabel for display)", () => { expect( getOverallExecutionStatusFromStats({ diff --git a/src/utils/executionStatus.ts b/src/utils/executionStatus.ts index d580eea08..ed1fe2269 100644 --- a/src/utils/executionStatus.ts +++ b/src/utils/executionStatus.ts @@ -98,8 +98,8 @@ const EXECUTION_STATUS_PRIORITY = [ "WAITING_FOR_UPSTREAM", "QUEUED", "UNINITIALIZED", - "SKIPPED", "SUCCEEDED", + "SKIPPED", ] as const; export type ExecutionStatusStats = Record;