Skip to content

Give every shells case the suite's two-minute case timeout - #1710

Merged
MattA-Official merged 1 commit into
unified/mainfrom
claude/admin-ux-shells-timeout
Oct 28, 2026
Merged

MattA-Official merged 1 commit into
unified/mainfrom
claude/admin-ux-shells-timeout

Conversation

@MattA-Official

Copy link
Copy Markdown
Member

Requested by Matt · project thread

tests/e2e/shells.test.ts "the public chrome is stage black while the page is not" timed out three times today: #1708 once, #1706 twice. Each time it was the first case of its CI slice, on a dev server that had just booted. It and eight other cases in the file ran under bun's 5-second default, which a cold home page can exceed.

What changes

  • shells.test.ts gains CASE_TIMEOUT_MS = 120_000, the value other suites use, and every case without its own timeout takes it. No assertion changes and no product code changes.

Checks

  • shells.test.ts passes locally (15 pass, 0 fail). lint, check (7/7) and typecheck:bun pass.

Generated by Claude Code

The first case runs on a dev server that has just booted, and under bun's
5-second default it timed out before the home page had compiled.
@MattA-Official MattA-Official self-assigned this Oct 28, 2026
@MattA-Official
MattA-Official merged commit dd4a042 into unified/main Oct 28, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant