From 7b68329d2aea136a659584837f35fc83f6f6d014 Mon Sep 17 00:00:00 2001 From: Dotta Date: Mon, 21 Sep 2026 19:51:10 -0500 Subject: [PATCH] fix(e2e): assert the fixture terminal status in the browser Blocker baseline cases correctly reach Blocked and must not inherit the generic completion-only Done assertion. Preserve run-count and durable-state assertions. Co-Authored-By: Paperclip --- tests/runner-e2e/runner.spec.ts | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/runner-e2e/runner.spec.ts b/tests/runner-e2e/runner.spec.ts index 1523fa36706..61480bcade8 100644 --- a/tests/runner-e2e/runner.spec.ts +++ b/tests/runner-e2e/runner.spec.ts @@ -2346,9 +2346,13 @@ for (const execution of executions) { await expect(terminalAgentReplies.first()).toHaveText(marker, { useInnerText: true, }); + // The fixture owns the expected disposition; blocked workflows must prove + // their Blocked UI rather than inheriting the completion-only Done check. + const expectedStatus = execution.task.expectedTerminalState.issue; + const expectedStatusLabel = expectedStatus.replace(/_/g, " ").replace(/^./, (letter) => letter.toUpperCase()); await expect( page.getByTestId("issue-detail-header").getByRole("button", { - name: "Change status (current: Done)", + name: `Change status (current: ${expectedStatusLabel})`, exact: true, }), ).toBeVisible({ timeout: 30_000 });