diff --git a/src/node-host/invoke.sanitize-env.test.ts b/src/node-host/invoke.sanitize-env.test.ts index 27f092d0aae..589d6196029 100644 --- a/src/node-host/invoke.sanitize-env.test.ts +++ b/src/node-host/invoke.sanitize-env.test.ts @@ -1,5 +1,6 @@ import { describe, expect, it } from "vitest"; import { sanitizeEnv } from "./invoke.js"; +import { buildNodeInvokeResultParams } from "./runner.js"; describe("node-host sanitizeEnv", () => { it("ignores PATH overrides", () => { @@ -46,3 +47,34 @@ describe("node-host sanitizeEnv", () => { } }); }); + +describe("buildNodeInvokeResultParams", () => { + it("omits optional fields when null/undefined", () => { + const params = buildNodeInvokeResultParams( + { id: "invoke-1", nodeId: "node-1", command: "system.run" }, + { ok: true, payloadJSON: null, error: null }, + ); + + expect(params).toEqual({ id: "invoke-1", nodeId: "node-1", ok: true }); + expect("payloadJSON" in params).toBe(false); + expect("error" in params).toBe(false); + }); + + it("includes payloadJSON when provided", () => { + const params = buildNodeInvokeResultParams( + { id: "invoke-2", nodeId: "node-2", command: "system.run" }, + { ok: true, payloadJSON: '{"ok":true}' }, + ); + + expect(params.payloadJSON).toBe('{"ok":true}'); + }); + + it("includes payload when provided", () => { + const params = buildNodeInvokeResultParams( + { id: "invoke-3", nodeId: "node-3", command: "system.run" }, + { ok: false, payload: { reason: "bad" } }, + ); + + expect(params.payload).toEqual({ reason: "bad" }); + }); +}); diff --git a/src/node-host/runner.test.ts b/src/node-host/runner.test.ts deleted file mode 100644 index 932f811ed44..00000000000 --- a/src/node-host/runner.test.ts +++ /dev/null @@ -1,33 +0,0 @@ -import { describe, expect, test } from "vitest"; -import { buildNodeInvokeResultParams } from "./runner.js"; - -describe("buildNodeInvokeResultParams", () => { - test("omits optional fields when null/undefined", () => { - const params = buildNodeInvokeResultParams( - { id: "invoke-1", nodeId: "node-1", command: "system.run" }, - { ok: true, payloadJSON: null, error: null }, - ); - - expect(params).toEqual({ id: "invoke-1", nodeId: "node-1", ok: true }); - expect("payloadJSON" in params).toBe(false); - expect("error" in params).toBe(false); - }); - - test("includes payloadJSON when provided", () => { - const params = buildNodeInvokeResultParams( - { id: "invoke-2", nodeId: "node-2", command: "system.run" }, - { ok: true, payloadJSON: '{"ok":true}' }, - ); - - expect(params.payloadJSON).toBe('{"ok":true}'); - }); - - test("includes payload when provided", () => { - const params = buildNodeInvokeResultParams( - { id: "invoke-3", nodeId: "node-3", command: "system.run" }, - { ok: false, payload: { reason: "bad" } }, - ); - - expect(params.payload).toEqual({ reason: "bad" }); - }); -});