vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-04-19 05:01:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: reuse memory temp fixtures

Browse Source
This commit is contained in:
Peter Steinberger
2026-04-06 19:27:51 +01:00
parent c185413a8e
commit b4ec7d77ce
3 changed files with 41 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files

46
extensions/memory-core/src/cli.test.ts
View File

@@ -2,7 +2,7 @@ import fs from "node:fs/promises";
import os from "node:os";
import path from "node:path";
import { Command } from "commander";
import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
import { afterAll, afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
import {
firstWrittenJsonArg,
spyRuntimeErrors,
@@ -54,11 +54,21 @@ let registerMemoryCli: typeof import("./cli.js").registerMemoryCli;
let defaultRuntime: typeof import("openclaw/plugin-sdk/memory-core-host-runtime-cli").defaultRuntime;
let isVerbose: typeof import("openclaw/plugin-sdk/memory-core-host-runtime-cli").isVerbose;
let setVerbose: typeof import("openclaw/plugin-sdk/memory-core-host-runtime-cli").setVerbose;
let fixtureRoot = "";
let workspaceFixtureRoot = "";
let qmdFixtureRoot = "";
let workspaceCaseId = 0;
let qmdCaseId = 0;
beforeAll(async () => {
({ registerMemoryCli } = await import("./cli.js"));
({ defaultRuntime, isVerbose, setVerbose } =
await import("openclaw/plugin-sdk/memory-core-host-runtime-cli"));
fixtureRoot = await fs.mkdtemp(path.join(os.tmpdir(), "memory-cli-fixtures-"));
workspaceFixtureRoot = path.join(fixtureRoot, "workspace");
qmdFixtureRoot = path.join(fixtureRoot, "qmd");
await fs.mkdir(workspaceFixtureRoot, { recursive: true });
await fs.mkdir(qmdFixtureRoot, { recursive: true });
});
beforeEach(() => {
@@ -77,6 +87,13 @@ afterEach(() => {
setVerbose(false);
});
afterAll(async () => {
if (!fixtureRoot) {
return;
}
await fs.rm(fixtureRoot, { recursive: true, force: true });
});
describe("memory cli", () => {
const inactiveMemorySecretDiagnostic = "agents.defaults.memorySearch.remote.apiKey inactive"; // pragma: allowlist secret
@@ -174,23 +191,17 @@ describe("memory cli", () => {
}
async function withQmdIndexDb(content: string, run: (dbPath: string) => Promise<void>) {
const tmpDir = await fs.mkdtemp(path.join(os.tmpdir(), "memory-cli-qmd-index-"));
const tmpDir = path.join(qmdFixtureRoot, `case-${qmdCaseId++}`);
await fs.mkdir(tmpDir, { recursive: true });
const dbPath = path.join(tmpDir, "index.sqlite");
try {
await fs.writeFile(dbPath, content, "utf-8");
await run(dbPath);
} finally {
await fs.rm(tmpDir, { recursive: true, force: true });
}
await fs.writeFile(dbPath, content, "utf-8");
await run(dbPath);
}
async function withTempWorkspace(run: (workspaceDir: string) => Promise<void>) {
const workspaceDir = await fs.mkdtemp(path.join(os.tmpdir(), "memory-cli-promote-"));
try {
await run(workspaceDir);
} finally {
await fs.rm(workspaceDir, { recursive: true, force: true });
}
const workspaceDir = path.join(workspaceFixtureRoot, `case-${workspaceCaseId++}`);
await fs.mkdir(workspaceDir, { recursive: true });
await run(workspaceDir);
}
async function writeDailyMemoryNote(
@@ -456,8 +467,7 @@ describe("memory cli", () => {
});
it("shows the fix hint only before --fix has been run", async () => {
const workspaceDir = await fs.mkdtemp(path.join(os.tmpdir(), "memory-cli-fix-hint-"));
try {
await withTempWorkspace(async (workspaceDir) => {
const storePath = path.join(workspaceDir, "memory", ".dreams", "short-term-recall.json");
await fs.mkdir(path.dirname(storePath), { recursive: true });
await fs.writeFile(storePath, " \n", "utf-8");
@@ -485,9 +495,7 @@ describe("memory cli", () => {
expect(log).not.toHaveBeenCalledWith(
expect.stringContaining("Fix: openclaw memory status --fix --agent main"),
);
} finally {
await fs.rm(workspaceDir, { recursive: true, force: true });
}
});
});
it("enables verbose logging with --verbose", async () => {

1
extensions/memory-core/src/short-term-promotion.test.ts
View File

@@ -33,7 +33,6 @@ describe("short-term promotion", () => {
async function withTempWorkspace(run: (workspaceDir: string) => Promise<void>) {
const workspaceDir = path.join(fixtureRoot, `case-${caseId++}`);
await fs.mkdir(workspaceDir, { recursive: true });
await run(workspaceDir);
}

22
extensions/memory-core/src/test-helpers.ts
View File

@@ -1,20 +1,26 @@
import fs from "node:fs/promises";
import os from "node:os";
import path from "node:path";
import { afterEach } from "vitest";
import { afterAll, beforeAll } from "vitest";
export function createMemoryCoreTestHarness() {
const tempDirs: string[] = [];
let fixtureRoot = "";
let caseId = 0;
afterEach(async () => {
await Promise.all(
tempDirs.splice(0).map((dir) => fs.rm(dir, { recursive: true, force: true })),
);
beforeAll(async () => {
fixtureRoot = await fs.mkdtemp(path.join(os.tmpdir(), "memory-core-test-fixtures-"));
});
afterAll(async () => {
if (!fixtureRoot) {
return;
}
await fs.rm(fixtureRoot, { recursive: true, force: true });
});
async function createTempWorkspace(prefix: string): Promise<string> {
const workspaceDir = await fs.mkdtemp(path.join(os.tmpdir(), prefix));
tempDirs.push(workspaceDir);
const workspaceDir = path.join(fixtureRoot, `${prefix}${caseId++}`);
await fs.mkdir(workspaceDir, { recursive: true });
return workspaceDir;
}