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

test(perf): merge overlapping preaction hook scenarios

Browse Source
This commit is contained in:
Peter Steinberger
2026-03-02 14:52:30 +00:00
parent 9f98d2766a
commit 87bd6226bd
1 changed files with 7 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/cli/program/preaction.test.ts
View File

@@ -128,7 +128,7 @@ describe("registerPreActionHooks", () => {
await preActionHook(program, actionCommand);
}
it("emits banner, resolves config, and enables verbose from --debug", async () => {
it("handles debug mode and plugin-required command preaction", async () => {
await runPreAction({
parseArgv: ["status"],
processArgv: ["node", "openclaw", "status", "--debug"],
@@ -142,9 +142,8 @@ describe("registerPreActionHooks", () => {
});
expect(ensurePluginRegistryLoadedMock).not.toHaveBeenCalled();
expect(process.title).toBe("openclaw-status");
});
it("loads plugin registry for plugin-required commands", async () => {
vi.clearAllMocks();
await runPreAction({
parseArgv: ["message", "send"],
processArgv: ["node", "openclaw", "message", "send"],
@@ -159,7 +158,7 @@ describe("registerPreActionHooks", () => {
expect(ensurePluginRegistryLoadedMock).toHaveBeenCalledTimes(1);
});
it("skips preaction work when argv indicates help/version", async () => {
it("skips help/version preaction and respects banner opt-out", async () => {
await runPreAction({
parseArgv: ["status"],
processArgv: ["node", "openclaw", "--version"],
@@ -168,10 +167,10 @@ describe("registerPreActionHooks", () => {
expect(emitCliBannerMock).not.toHaveBeenCalled();
expect(setVerboseMock).not.toHaveBeenCalled();
expect(ensureConfigReadyMock).not.toHaveBeenCalled();
});
it("hides banner when OPENCLAW_HIDE_BANNER is truthy", async () => {
vi.clearAllMocks();
process.env.OPENCLAW_HIDE_BANNER = "1";
await runPreAction({
parseArgv: ["status"],
processArgv: ["node", "openclaw", "status"],
@@ -181,7 +180,7 @@ describe("registerPreActionHooks", () => {
expect(ensureConfigReadyMock).toHaveBeenCalledTimes(1);
});
it("suppresses doctor stdout for --json output command", async () => {
it("applies --json stdout suppression only for explicit JSON output commands", async () => {
await runPreAction({
parseArgv: ["update", "status", "--json"],
processArgv: ["node", "openclaw", "update", "status", "--json"],
@@ -192,9 +191,8 @@ describe("registerPreActionHooks", () => {
commandPath: ["update", "status"],
suppressDoctorStdout: true,
});
});
it("does not treat config set --json (strict-parse alias) as json output mode", async () => {
vi.clearAllMocks();
await runPreAction({
parseArgv: ["config", "set", "gateway.auth.mode", "{bad", "--json"],
processArgv: ["node", "openclaw", "config", "set", "gateway.auth.mode", "{bad", "--json"],