farcasclaudiu/openclaw
1
0
Fork 0
mirror of https://github.com/farcasclaudiu/openclaw.git synced 2026-06-29 15:01:48 +03:00
Code Issues Packages Projects Releases Wiki Activity

perf(test): reduce web logout fs churn

Browse Source
This commit is contained in:
Peter Steinberger
2026-02-14 15:41:26 +00:00
parent 2b5e0a6075
commit ee82c173ae
1 changed files with 42 additions and 39 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/web/logout.test.ts
+21 -18
View File
@@ -2,7 +2,7 @@ import fs from "node:fs";
import fsPromises from "node:fs/promises"; import fsPromises from "node:fs/promises";
import os from "node:os"; import os from "node:os";
import path from "node:path"; import path from "node:path";
import { afterEach, beforeEach, describe, expect, it, vi } from "vitest"; import { afterAll, afterEach, beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
const runtime = { const runtime = {
log: vi.fn(), log: vi.fn(),
@@ -10,16 +10,24 @@ const runtime = {
exit: vi.fn(), exit: vi.fn(),
}; };
async function withTempDir<T>(fn: (dir: string) => Promise<T>): Promise<T> {
const dir = await fsPromises.mkdtemp(path.join(os.tmpdir(), "openclaw-test-web-logout-"));
try {
return await fn(dir);
} finally {
await fsPromises.rm(dir, { recursive: true, force: true });
}
}
describe("web logout", () => { describe("web logout", () => {
let fixtureRoot = "";
let caseId = 0;
beforeAll(async () => {
fixtureRoot = await fsPromises.mkdtemp(path.join(os.tmpdir(), "openclaw-test-web-logout-"));
});
afterAll(async () => {
await fsPromises.rm(fixtureRoot, { recursive: true, force: true });
});
const makeCaseDir = async () => {
const dir = path.join(fixtureRoot, `case-${caseId++}`);
await fsPromises.mkdir(dir, { recursive: true });
return dir;
};
beforeEach(() => { beforeEach(() => {
vi.clearAllMocks(); vi.clearAllMocks();
}); });
@@ -29,29 +37,25 @@ describe("web logout", () => {
}); });
it("deletes cached credentials when present", { timeout: 60_000 }, async () => { it("deletes cached credentials when present", { timeout: 60_000 }, async () => {
await withTempDir(async (authDir) => { const authDir = await makeCaseDir();
const { logoutWeb } = await import("./session.js"); const { logoutWeb } = await import("./session.js");
fs.mkdirSync(authDir, { recursive: true });
fs.writeFileSync(path.join(authDir, "creds.json"), "{}"); fs.writeFileSync(path.join(authDir, "creds.json"), "{}");
const result = await logoutWeb({ authDir, runtime: runtime as never }); const result = await logoutWeb({ authDir, runtime: runtime as never });
expect(result).toBe(true); expect(result).toBe(true);
expect(fs.existsSync(authDir)).toBe(false); expect(fs.existsSync(authDir)).toBe(false);
}); });
});
it("no-ops when nothing to delete", { timeout: 60_000 }, async () => { it("no-ops when nothing to delete", { timeout: 60_000 }, async () => {
await withTempDir(async (authDir) => { const authDir = await makeCaseDir();
const { logoutWeb } = await import("./session.js"); const { logoutWeb } = await import("./session.js");
const result = await logoutWeb({ authDir, runtime: runtime as never }); const result = await logoutWeb({ authDir, runtime: runtime as never });
expect(result).toBe(false); expect(result).toBe(false);
expect(runtime.log).toHaveBeenCalled(); expect(runtime.log).toHaveBeenCalled();
}); });
});
it("keeps shared oauth.json when using legacy auth dir", async () => { it("keeps shared oauth.json when using legacy auth dir", async () => {
await withTempDir(async (credsDir) => { const credsDir = await makeCaseDir();
const { logoutWeb } = await import("./session.js"); const { logoutWeb } = await import("./session.js");
fs.mkdirSync(credsDir, { recursive: true });
fs.writeFileSync(path.join(credsDir, "creds.json"), "{}"); fs.writeFileSync(path.join(credsDir, "creds.json"), "{}");
fs.writeFileSync(path.join(credsDir, "oauth.json"), '{"token":true}'); fs.writeFileSync(path.join(credsDir, "oauth.json"), '{"token":true}');
fs.writeFileSync(path.join(credsDir, "session-abc.json"), "{}"); fs.writeFileSync(path.join(credsDir, "session-abc.json"), "{}");
@@ -67,4 +71,3 @@ describe("web logout", () => {
expect(fs.existsSync(path.join(credsDir, "session-abc.json"))).toBe(false); expect(fs.existsSync(path.join(credsDir, "session-abc.json"))).toBe(false);
}); });
}); });
});