admin/Moltbot
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

test(cli): dedupe route assertions and cover missing-flag guards

Browse Source
This commit is contained in:
Peter Steinberger
2026-02-21 20:00:40 +00:00
parent adedacbfe1
commit 8581e6b52d
1 changed files with 58 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

72
src/cli/program/routes.test.ts
View File

@@ -2,22 +2,32 @@ import { describe, expect, it } from "vitest";
import { findRoutedCommand } from "./routes.js";
describe("program routes", () => {
it("matches status route and preserves plugin loading", () => {
const route = findRoutedCommand(["status"]);
function expectRoute(path: string[]) {
const route = findRoutedCommand(path);
expect(route).not.toBeNull();
return route;
}
async function expectRunFalse(path: string[], argv: string[]) {
const route = expectRoute(path);
await expect(route?.run(argv)).resolves.toBe(false);
}
it("matches status route and preserves plugin loading", () => {
const route = expectRoute(["status"]);
expect(route?.loadPlugins).toBe(true);
});
it("returns false when status timeout flag value is missing", async () => {
const route = findRoutedCommand(["status"]);
expect(route).not.toBeNull();
await expect(route?.run(["node", "openclaw", "status", "--timeout"])).resolves.toBe(false);
await expectRunFalse(["status"], ["node", "openclaw", "status", "--timeout"]);
});
it("returns false for sessions route when --store value is missing", async () => {
const route = findRoutedCommand(["sessions"]);
expect(route).not.toBeNull();
await expect(route?.run(["node", "openclaw", "sessions", "--store"])).resolves.toBe(false);
await expectRunFalse(["sessions"], ["node", "openclaw", "sessions", "--store"]);
});
it("returns false for sessions route when --active value is missing", async () => {
await expectRunFalse(["sessions"], ["node", "openclaw", "sessions", "--active"]);
});
it("does not match unknown routes", () => {
@@ -25,14 +35,48 @@ describe("program routes", () => {
});
it("returns false for config get route when path argument is missing", async () => {
const route = findRoutedCommand(["config", "get"]);
expect(route).not.toBeNull();
await expect(route?.run(["node", "openclaw", "config", "get", "--json"])).resolves.toBe(false);
await expectRunFalse(["config", "get"], ["node", "openclaw", "config", "get", "--json"]);
});
it("returns false for config unset route when path argument is missing", async () => {
const route = findRoutedCommand(["config", "unset"]);
expect(route).not.toBeNull();
await expect(route?.run(["node", "openclaw", "config", "unset"])).resolves.toBe(false);
await expectRunFalse(["config", "unset"], ["node", "openclaw", "config", "unset"]);
});
it("returns false for memory status route when --agent value is missing", async () => {
await expectRunFalse(["memory", "status"], ["node", "openclaw", "memory", "status", "--agent"]);
});
it("returns false for models list route when --provider value is missing", async () => {
await expectRunFalse(["models", "list"], ["node", "openclaw", "models", "list", "--provider"]);
});
it("returns false for models status route when probe flags are missing values", async () => {
await expectRunFalse(
["models", "status"],
["node", "openclaw", "models", "status", "--probe-provider"],
);
await expectRunFalse(
["models", "status"],
["node", "openclaw", "models", "status", "--probe-timeout"],
);
await expectRunFalse(
["models", "status"],
["node", "openclaw", "models", "status", "--probe-concurrency"],
);
await expectRunFalse(
["models", "status"],
["node", "openclaw", "models", "status", "--probe-max-tokens"],
);
await expectRunFalse(
["models", "status"],
["node", "openclaw", "models", "status", "--probe-provider", "openai", "--agent"],
);
});
it("returns false for models status route when --probe-profile has no value", async () => {
await expectRunFalse(
["models", "status"],
["node", "openclaw", "models", "status", "--probe-profile"],
);
});
});