mirror of
https://github.com/openclaw/openclaw.git
synced 2026-10-03 01:29:56 +00:00
fix(test): route bootstrap state consumers through host broker
6c1e1667bdmoved existing canonical workspace snapshot reads and conditional alias registration into the shared-state worker. Embedded bootstrap preparation and session reset prompt tests still ran in Vitest threads, which the host-only SQLite broker correctly refuses. Add both suites to the canonical database-worker inventory consumed by the fork-based infra owner and CI planner. Extend the existing ownership regression with both required consumers and auto-reply exclusion coverage. Keep product behavior and all existing runtime assertions unchanged. On pinned main3df4459de8, the suites reproduced 4/4 and 6/8 failures with the exact host-broker refusal. After routing, they pass 4/4 and 8/8. The extended ownership assertion fails before the inventory fix; all 115 planner tests and 50 bootstrap sibling tests pass afterward. Changed-scope checks and independent P0 review pass. Main6b2af49f71has no duplicate fix in the relevant paths. Production delta: zero; test/support: +6 lines. Test cost: node scripts/run-vitest.mjs <file> --maxWorkers=1 wall times: attempt-bootstrap-prepare.test.ts 326.31s (cold build); session-reset-prompt.test.ts 146.74s; ci-node-test-plan.test.ts 427.46s; bootstrap-files.test.ts 486.50s, on the loaded macOS host.
This commit is contained in:
parent
688f68e777
commit
b1ad1d6455
2 changed files with 6 additions and 0 deletions
|
|
@ -55,6 +55,7 @@ import {
|
|||
import { createAgentsSupportVitestConfig } from "../vitest/vitest.agents-support.config.ts";
|
||||
import { createAgentsToolsVitestConfig } from "../vitest/vitest.agents-tools.config.ts";
|
||||
import { createAgentsVitestConfig } from "../vitest/vitest.agents.config.ts";
|
||||
import { createAutoReplyReplyVitestConfig } from "../vitest/vitest.auto-reply-reply.config.ts";
|
||||
import { cliProcessTestFiles } from "../vitest/vitest.cli-process-paths.mjs";
|
||||
import { createCliProcessVitestConfig } from "../vitest/vitest.cli-process.config.ts";
|
||||
import { createCommandsVitestConfig } from "../vitest/vitest.commands.config.ts";
|
||||
|
|
@ -3761,6 +3762,8 @@ describe("scripts/lib/ci-node-test-plan.mts", () => {
|
|||
expect(infra.test?.setupFiles).toEqual(support.test?.setupFiles);
|
||||
const admitted = new Set(listMatchedTestFiles(infra));
|
||||
for (const file of [
|
||||
"src/agents/embedded-agent-runner/run/attempt-bootstrap-prepare.test.ts",
|
||||
"src/auto-reply/reply/session-reset-prompt.test.ts",
|
||||
"src/agents/prepared-model-runtime.hot-reload-dispatch.test.ts",
|
||||
"src/agents/subagents/registry/subagent-registry.session-failure.test.ts",
|
||||
"src/plugin-sdk/session-transcript-runtime.test.ts",
|
||||
|
|
@ -3781,6 +3784,7 @@ describe("scripts/lib/ci-node-test-plan.mts", () => {
|
|||
support,
|
||||
createAgentsToolsVitestConfig({}),
|
||||
createAgentsVitestConfig({}),
|
||||
createAutoReplyReplyVitestConfig({}),
|
||||
createPluginSdkLightVitestConfig({}),
|
||||
createPluginSdkVitestConfig({}),
|
||||
createPluginsVitestConfig({}),
|
||||
|
|
|
|||
|
|
@ -9,9 +9,11 @@ export const databaseWorkerCoreTestFiles = [
|
|||
"src/system-agent/transcript-store.test.ts",
|
||||
"src/infra/restart-sentinel.update-result.test.ts",
|
||||
"src/auto-reply/reply/conversation-turn-capture.test.ts",
|
||||
"src/auto-reply/reply/session-reset-prompt.test.ts",
|
||||
"src/status/status-plugin-health.runtime.test.ts",
|
||||
"src/status/status-plugin-health.installed.test.ts",
|
||||
"src/agents/tool-schema-quarantine.test.ts",
|
||||
"src/agents/embedded-agent-runner/run/attempt-bootstrap-prepare.test.ts",
|
||||
"src/agents/embedded-agent-runner/run/attempt-bundle-tools.test.ts",
|
||||
"src/agents/embedded-agent-runner/run/attempt-transcript-recovery.test.ts",
|
||||
"src/agents/embedded-agent-runner/run.compaction-runtime.test.ts",
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue