Skip to content

host: Update two test modules with multi-realm index cache - #5737

Draft
backspace wants to merge 1 commit into
mainfrom
adopt-multi-realm
Draft

host: Update two test modules with multi-realm index cache#5737
backspace wants to merge 1 commit into
mainfrom
adopt-multi-realm

Conversation

@backspace

Copy link
Copy Markdown
Contributor

No description provided.

operator-mode-acceptance (4.09m) and ai-assistant-panel/tools (3.84m) each
build two realms per test with identical fixtures, in a shared beforeEach, so
they qualify for the multi-realm path: an explicit capture after the second
realm, since the automatic one lands after the first and would omit the other.

These are the only two of the eight remaining multi-realm candidates that fit.
The rest divide into two groups, neither of which belongs here:

host-submode, code-submode, card-playground build realms inside nested
modules, and card-copy builds one inside a test body — so those realms exist
for only some tests, which the snapshot cannot represent. The guard rejects
them rather than serving an empty index.

inspector, create-file, card-playground and code-submode already cache their
realm setup through `withCachedRealmSetup`, which restores the snapshot and
then still runs setup, leaving the boot index a no-op instead of skipping it.
That tolerates realms some tests don't build, and keeps the loader bookkeeping
that `skipBootIndex` omits — the same bookkeeping two store tests needed.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Preview deployments

Host Test Results

    1 files      1 suites   1h 49m 15s ⏱️
4 066 tests 4 052 ✅ 14 💤 0 ❌
4 085 runs  4 071 ✅ 14 💤 0 ❌

Results for commit cebfd96.

Realm Server Test Results

    1 files      1 suites   14m 48s ⏱️
2 096 tests 2 096 ✅ 0 💤 0 ❌
2 175 runs  2 175 ✅ 0 💤 0 ❌

Results for commit cebfd96.

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