github/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-05-08 10:41:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor(test): share signal receive harness

Browse Source
This commit is contained in:
Peter Steinberger
2026-02-15 15:14:23 +00:00
parent 89dccc79a7
commit e2f73650d4
1 changed files with 139 additions and 214 deletions
Download Patch File Download Diff File Expand all files Collapse all files

219
src/signal/monitor.tool-result.sends-tool-summaries-responseprefix.test.ts
View File

@@ -25,12 +25,40 @@ const {
waitForTransportReadyMock, waitForTransportReadyMock,
} = getSignalToolResultTestMocks(); } = getSignalToolResultTestMocks();
const SIGNAL_BASE_URL = "http://127.0.0.1:8080";
async function runMonitorWithMocks( async function runMonitorWithMocks(
opts: Parameters<(typeof import("./monitor.js"))["monitorSignalProvider"]>[0], opts: Parameters<(typeof import("./monitor.js"))["monitorSignalProvider"]>[0],
) { ) {
const { monitorSignalProvider } = await import("./monitor.js"); const { monitorSignalProvider } = await import("./monitor.js");
return monitorSignalProvider(opts); return monitorSignalProvider(opts);
} }
async function receiveSignalPayloads(params: {
payloads: unknown[];
opts?: Partial<Parameters<(typeof import("./monitor.js"))["monitorSignalProvider"]>[0]>;
}) {
const abortController = new AbortController();
streamMock.mockImplementation(async ({ onEvent }) => {
for (const payload of params.payloads) {
await onEvent({
event: "receive",
data: JSON.stringify(payload),
});
}
abortController.abort();
});
await runMonitorWithMocks({
autoStart: false,
baseUrl: SIGNAL_BASE_URL,
abortSignal: abortController.signal,
...params.opts,
});
await flush();
}
describe("monitorSignalProvider tool results", () => { describe("monitorSignalProvider tool results", () => {
it("uses bounded readiness checks when auto-starting the daemon", async () => { it("uses bounded readiness checks when auto-starting the daemon", async () => {
const runtime = { const runtime = {
@@ -54,7 +82,7 @@ describe("monitorSignalProvider tool results", () => {
}); });
await runMonitorWithMocks({ await runMonitorWithMocks({
autoStart: true, autoStart: true,
baseUrl: "http://127.0.0.1:8080", baseUrl: SIGNAL_BASE_URL,
abortSignal: abortController.signal, abortSignal: abortController.signal,
runtime, runtime,
}); });
@@ -101,7 +129,7 @@ describe("monitorSignalProvider tool results", () => {
await runMonitorWithMocks({ await runMonitorWithMocks({
autoStart: true, autoStart: true,
baseUrl: "http://127.0.0.1:8080", baseUrl: SIGNAL_BASE_URL,
abortSignal: abortController.signal, abortSignal: abortController.signal,
runtime, runtime,
startupTimeoutMs: 90_000, startupTimeoutMs: 90_000,
@@ -143,7 +171,7 @@ describe("monitorSignalProvider tool results", () => {
await runMonitorWithMocks({ await runMonitorWithMocks({
autoStart: true, autoStart: true,
baseUrl: "http://127.0.0.1:8080", baseUrl: SIGNAL_BASE_URL,
abortSignal: abortController.signal, abortSignal: abortController.signal,
runtime, runtime,
}); });
@@ -157,11 +185,11 @@ describe("monitorSignalProvider tool results", () => {
}); });
it("skips tool summaries with responsePrefix", async () => { it("skips tool summaries with responsePrefix", async () => {
const abortController = new AbortController();
replyMock.mockResolvedValue({ text: "final reply" }); replyMock.mockResolvedValue({ text: "final reply" });
streamMock.mockImplementation(async ({ onEvent }) => { await receiveSignalPayloads({
const payload = { payloads: [
{
envelope: { envelope: {
sourceNumber: "+15550001111", sourceNumber: "+15550001111",
sourceName: "Ada", sourceName: "Ada",
@@ -170,21 +198,9 @@ describe("monitorSignalProvider tool results", () => {
message: "hello", message: "hello",
}, },
}, },
}; },
await onEvent({ ],
event: "receive",
data: JSON.stringify(payload),
}); });
abortController.abort();
});
await runMonitorWithMocks({
autoStart: false,
baseUrl: "http://127.0.0.1:8080",
abortSignal: abortController.signal,
});
await flush();
expect(sendMock).toHaveBeenCalledTimes(1); expect(sendMock).toHaveBeenCalledTimes(1);
expect(sendMock.mock.calls[0][1]).toBe("PFX final reply"); expect(sendMock.mock.calls[0][1]).toBe("PFX final reply");
@@ -203,10 +219,9 @@ describe("monitorSignalProvider tool results", () => {
}, },
}, },
}); });
const abortController = new AbortController(); await receiveSignalPayloads({
payloads: [
streamMock.mockImplementation(async ({ onEvent }) => { {
const payload = {
envelope: { envelope: {
sourceNumber: "+15550001111", sourceNumber: "+15550001111",
sourceName: "Ada", sourceName: "Ada",
@@ -215,21 +230,9 @@ describe("monitorSignalProvider tool results", () => {
message: "hello", message: "hello",
}, },
}, },
}; },
await onEvent({ ],
event: "receive",
data: JSON.stringify(payload),
}); });
abortController.abort();
});
await runMonitorWithMocks({
autoStart: false,
baseUrl: "http://127.0.0.1:8080",
abortSignal: abortController.signal,
});
await flush();
expect(replyMock).not.toHaveBeenCalled(); expect(replyMock).not.toHaveBeenCalled();
expect(upsertPairingRequestMock).toHaveBeenCalled(); expect(upsertPairingRequestMock).toHaveBeenCalled();
@@ -239,10 +242,9 @@ describe("monitorSignalProvider tool results", () => {
}); });
it("ignores reaction-only messages", async () => { it("ignores reaction-only messages", async () => {
const abortController = new AbortController(); await receiveSignalPayloads({
payloads: [
streamMock.mockImplementation(async ({ onEvent }) => { {
const payload = {
envelope: { envelope: {
sourceNumber: "+15550001111", sourceNumber: "+15550001111",
sourceName: "Ada", sourceName: "Ada",
@@ -253,21 +255,9 @@ describe("monitorSignalProvider tool results", () => {
targetSentTimestamp: 2, targetSentTimestamp: 2,
}, },
}, },
}; },
await onEvent({ ],
event: "receive",
data: JSON.stringify(payload),
}); });
abortController.abort();
});
await runMonitorWithMocks({
autoStart: false,
baseUrl: "http://127.0.0.1:8080",
abortSignal: abortController.signal,
});
await flush();
expect(replyMock).not.toHaveBeenCalled(); expect(replyMock).not.toHaveBeenCalled();
expect(sendMock).not.toHaveBeenCalled(); expect(sendMock).not.toHaveBeenCalled();
@@ -275,10 +265,9 @@ describe("monitorSignalProvider tool results", () => {
}); });
it("ignores reaction-only dataMessage.reaction events (don’t treat as broken attachments)", async () => { it("ignores reaction-only dataMessage.reaction events (don’t treat as broken attachments)", async () => {
const abortController = new AbortController(); await receiveSignalPayloads({
payloads: [
streamMock.mockImplementation(async ({ onEvent }) => { {
const payload = {
envelope: { envelope: {
sourceNumber: "+15550001111", sourceNumber: "+15550001111",
sourceName: "Ada", sourceName: "Ada",
@@ -292,21 +281,9 @@ describe("monitorSignalProvider tool results", () => {
attachments: [{}], attachments: [{}],
}, },
}, },
}; },
await onEvent({ ],
event: "receive",
data: JSON.stringify(payload),
}); });
abortController.abort();
});
await runMonitorWithMocks({
autoStart: false,
baseUrl: "http://127.0.0.1:8080",
abortSignal: abortController.signal,
});
await flush();
expect(replyMock).not.toHaveBeenCalled(); expect(replyMock).not.toHaveBeenCalled();
expect(sendMock).not.toHaveBeenCalled(); expect(sendMock).not.toHaveBeenCalled();
@@ -327,10 +304,9 @@ describe("monitorSignalProvider tool results", () => {
}, },
}, },
}); });
const abortController = new AbortController(); await receiveSignalPayloads({
payloads: [
streamMock.mockImplementation(async ({ onEvent }) => { {
const payload = {
envelope: { envelope: {
sourceNumber: "+15550001111", sourceNumber: "+15550001111",
sourceName: "Ada", sourceName: "Ada",
@@ -341,21 +317,9 @@ describe("monitorSignalProvider tool results", () => {
targetSentTimestamp: 2, targetSentTimestamp: 2,
}, },
}, },
}; },
await onEvent({ ],
event: "receive",
data: JSON.stringify(payload),
}); });
abortController.abort();
});
await runMonitorWithMocks({
autoStart: false,
baseUrl: "http://127.0.0.1:8080",
abortSignal: abortController.signal,
});
await flush();
const route = resolveAgentRoute({ const route = resolveAgentRoute({
cfg: config as OpenClawConfig, cfg: config as OpenClawConfig,
@@ -382,10 +346,9 @@ describe("monitorSignalProvider tool results", () => {
}, },
}, },
}); });
const abortController = new AbortController(); await receiveSignalPayloads({
payloads: [
streamMock.mockImplementation(async ({ onEvent }) => { {
const payload = {
envelope: { envelope: {
sourceNumber: "+15550001111", sourceNumber: "+15550001111",
sourceName: "Ada", sourceName: "Ada",
@@ -397,21 +360,9 @@ describe("monitorSignalProvider tool results", () => {
targetSentTimestamp: 2, targetSentTimestamp: 2,
}, },
}, },
}; },
await onEvent({ ],
event: "receive",
data: JSON.stringify(payload),
}); });
abortController.abort();
});
await runMonitorWithMocks({
autoStart: false,
baseUrl: "http://127.0.0.1:8080",
abortSignal: abortController.signal,
});
await flush();
const route = resolveAgentRoute({ const route = resolveAgentRoute({
cfg: config as OpenClawConfig, cfg: config as OpenClawConfig,
@@ -424,11 +375,11 @@ describe("monitorSignalProvider tool results", () => {
}); });
it("processes messages when reaction metadata is present", async () => { it("processes messages when reaction metadata is present", async () => {
const abortController = new AbortController();
replyMock.mockResolvedValue({ text: "pong" }); replyMock.mockResolvedValue({ text: "pong" });
streamMock.mockImplementation(async ({ onEvent }) => { await receiveSignalPayloads({
const payload = { payloads: [
{
envelope: { envelope: {
sourceNumber: "+15550001111", sourceNumber: "+15550001111",
sourceName: "Ada", sourceName: "Ada",
@@ -442,21 +393,9 @@ describe("monitorSignalProvider tool results", () => {
message: "ping", message: "ping",
}, },
}, },
}; },
await onEvent({ ],
event: "receive",
data: JSON.stringify(payload),
}); });
abortController.abort();
});
await runMonitorWithMocks({
autoStart: false,
baseUrl: "http://127.0.0.1:8080",
abortSignal: abortController.signal,
});
await flush();
expect(sendMock).toHaveBeenCalledTimes(1); expect(sendMock).toHaveBeenCalledTimes(1);
expect(updateLastRouteMock).toHaveBeenCalled(); expect(updateLastRouteMock).toHaveBeenCalled();
@@ -475,12 +414,10 @@ describe("monitorSignalProvider tool results", () => {
}, },
}, },
}); });
const abortController = new AbortController();
upsertPairingRequestMock upsertPairingRequestMock
.mockResolvedValueOnce({ code: "PAIRCODE", created: true }) .mockResolvedValueOnce({ code: "PAIRCODE", created: true })
.mockResolvedValueOnce({ code: "PAIRCODE", created: false }); .mockResolvedValueOnce({ code: "PAIRCODE", created: false });
streamMock.mockImplementation(async ({ onEvent }) => {
const payload = { const payload = {
envelope: { envelope: {
sourceNumber: "+15550001111", sourceNumber: "+15550001111",
@@ -491,27 +428,15 @@ describe("monitorSignalProvider tool results", () => {
}, },
}, },
}; };
await onEvent({ await receiveSignalPayloads({
event: "receive", payloads: [
data: JSON.stringify(payload), payload,
}); {
await onEvent({
event: "receive",
data: JSON.stringify({
...payload, ...payload,
envelope: { ...payload.envelope, timestamp: 2 }, envelope: { ...payload.envelope, timestamp: 2 },
}), },
],
}); });
abortController.abort();
});
await runMonitorWithMocks({
autoStart: false,
baseUrl: "http://127.0.0.1:8080",
abortSignal: abortController.signal,
});
await flush();
expect(sendMock).toHaveBeenCalledTimes(1); expect(sendMock).toHaveBeenCalledTimes(1);
}); });