mirror of
https://github.com/openclaw/openclaw.git
synced 2026-08-25 03:45:46 -06:00
test: consolidate OpenClaw test state fixtures (#113576)
* test: consolidate OpenClaw test state fixtures * test(plugin-sdk): expose isolated test state Promote the isolated OpenClaw test-state lifecycle through a narrow published Plugin SDK subpath so extension tests no longer import private core helpers. This intentional SDK surface addition is maintainer-approved. * test: use SDK test-state seam in extensions Route bundled extension suites through the focused repo-local Plugin SDK test-state entrypoint and remove the Codex projector harness exports made stale by fixture consolidation. Keep the seam out of production builds and published package artifacts while auditing its real consumers in the full-tree deadcode scan. * test(plugins): map test-state in package boundaries
This commit is contained in:
committed by
GitHub
parent
3d1369ff4f
commit
332006bc1f
@@ -7,6 +7,7 @@ import type { ChannelAccountSnapshot } from "../channels/plugins/types.public.js
|
||||
import type { ChannelPlugin } from "../channels/plugins/types.public.js";
|
||||
import { createPluginRecord } from "../plugins/status.test-fixtures.js";
|
||||
import { MAX_TIMER_TIMEOUT_MS } from "../shared/number-coercion.js";
|
||||
import { createOpenClawTestState } from "../test-utils/openclaw-test-state.js";
|
||||
import type { HealthSummary } from "./health.js";
|
||||
|
||||
let testConfig: Record<string, unknown> = {};
|
||||
@@ -558,9 +559,10 @@ describe("getHealthSnapshot", () => {
|
||||
testConfig = { session: { store: "/tmp/x" } };
|
||||
testStore = {};
|
||||
setActivePluginRegistry(createTestRegistry([]));
|
||||
const tmpStateDir = fs.mkdtempSync(path.join(os.tmpdir(), "openclaw-health-dq-"));
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
process.env.OPENCLAW_STATE_DIR = tmpStateDir;
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-health-dq-",
|
||||
});
|
||||
try {
|
||||
const { moveDeliveryQueueEntryToFailed, upsertDeliveryQueueEntry } =
|
||||
await import("../infra/delivery-queue-sqlite.js");
|
||||
@@ -592,14 +594,7 @@ describe("getHealthSnapshot", () => {
|
||||
],
|
||||
});
|
||||
} finally {
|
||||
const { closeOpenClawStateDatabaseForTest } = await import("../state/openclaw-state-db.js");
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
fs.rmSync(tmpStateDir, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
|
||||
|
||||
@@ -1,6 +1,5 @@
|
||||
// Doctor health contribution tests cover plugin-provided health checks.
|
||||
import fs from "node:fs";
|
||||
import os from "node:os";
|
||||
import nodePath from "node:path";
|
||||
import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
|
||||
import type { DoctorPrompter } from "../commands/doctor-prompter.js";
|
||||
@@ -8,6 +7,7 @@ import type { OpenClawConfig } from "../config/types.openclaw.js";
|
||||
import { LEGACY_SECRETREF_ENV_MARKER_PREFIX } from "../config/types.secrets.js";
|
||||
import { migrateLegacySecretRefEnvMarkers } from "../secrets/legacy-secretref-env-marker.js";
|
||||
import { readConfigMachineState } from "../state/config-machine-state.js";
|
||||
import { createOpenClawTestState } from "../test-utils/openclaw-test-state.js";
|
||||
import { CORE_HEALTH_CHECKS } from "./doctor-core-checks.js";
|
||||
import { resolveDoctorContributionHealthChecks } from "./doctor-health-contributions.js";
|
||||
import {
|
||||
@@ -1919,12 +1919,13 @@ describe("doctor health contributions", () => {
|
||||
});
|
||||
|
||||
it("keeps legacy plugin dependency lint opt-in and read-only", async () => {
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
const tempDir = fs.mkdtempSync(nodePath.join(os.tmpdir(), "openclaw-legacy-plugin-deps-lint-"));
|
||||
const stateDir = nodePath.join(tempDir, "state");
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-legacy-plugin-deps-lint-",
|
||||
});
|
||||
const stateDir = openClawState.stateDir;
|
||||
const legacyRuntimeRoot = nodePath.join(stateDir, "plugin-runtime-deps");
|
||||
fs.mkdirSync(legacyRuntimeRoot, { recursive: true });
|
||||
process.env.OPENCLAW_STATE_DIR = stateDir;
|
||||
try {
|
||||
const contributionChecks = await resolveDoctorContributionHealthChecks();
|
||||
const check = contributionChecks.find(
|
||||
@@ -1961,12 +1962,7 @@ describe("doctor health contributions", () => {
|
||||
});
|
||||
expect(fs.existsSync(legacyRuntimeRoot)).toBe(true);
|
||||
} finally {
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
fs.rmSync(tempDir, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
|
||||
|
||||
@@ -28,6 +28,7 @@ import {
|
||||
buildSystemRunApprovalEnvBinding,
|
||||
} from "../../infra/system-run-approval-binding.js";
|
||||
import { resetLogger, setLoggerOverride } from "../../logging.js";
|
||||
import { createOpenClawTestState } from "../../test-utils/openclaw-test-state.js";
|
||||
import {
|
||||
DEFAULT_CHAT_HISTORY_TEXT_MAX_CHARS,
|
||||
augmentChatHistoryWithCanvasBlocks,
|
||||
@@ -5362,9 +5363,10 @@ describe("gateway healthHandlers.health cache freshness", () => {
|
||||
});
|
||||
|
||||
it("merges live dead-lettered delivery queue counts into cached health responses", async () => {
|
||||
const tmpStateDir = fs.mkdtempSync(path.join(os.tmpdir(), "openclaw-health-cached-dq-"));
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
process.env.OPENCLAW_STATE_DIR = tmpStateDir;
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-health-cached-dq-",
|
||||
});
|
||||
try {
|
||||
const { moveDeliveryQueueEntryToFailed, upsertDeliveryQueueEntry } =
|
||||
await import("../../infra/delivery-queue-sqlite.js");
|
||||
@@ -5422,12 +5424,7 @@ describe("gateway healthHandlers.health cache freshness", () => {
|
||||
expect(typeof payload?.deliveryQueues?.failed?.[0]?.oldestFailedAt).toBe("number");
|
||||
expect(mockCallArg(respond, 0, 3)).toEqual({ cached: true });
|
||||
} finally {
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
fs.rmSync(tmpStateDir, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
|
||||
|
||||
@@ -34,6 +34,7 @@ import {
|
||||
resolveIncognitoOpenClawAgentSqlitePath,
|
||||
} from "../state/openclaw-agent-db.js";
|
||||
import { closeOpenClawStateDatabaseForTest } from "../state/openclaw-state-db.js";
|
||||
import { createOpenClawTestState } from "../test-utils/openclaw-test-state.js";
|
||||
import { GATEWAY_CLIENT_MODES, GATEWAY_CLIENT_NAMES } from "../utils/message-channel.js";
|
||||
import { resolveGatewaySessionStoreTarget } from "./session-utils.js";
|
||||
import {
|
||||
@@ -660,12 +661,12 @@ test("sessions.create rejects draft visibility when policy disables drafts", asy
|
||||
});
|
||||
|
||||
test("sessions.create provisions and reuses a session worktree for later runs", async () => {
|
||||
const root = await fs.mkdtemp(
|
||||
path.join(await fs.realpath(os.tmpdir()), "openclaw-session-worktree-"),
|
||||
);
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-session-worktree-",
|
||||
});
|
||||
const root = openClawState.root;
|
||||
const workspace = await initializeGitWorkspace(root);
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
process.env.OPENCLAW_STATE_DIR = path.join(root, "state");
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
testState.agentConfig = { workspace };
|
||||
await createSessionStoreDir();
|
||||
@@ -733,20 +734,17 @@ test("sessions.create provisions and reuses a session worktree for later runs",
|
||||
await managedWorktrees.remove({ id: worktreeId, reason: "test-cleanup", force: true });
|
||||
}
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
testState.agentConfig = undefined;
|
||||
await fs.rm(root, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
|
||||
test("sessions.create honors worktree name/base ref and persists worktree info", async () => {
|
||||
const root = await fs.mkdtemp(
|
||||
path.join(await fs.realpath(os.tmpdir()), "openclaw-session-worktree-target-"),
|
||||
);
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-session-worktree-target-",
|
||||
});
|
||||
const root = openClawState.root;
|
||||
const workspace = await initializeGitWorkspace(root);
|
||||
await execFileAsync("git", ["-C", workspace, "checkout", "-b", "base-branch"]);
|
||||
await fs.writeFile(path.join(workspace, "base.txt"), "base\n");
|
||||
@@ -769,8 +767,6 @@ test("sessions.create honors worktree name/base ref and persists worktree info",
|
||||
"HEAD",
|
||||
]);
|
||||
await execFileAsync("git", ["-C", workspace, "checkout", "main"]);
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
process.env.OPENCLAW_STATE_DIR = path.join(root, "state");
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
testState.agentConfig = { workspace };
|
||||
await createSessionStoreDir();
|
||||
@@ -819,13 +815,8 @@ test("sessions.create honors worktree name/base ref and persists worktree info",
|
||||
await managedWorktrees.remove({ id: worktreeId, reason: "test-cleanup", force: true });
|
||||
}
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
testState.agentConfig = undefined;
|
||||
await fs.rm(root, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
|
||||
@@ -956,16 +947,16 @@ test("sessions.create rejects a Gateway worktree targeting a node", async () =>
|
||||
});
|
||||
|
||||
test("sessions.create provisions a worktree from an admin-selected cwd", async () => {
|
||||
const configuredRoot = await fs.mkdtemp(
|
||||
path.join(await fs.realpath(os.tmpdir()), "openclaw-configured-workspace-"),
|
||||
);
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-configured-workspace-",
|
||||
});
|
||||
const configuredRoot = openClawState.root;
|
||||
const selectedRoot = await fs.mkdtemp(
|
||||
path.join(await fs.realpath(os.tmpdir()), "openclaw-selected-workspace-"),
|
||||
);
|
||||
const configuredWorkspace = await initializeGitWorkspace(configuredRoot);
|
||||
const selectedWorkspace = await initializeGitWorkspace(selectedRoot);
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
process.env.OPENCLAW_STATE_DIR = path.join(configuredRoot, "state");
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
testState.agentConfig = { workspace: configuredWorkspace };
|
||||
await createSessionStoreDir();
|
||||
@@ -1011,13 +1002,8 @@ test("sessions.create provisions a worktree from an admin-selected cwd", async (
|
||||
await managedWorktrees.remove({ id: worktreeId, reason: "test-cleanup", force: true });
|
||||
}
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
testState.agentConfig = undefined;
|
||||
await fs.rm(configuredRoot, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
await fs.rm(selectedRoot, { recursive: true, force: true });
|
||||
}
|
||||
});
|
||||
@@ -1085,16 +1071,16 @@ test("sessions.create allows cwd within a sandboxed agent workspace", async () =
|
||||
});
|
||||
|
||||
test("sessions.create skips the worktree setup script for non-admin callers", async () => {
|
||||
const root = await fs.mkdtemp(
|
||||
path.join(await fs.realpath(os.tmpdir()), "openclaw-worktree-setup-scope-"),
|
||||
);
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-worktree-setup-scope-",
|
||||
});
|
||||
const root = openClawState.root;
|
||||
const workspace = await initializeGitWorkspace(root);
|
||||
await fs.mkdir(path.join(workspace, ".openclaw"), { recursive: true });
|
||||
const setupScript = path.join(workspace, ".openclaw", "worktree-setup.sh");
|
||||
await fs.writeFile(setupScript, "#!/bin/sh\ntouch setup-marker.txt\n");
|
||||
await fs.chmod(setupScript, 0o755);
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
process.env.OPENCLAW_STATE_DIR = path.join(root, "state");
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
testState.agentConfig = { workspace };
|
||||
await createSessionStoreDir();
|
||||
@@ -1118,27 +1104,22 @@ test("sessions.create skips the worktree setup script for non-admin callers", as
|
||||
await managedWorktrees.remove({ id: worktreeId, reason: "test-cleanup", force: true });
|
||||
}
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
testState.agentConfig = undefined;
|
||||
await fs.rm(root, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
|
||||
test("sessions.create preserves a linked-worktree subdirectory", async () => {
|
||||
const root = await fs.mkdtemp(
|
||||
path.join(await fs.realpath(os.tmpdir()), "openclaw-subdir-session-worktree-"),
|
||||
);
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-subdir-session-worktree-",
|
||||
});
|
||||
const root = openClawState.root;
|
||||
const repoRoot = await initializeGitWorkspace(root);
|
||||
const linkedRoot = path.join(root, "linked");
|
||||
await execFileAsync("git", ["-C", repoRoot, "worktree", "add", "-b", "linked", linkedRoot]);
|
||||
const workspace = path.join(linkedRoot, "packages", "app");
|
||||
await fs.mkdir(workspace, { recursive: true });
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
process.env.OPENCLAW_STATE_DIR = path.join(root, "state");
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
testState.agentConfig = { workspace };
|
||||
await createSessionStoreDir();
|
||||
@@ -1167,20 +1148,17 @@ test("sessions.create preserves a linked-worktree subdirectory", async () => {
|
||||
await managedWorktrees.remove({ id: worktreeId, reason: "test-cleanup", force: true });
|
||||
}
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
testState.agentConfig = undefined;
|
||||
await fs.rm(root, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
|
||||
test("sessions.create reset-in-place persists the returned worktree cwd", async () => {
|
||||
const root = await fs.mkdtemp(
|
||||
path.join(await fs.realpath(os.tmpdir()), "openclaw-reset-session-worktree-"),
|
||||
);
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-reset-session-worktree-",
|
||||
});
|
||||
const root = openClawState.root;
|
||||
const workspace = await initializeGitWorkspace(root);
|
||||
// A remote makes the base commit reachable from `--remotes`, so leaving the worktree via a
|
||||
// plain New Chat is lossless and the reset can remove it (the real leave-worktree flow).
|
||||
@@ -1188,8 +1166,6 @@ test("sessions.create reset-in-place persists the returned worktree cwd", async
|
||||
await execFileAsync("git", ["init", "--bare", origin]);
|
||||
await execFileAsync("git", ["-C", workspace, "remote", "add", "origin", origin]);
|
||||
await execFileAsync("git", ["-C", workspace, "push", "-u", "origin", "main"]);
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
process.env.OPENCLAW_STATE_DIR = path.join(root, "state");
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
testState.agentConfig = { workspace, model: { primary: "openai/current-model" } };
|
||||
testState.sessionConfig = { dmScope: "main" };
|
||||
@@ -1257,14 +1233,9 @@ test("sessions.create reset-in-place persists the returned worktree cwd", async
|
||||
await managedWorktrees.remove({ id: worktreeId, reason: "test-cleanup", force: true });
|
||||
}
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
testState.agentConfig = undefined;
|
||||
testState.sessionConfig = undefined;
|
||||
await fs.rm(root, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
|
||||
|
||||
@@ -2,7 +2,6 @@
|
||||
// active-run cleanup, hooks, thread bindings, and browser/MCP cleanup.
|
||||
import { execFile } from "node:child_process";
|
||||
import fs from "node:fs/promises";
|
||||
import os from "node:os";
|
||||
import path from "node:path";
|
||||
import { promisify } from "node:util";
|
||||
import { afterEach, expect, test, vi } from "vitest";
|
||||
@@ -23,6 +22,7 @@ import {
|
||||
runExclusiveSessionLifecycleMutation,
|
||||
} from "../sessions/session-lifecycle-admission.js";
|
||||
import { closeOpenClawStateDatabaseForTest } from "../state/openclaw-state-db.js";
|
||||
import { createOpenClawTestState } from "../test-utils/openclaw-test-state.js";
|
||||
import { embeddedRunMock, rpcReq, testState, writeSessionStore } from "./test-helpers.js";
|
||||
import {
|
||||
setupGatewaySessionsTestHarness,
|
||||
@@ -119,12 +119,12 @@ function expectThreadBindingsUnbound(targetSessionKey: string) {
|
||||
}
|
||||
|
||||
test("sessions.delete snapshots and removes session worktrees", async () => {
|
||||
const root = await fs.mkdtemp(
|
||||
path.join(await fs.realpath(os.tmpdir()), "openclaw-delete-worktree-"),
|
||||
);
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-delete-worktree-",
|
||||
});
|
||||
const root = openClawState.root;
|
||||
const workspace = await initializeRemoteBackedGitWorkspace(root);
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
process.env.OPENCLAW_STATE_DIR = path.join(root, "state");
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
testState.agentConfig = { workspace };
|
||||
await createSessionStoreDir();
|
||||
@@ -194,13 +194,8 @@ test("sessions.delete snapshots and removes session worktrees", async () => {
|
||||
await managedWorktrees.remove({ id: dirtyWorktreeId, reason: "test-cleanup", force: true });
|
||||
}
|
||||
closeOpenClawStateDatabaseForTest();
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
testState.agentConfig = undefined;
|
||||
await fs.rm(root, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
|
||||
|
||||
@@ -1,7 +1,6 @@
|
||||
// Covers outbound delivery core: hooks, queue cleanup, durable capability
|
||||
// checks, adapter sends, transcript mirroring, and payload outcomes.
|
||||
import fsPromises from "node:fs/promises";
|
||||
import os from "node:os";
|
||||
import path from "node:path";
|
||||
import { expectDefined } from "@openclaw/normalization-core";
|
||||
import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
|
||||
@@ -32,6 +31,7 @@ import {
|
||||
createTestRegistry,
|
||||
} from "../../test-utils/channel-plugins.js";
|
||||
import { createInternalHookEventPayload } from "../../test-utils/internal-hook-event-payload.js";
|
||||
import { createOpenClawTestState } from "../../test-utils/openclaw-test-state.js";
|
||||
import {
|
||||
onInternalDiagnosticEvent,
|
||||
resetDiagnosticEventsForTest,
|
||||
@@ -4262,10 +4262,10 @@ describe("deliverOutboundPayloads", () => {
|
||||
const sourceDir = await fsPromises.realpath(
|
||||
await fsPromises.mkdtemp(path.join(resolvePreferredOpenClawTmpDir(), "deliver-spool-")),
|
||||
);
|
||||
const stateDir = await fsPromises.realpath(
|
||||
await fsPromises.mkdtemp(path.join(os.tmpdir(), "openclaw-deliver-spool-state-")),
|
||||
);
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-deliver-spool-state-",
|
||||
});
|
||||
// Real MPEG-1 Layer III frames: host-local media sends are buffer-verified,
|
||||
// so placeholder text would be rejected before staging is even exercised.
|
||||
const source = path.join(sourceDir, "voice.mp3");
|
||||
@@ -4281,7 +4281,6 @@ describe("deliverOutboundPayloads", () => {
|
||||
const payload = { mediaUrl: source, audioAsVoice: true };
|
||||
|
||||
try {
|
||||
process.env.OPENCLAW_STATE_DIR = stateDir;
|
||||
await deliverOutboundPayloads({
|
||||
cfg: matrixChunkConfig,
|
||||
channel: "matrix",
|
||||
@@ -4303,13 +4302,8 @@ describe("deliverOutboundPayloads", () => {
|
||||
expect(payload.mediaUrl).toBe(source);
|
||||
expect(sendMatrix.mock.calls[0]?.[0]?.mediaUrl ?? source).toBe(source);
|
||||
} finally {
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
await fsPromises.rm(sourceDir, { recursive: true, force: true });
|
||||
await fsPromises.rm(stateDir, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
|
||||
@@ -4330,10 +4324,11 @@ describe("deliverOutboundPayloads", () => {
|
||||
// Deliberately outside the OpenClaw temp root: that root is itself a default
|
||||
// media root, so a state dir inside it would admit the source by containment
|
||||
// and hide whether the agent-scoped capability is what grants access.
|
||||
const stateDir = await fsPromises.realpath(
|
||||
await fsPromises.mkdtemp(path.join(os.tmpdir(), "openclaw-deliver-ws-")),
|
||||
);
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-deliver-ws-",
|
||||
});
|
||||
const stateDir = openClawState.stateDir;
|
||||
const workspaceDir = path.join(stateDir, "workspace-proofagent");
|
||||
// Host-local sends are buffer-verified, so the fixture needs real audio.
|
||||
const source = path.join(workspaceDir, "voice.mp3");
|
||||
@@ -4348,7 +4343,6 @@ describe("deliverOutboundPayloads", () => {
|
||||
const payload = { mediaUrl: source, audioAsVoice: true };
|
||||
|
||||
try {
|
||||
process.env.OPENCLAW_STATE_DIR = stateDir;
|
||||
await fsPromises.mkdir(workspaceDir, { recursive: true });
|
||||
await fsPromises.writeFile(source, mp3);
|
||||
await deliverOutboundPayloads({
|
||||
@@ -4375,12 +4369,7 @@ describe("deliverOutboundPayloads", () => {
|
||||
expect(payload.mediaUrl).toBe(source);
|
||||
expect(sendMatrix).toHaveBeenCalled();
|
||||
} finally {
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
await fsPromises.rm(stateDir, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
|
||||
|
||||
@@ -6,6 +6,7 @@ import path from "node:path";
|
||||
import { beforeEach, describe, expect, it, vi } from "vitest";
|
||||
import type { OpenClawConfig } from "../../config/config.js";
|
||||
import { MEDIA_MAX_BYTES } from "../../media/store.js";
|
||||
import { withOpenClawTestState } from "../../test-utils/openclaw-test-state.js";
|
||||
|
||||
const { resolveChannelMessageToolMediaSourceParamKeysMock } = vi.hoisted(() => ({
|
||||
resolveChannelMessageToolMediaSourceParamKeysMock: vi.fn(() => ["avatarPath", "avatarUrl"]),
|
||||
@@ -29,19 +30,10 @@ const maybeIt = process.platform === "win32" ? it.skip : it;
|
||||
const matrixMediaSourceParamKeys = ["avatarPath", "avatarUrl"] as const;
|
||||
|
||||
async function withTempOpenClawStateDir<T>(test: (stateDir: string) => Promise<T>): Promise<T> {
|
||||
const previous = process.env.OPENCLAW_STATE_DIR;
|
||||
const stateDir = await fs.mkdtemp(path.join(os.tmpdir(), "msg-params-state-"));
|
||||
process.env.OPENCLAW_STATE_DIR = stateDir;
|
||||
try {
|
||||
return await test(stateDir);
|
||||
} finally {
|
||||
if (previous === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previous;
|
||||
}
|
||||
await fs.rm(stateDir, { recursive: true, force: true });
|
||||
}
|
||||
return await withOpenClawTestState(
|
||||
{ layout: "state-only", prefix: "msg-params-state-" },
|
||||
(state) => test(state.stateDir),
|
||||
);
|
||||
}
|
||||
|
||||
describe("message action media helpers", () => {
|
||||
|
||||
@@ -15,6 +15,7 @@ import {
|
||||
createChannelTestPluginBase,
|
||||
createTestRegistry,
|
||||
} from "../../test-utils/channel-plugins.js";
|
||||
import { withOpenClawTestState } from "../../test-utils/openclaw-test-state.js";
|
||||
import { resolvePreferredOpenClawTmpDir } from "../tmp-openclaw-dir.js";
|
||||
import { runMessageAction } from "./message-action-runner.js";
|
||||
|
||||
@@ -91,19 +92,10 @@ async function withSandbox(test: (sandboxDir: string) => Promise<void>) {
|
||||
}
|
||||
|
||||
async function withTempOpenClawStateDir<T>(test: (stateDir: string) => Promise<T>): Promise<T> {
|
||||
const previous = process.env.OPENCLAW_STATE_DIR;
|
||||
const stateDir = await fs.mkdtemp(path.join(os.tmpdir(), "msg-runner-state-"));
|
||||
process.env.OPENCLAW_STATE_DIR = stateDir;
|
||||
try {
|
||||
return await test(stateDir);
|
||||
} finally {
|
||||
if (previous === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previous;
|
||||
}
|
||||
await fs.rm(stateDir, { recursive: true, force: true });
|
||||
}
|
||||
return await withOpenClawTestState(
|
||||
{ layout: "state-only", prefix: "msg-runner-state-" },
|
||||
(state) => test(state.stateDir),
|
||||
);
|
||||
}
|
||||
|
||||
const runDrySend = (params: {
|
||||
|
||||
@@ -1,9 +1,9 @@
|
||||
// Stuck session recovery runtime tests cover recovery inspection and event output.
|
||||
import fs from "node:fs";
|
||||
import os from "node:os";
|
||||
import path from "node:path";
|
||||
import { beforeEach, describe, expect, it, vi } from "vitest";
|
||||
import { saveCronStore } from "../cron/store.js";
|
||||
import { createOpenClawTestState } from "../test-utils/openclaw-test-state.js";
|
||||
|
||||
const mocks = vi.hoisted(() => ({
|
||||
abortEmbeddedAgentRun: vi.fn(),
|
||||
@@ -289,10 +289,12 @@ describe("stuck session recovery", () => {
|
||||
});
|
||||
|
||||
it("logs stopped cron context when aborting an active embedded run", async () => {
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "openclaw-recovery-context-"));
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-recovery-context-",
|
||||
});
|
||||
const tempDir = openClawState.stateDir;
|
||||
try {
|
||||
process.env.OPENCLAW_STATE_DIR = tempDir;
|
||||
await saveCronStore(path.join(tempDir, "cron", "jobs.json"), {
|
||||
version: 1,
|
||||
jobs: [
|
||||
@@ -330,12 +332,7 @@ describe("stuck session recovery", () => {
|
||||
allowActiveAbort: true,
|
||||
});
|
||||
} finally {
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
fs.rmSync(tempDir, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
|
||||
expect(warnLogMessages()).toEqual([
|
||||
|
||||
@@ -6,6 +6,7 @@ import { afterEach, describe, expect, it, vi } from "vitest";
|
||||
import { root as createFsSafeRoot } from "../infra/fs-safe.js";
|
||||
import { resetPluginStateStoreForTests } from "../plugin-state/plugin-state-store.js";
|
||||
import { clearMemoryPluginState } from "../plugins/memory-state.test-fixtures.js";
|
||||
import { createOpenClawTestState } from "../test-utils/openclaw-test-state.js";
|
||||
import { listMemoryHostPublicArtifacts } from "./memory-host-core.js";
|
||||
import {
|
||||
memoryHostEventExportOwnerContent,
|
||||
@@ -103,9 +104,11 @@ describe("memory host event export recovery", () => {
|
||||
});
|
||||
|
||||
it("finishes an inode-owned empty event export after interruption", async () => {
|
||||
const fixtureRoot = await fs.mkdtemp(path.join(os.tmpdir(), "memory-host-inode-owner-"));
|
||||
const stateDir = path.join(fixtureRoot, "state");
|
||||
const workspaceDir = path.join(fixtureRoot, "workspace");
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "memory-host-inode-owner-",
|
||||
});
|
||||
const { stateDir, workspaceDir } = openClawState;
|
||||
const event = {
|
||||
type: "memory.recall.recorded" as const,
|
||||
timestamp: "2026-05-18T12:00:00.000Z",
|
||||
@@ -114,8 +117,6 @@ describe("memory host event export recovery", () => {
|
||||
results: [],
|
||||
};
|
||||
try {
|
||||
vi.stubEnv("OPENCLAW_STATE_DIR", stateDir);
|
||||
await fs.mkdir(workspaceDir);
|
||||
await appendMemoryHostEvent(workspaceDir, event);
|
||||
const stateHash = createHash("sha256")
|
||||
.update(await fs.realpath(stateDir))
|
||||
@@ -165,14 +166,16 @@ describe("memory host event export recovery", () => {
|
||||
});
|
||||
expect(owner.pendingContentSha256).toBeUndefined();
|
||||
} finally {
|
||||
await fs.rm(fixtureRoot, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
|
||||
it("does not claim an empty export after exclusive-create interruption", async () => {
|
||||
const fixtureRoot = await fs.mkdtemp(path.join(os.tmpdir(), "memory-host-empty-export-"));
|
||||
const stateDir = path.join(fixtureRoot, "state");
|
||||
const workspaceDir = path.join(fixtureRoot, "workspace");
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "memory-host-empty-export-",
|
||||
});
|
||||
const { stateDir, workspaceDir } = openClawState;
|
||||
const event = {
|
||||
type: "memory.recall.recorded" as const,
|
||||
timestamp: "2026-05-18T12:00:00.000Z",
|
||||
@@ -181,8 +184,6 @@ describe("memory host event export recovery", () => {
|
||||
results: [],
|
||||
};
|
||||
try {
|
||||
vi.stubEnv("OPENCLAW_STATE_DIR", stateDir);
|
||||
await fs.mkdir(workspaceDir);
|
||||
await appendMemoryHostEvent(workspaceDir, event);
|
||||
const stateHash = createHash("sha256")
|
||||
.update(await fs.realpath(stateDir))
|
||||
@@ -217,7 +218,7 @@ describe("memory host event export recovery", () => {
|
||||
expect(listed.some((artifact) => artifact.kind === "event-log")).toBe(false);
|
||||
await expect(fs.readFile(exportPath, "utf8")).resolves.toBe("");
|
||||
} finally {
|
||||
await fs.rm(fixtureRoot, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
});
|
||||
|
||||
@@ -1,5 +1,4 @@
|
||||
// Run context lifecycle contract tests cover plugin run context setup and cleanup.
|
||||
import fs from "node:fs/promises";
|
||||
import path from "node:path";
|
||||
import {
|
||||
createPluginRegistryFixture,
|
||||
@@ -8,8 +7,8 @@ import {
|
||||
import { afterEach, describe, expect, it, vi } from "vitest";
|
||||
import { withTempConfig } from "../../gateway/test-temp-config.js";
|
||||
import { emitAgentEvent, resetAgentEventsForTest } from "../../infra/agent-events.js";
|
||||
import { resolvePreferredOpenClawTmpDir } from "../../infra/tmp-openclaw-dir.js";
|
||||
import { loadSessionStore, updateSessionStore } from "../../plugin-sdk/session-store-runtime.js";
|
||||
import { createOpenClawTestState } from "../../test-utils/openclaw-test-state.js";
|
||||
import { runPluginHostCleanup } from "../host-hook-cleanup.js";
|
||||
import {
|
||||
clearPluginHostRuntimeState,
|
||||
@@ -686,16 +685,16 @@ describe("plugin run context lifecycle", () => {
|
||||
},
|
||||
});
|
||||
|
||||
const stateDir = await fs.mkdtemp(
|
||||
path.join(resolvePreferredOpenClawTmpDir(), "openclaw-run-context-restart-state-"),
|
||||
);
|
||||
const openClawState = await createOpenClawTestState({
|
||||
layout: "state-only",
|
||||
prefix: "openclaw-run-context-restart-state-",
|
||||
});
|
||||
const stateDir = openClawState.stateDir;
|
||||
const storePath = path.join(stateDir, "sessions.json");
|
||||
const tempConfig = {
|
||||
session: { store: storePath },
|
||||
};
|
||||
const previousStateDir = process.env.OPENCLAW_STATE_DIR;
|
||||
try {
|
||||
process.env.OPENCLAW_STATE_DIR = stateDir;
|
||||
await withTempConfig({
|
||||
cfg: tempConfig,
|
||||
run: async () => {
|
||||
@@ -748,12 +747,7 @@ describe("plugin run context lifecycle", () => {
|
||||
},
|
||||
});
|
||||
} finally {
|
||||
if (previousStateDir === undefined) {
|
||||
delete process.env.OPENCLAW_STATE_DIR;
|
||||
} else {
|
||||
process.env.OPENCLAW_STATE_DIR = previousStateDir;
|
||||
}
|
||||
await fs.rm(stateDir, { recursive: true, force: true });
|
||||
await openClawState.cleanup();
|
||||
}
|
||||
});
|
||||
|
||||
|
||||
Reference in New Issue
Block a user