LLM/moltbot
1
0
Fork 0
mirror of https://github.com/moltbot/moltbot.git synced 2026-05-13 23:56:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: tighten agents delete assertions

Browse Source
This commit is contained in:
Peter Steinberger
2026-05-10 01:34:27 +01:00
parent 0e66d6ae11
commit 77d608d02a
1 changed files with 28 additions and 36 deletions
Download Patch File Download Diff File Expand all files Collapse all files

64
src/commands/agents.delete.test.ts
View File

@@ -127,20 +127,17 @@ describe("agents delete command", () => {
await agentsDeleteCommand({ id: "ops", force: true, json: true }, runtime);
expect(gatewayMocks.callGateway).toHaveBeenCalledWith(
expect.objectContaining({
method: "agents.delete",
params: { agentId: "ops", deleteFiles: true },
requiredMethods: ["agents.delete"],
}),
);
expect(gatewayMocks.callGateway).toHaveBeenCalledOnce();
const gatewayCall = gatewayMocks.callGateway.mock.calls[0]?.[0];
expect(gatewayCall?.method).toBe("agents.delete");
expect(gatewayCall?.params).toEqual({ agentId: "ops", deleteFiles: true });
expect(gatewayCall?.requiredMethods).toEqual(["agents.delete"]);
expect(configMocks.replaceConfigFile).not.toHaveBeenCalled();
expectSessionStore(storePath, sessions);
expect(readJsonLogs()[0]).toMatchObject({
agentId: "ops",
removedBindings: 0,
transport: "gateway",
});
const output = readJsonLogs()[0];
expect(output?.agentId).toBe("ops");
expect(output?.removedBindings).toBe(0);
expect(output?.transport).toBe("gateway");
});
});
@@ -168,13 +165,13 @@ describe("agents delete command", () => {
await agentsDeleteCommand({ id: "ops", force: true, json: true }, runtime);
expect(runtime.exit).not.toHaveBeenCalled();
expect(configMocks.replaceConfigFile).toHaveBeenCalledWith(
expect.objectContaining({
nextConfig: {
agents: { list: [{ id: "main", workspace: path.join(stateDir, "workspace-main") }] },
},
}),
);
expect(configMocks.replaceConfigFile).toHaveBeenCalledOnce();
const replaceConfigFileCalls = configMocks.replaceConfigFile.mock.calls as unknown as Array<
[{ nextConfig: OpenClawConfig }]
>;
expect(replaceConfigFileCalls[0]?.[0].nextConfig).toEqual({
agents: { list: [{ id: "main", workspace: path.join(stateDir, "workspace-main") }] },
});
expectSessionStore(storePath, {
"agent:main:main": { sessionId: "sess-main", updatedAt: now + 3 },
});
@@ -282,11 +279,9 @@ describe("agents delete command", () => {
// The JSON output should report why the workspace was retained.
const jsonOutput = readJsonLogs();
expect(jsonOutput).toHaveLength(1);
expect(jsonOutput[0]).toMatchObject({
workspaceRetained: true,
workspaceRetainedReason: "shared",
workspaceSharedWith: ["main"],
});
expect(jsonOutput[0]?.workspaceRetained).toBe(true);
expect(jsonOutput[0]?.workspaceRetainedReason).toBe("shared");
expect(jsonOutput[0]?.workspaceSharedWith).toEqual(["main"]);
expect(processMocks.runCommandWithTimeout).not.toHaveBeenCalledWith(
["trash", sharedWorkspace],
{ timeoutMs: 5000 },
@@ -321,10 +316,9 @@ describe("agents delete command", () => {
await agentsDeleteCommand({ id: "ops", force: true, json: true }, runtime);
expect(readJsonLogs()[0]).toMatchObject({
workspaceRetained: true,
workspaceSharedWith: ["main"],
});
const output = readJsonLogs()[0];
expect(output?.workspaceRetained).toBe(true);
expect(output?.workspaceSharedWith).toEqual(["main"]);
expect(processMocks.runCommandWithTimeout).not.toHaveBeenCalledWith(
["trash", childWorkspace],
{ timeoutMs: 5000 },
@@ -359,10 +353,9 @@ describe("agents delete command", () => {
await agentsDeleteCommand({ id: "ops", force: true, json: true }, runtime);
expect(readJsonLogs()[0]).toMatchObject({
workspaceRetained: true,
workspaceSharedWith: ["main"],
});
const output = readJsonLogs()[0];
expect(output?.workspaceRetained).toBe(true);
expect(output?.workspaceSharedWith).toEqual(["main"]);
expect(processMocks.runCommandWithTimeout).not.toHaveBeenCalledWith(
["trash", sharedWorkspace],
{ timeoutMs: 5000 },
@@ -400,10 +393,9 @@ describe("agents delete command", () => {
await agentsDeleteCommand({ id: "ops", force: true, json: true }, runtime);
expect(readJsonLogs()[0]).toMatchObject({
workspaceRetained: true,
workspaceSharedWith: ["main"],
});
const output = readJsonLogs()[0];
expect(output?.workspaceRetained).toBe(true);
expect(output?.workspaceSharedWith).toEqual(["main"]);
expect(processMocks.runCommandWithTimeout).not.toHaveBeenCalledWith(
["trash", aliasWorkspace],
{ timeoutMs: 5000 },