Skip to content

ci(otter): web browser tests restart Chromium between shards - #80

Merged
Laurin-Notemann merged 1 commit into
mainfrom
fix/web-tests-fresh-browser
Nov 11, 2026
Merged

Laurin-Notemann merged 1 commit into
mainfrom
fix/web-tests-fresh-browser

Conversation

@Laurin-Notemann

Copy link
Copy Markdown

After the 2026-11-11 sync, Test Web and the nightly's Release web tests fail on GitHub's 16 GB runners. Chromium's page closes partway through the browser suite, at a different file each run, after 420 to 480 of 549 files. Every test that ran passed. Upstream's 8-vCPU runners pass the same commit.

The adaptation script now runs the web unit tests and the browser tests as separate processes, with the browser tests in two shards, so each shard gets a fresh Chromium. The generated workflow lines are updated to match.

🤖 Generated with Claude Code

One browser session over all web browser tests runs Chromium out of memory
on the 16 GB GitHub-hosted runners after the 2026-11-11 sync, so the page
closes partway through and Test Web and the nightly fail.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Laurin-Notemann
Laurin-Notemann merged commit 51b36fd into main Nov 11, 2026
19 checks passed
@Laurin-Notemann
Laurin-Notemann deleted the fix/web-tests-fresh-browser branch November 11, 2026 05:28
Laurin-Notemann added a commit that referenced this pull request Nov 12, 2026
* fix(otter): release checks pass after the 2026-11-12 sync

The first nightly after the sync failed its test gates: two files were not
formatted, two server tests still used T3's launchd label and data home, the
settings Tab test expected the Extensions section that web hides, and the web
browser tests again ran Chromium out of memory on GitHub-hosted runners.

Each web runner now runs unit tests on their own and splits its browser share
in two, as #80 did before upstream sharded the job.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* fix(otter): new git refresh test skips Otter's RPCs

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* test(web): command palette waits for its root list before ArrowDown

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* ci(otter): web runners split browser tests into four Chromium sessions

Two sessions per runner still ran Chromium out of memory on both web runners.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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