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

chore: chore: Fix types in tests 12/N.

Browse Source
This commit is contained in:
cpojer
2026-02-17 11:17:45 +09:00
parent e02feaff83
commit 262b7a157a
7 changed files with 107 additions and 90 deletions
Download Patch File Download Diff File Expand all files Collapse all files

66
src/web/auto-reply.web-auto-reply.compresses-common-formats-jpeg-cap.e2e.test.ts
View File

@@ -8,11 +8,13 @@ import {
resetLoadConfigMock,
setLoadConfigMock,
} from "./auto-reply.test-harness.js";
import type { WebInboundMessage } from "./inbound.js";
installWebAutoReplyTestHomeHooks();
describe("web auto-reply", () => {
installWebAutoReplyUnitTestHooks({ pinDns: true });
type ListenerFactory = NonNullable<Parameters<typeof monitorWebChannel>[1]>;
async function setupSingleInboundMessage(params: {
resolverValue: { text: string; mediaUrl: string };
@@ -20,16 +22,12 @@ describe("web auto-reply", () => {
reply?: ReturnType<typeof vi.fn>;
}) {
const reply = params.reply ?? vi.fn().mockResolvedValue(undefined);
const sendComposing = vi.fn();
const sendComposing = vi.fn(async () => undefined);
const resolver = vi.fn().mockResolvedValue(params.resolverValue);
let capturedOnMessage:
| ((msg: import("./inbound.js").WebInboundMessage) => Promise<void>)
| undefined;
const listenerFactory = async (opts: {
onMessage: (msg: import("./inbound.js").WebInboundMessage) => Promise<void>;
}) => {
capturedOnMessage = opts.onMessage;
let capturedOnMessage: ((msg: WebInboundMessage) => Promise<void>) | undefined;
const listenerFactory: ListenerFactory = async ({ onMessage }) => {
capturedOnMessage = onMessage;
return { close: vi.fn() };
};
@@ -42,12 +40,16 @@ describe("web auto-reply", () => {
await capturedOnMessage?.({
body: "hello",
from: "+1",
conversationId: "+1",
to: "+2",
accountId: "default",
chatType: "direct",
chatId: "+1",
id,
sendComposing,
reply,
sendMedia: params.sendMedia,
});
} as WebInboundMessage);
},
};
}
@@ -104,19 +106,15 @@ describe("web auto-reply", () => {
setLoadConfigMock(() => ({ agents: { defaults: { mediaMaxMb: 1 } } }));
const sendMedia = vi.fn();
const reply = vi.fn().mockResolvedValue(undefined);
const sendComposing = vi.fn();
const sendComposing = vi.fn(async () => undefined);
const resolver = vi.fn().mockResolvedValue({
text: "hi",
mediaUrl: `https://example.com/big.${fmt.name}`,
});
let capturedOnMessage:
| ((msg: import("./inbound.js").WebInboundMessage) => Promise<void>)
| undefined;
const listenerFactory = async (opts: {
onMessage: (msg: import("./inbound.js").WebInboundMessage) => Promise<void>;
}) => {
capturedOnMessage = opts.onMessage;
let capturedOnMessage: ((msg: WebInboundMessage) => Promise<void>) | undefined;
const listenerFactory: ListenerFactory = async ({ onMessage }) => {
capturedOnMessage = onMessage;
return { close: vi.fn() };
};
@@ -129,7 +127,7 @@ describe("web auto-reply", () => {
arrayBuffer: async () => big.buffer.slice(big.byteOffset, big.byteOffset + big.byteLength),
headers: { get: () => fmt.mime },
status: 200,
} as Response);
} as unknown as Response);
await monitorWebChannel(false, listenerFactory, false, resolver);
expect(capturedOnMessage).toBeDefined();
@@ -137,12 +135,16 @@ describe("web auto-reply", () => {
await capturedOnMessage?.({
body: "hello",
from: "+1",
conversationId: "+1",
to: "+2",
accountId: "default",
chatType: "direct",
chatId: "+1",
id: `msg-${fmt.name}`,
sendComposing,
reply,
sendMedia,
});
} as WebInboundMessage);
expect(sendMedia).toHaveBeenCalledTimes(1);
const payload = sendMedia.mock.calls[0][0] as {
@@ -162,19 +164,15 @@ describe("web auto-reply", () => {
setLoadConfigMock(() => ({ agents: { defaults: { mediaMaxMb: 1 } } }));
const sendMedia = vi.fn();
const reply = vi.fn().mockResolvedValue(undefined);
const sendComposing = vi.fn();
const sendComposing = vi.fn(async () => undefined);
const resolver = vi.fn().mockResolvedValue({
text: "hi",
mediaUrl: "https://example.com/big.png",
});
let capturedOnMessage:
| ((msg: import("./inbound.js").WebInboundMessage) => Promise<void>)
| undefined;
const listenerFactory = async (opts: {
onMessage: (msg: import("./inbound.js").WebInboundMessage) => Promise<void>;
}) => {
capturedOnMessage = opts.onMessage;
let capturedOnMessage: ((msg: WebInboundMessage) => Promise<void>) | undefined;
const listenerFactory: ListenerFactory = async ({ onMessage }) => {
capturedOnMessage = onMessage;
return { close: vi.fn() };
};
@@ -197,7 +195,7 @@ describe("web auto-reply", () => {
bigPng.buffer.slice(bigPng.byteOffset, bigPng.byteOffset + bigPng.byteLength),
headers: { get: () => "image/png" },
status: 200,
} as Response);
} as unknown as Response);
await monitorWebChannel(false, listenerFactory, false, resolver);
expect(capturedOnMessage).toBeDefined();
@@ -205,12 +203,16 @@ describe("web auto-reply", () => {
await capturedOnMessage?.({
body: "hello",
from: "+1",
conversationId: "+1",
to: "+2",
accountId: "default",
chatType: "direct",
chatId: "+1",
id: "msg1",
sendComposing,
reply,
sendMedia,
});
} as WebInboundMessage);
expect(sendMedia).toHaveBeenCalledTimes(1);
const payload = sendMedia.mock.calls[0][0] as {
@@ -237,7 +239,7 @@ describe("web auto-reply", () => {
arrayBuffer: async () => Buffer.from("%PDF-1.4").buffer,
headers: { get: () => "application/pdf" },
status: 200,
} as Response);
} as unknown as Response);
await dispatch("msg-pdf");
@@ -282,7 +284,7 @@ describe("web auto-reply", () => {
smallPng.buffer.slice(smallPng.byteOffset, smallPng.byteOffset + smallPng.byteLength),
headers: { get: () => "image/png" },
status: 200,
} as Response);
} as unknown as Response);
await dispatch("msg1");
@@ -347,7 +349,7 @@ describe("web auto-reply", () => {
arrayBuffer: async () => png.buffer.slice(png.byteOffset, png.byteOffset + png.byteLength),
headers: { get: () => "image/png" },
status: 200,
} as Response);
} as unknown as Response);
await dispatch("msg1");

32
src/web/auto-reply/heartbeat-runner.test.ts
View File

@@ -1,5 +1,7 @@
import { beforeEach, describe, expect, it, vi } from "vitest";
import type { getReplyFromConfig } from "../../auto-reply/reply.js";
import { HEARTBEAT_TOKEN } from "../../auto-reply/tokens.js";
import type { sendMessageWhatsApp } from "../outbound.js";
const state = vi.hoisted(() => ({
visibility: { showAlerts: true, showOk: true, useIndicator: false },
@@ -87,8 +89,10 @@ vi.mock("../session.js", () => ({
}));
describe("runWebHeartbeatOnce", () => {
let sender: ReturnType<typeof vi.fn>;
let replyResolver: ReturnType<typeof vi.fn>;
let senderMock: ReturnType<typeof vi.fn>;
let sender: typeof sendMessageWhatsApp;
let replyResolverMock: ReturnType<typeof vi.fn>;
let replyResolver: typeof getReplyFromConfig;
const getModules = async () => await import("./heartbeat-runner.js");
@@ -105,8 +109,10 @@ describe("runWebHeartbeatOnce", () => {
};
state.events = [];
sender = vi.fn(async () => ({ messageId: "m1" }));
replyResolver = vi.fn(async () => undefined);
senderMock = vi.fn(async () => ({ messageId: "m1" }));
sender = senderMock as unknown as typeof sendMessageWhatsApp;
replyResolverMock = vi.fn(async () => undefined);
replyResolver = replyResolverMock as unknown as typeof getReplyFromConfig;
});
it("supports manual override body dry-run without sending", async () => {
@@ -119,7 +125,7 @@ describe("runWebHeartbeatOnce", () => {
overrideBody: "hello",
dryRun: true,
});
expect(sender).not.toHaveBeenCalled();
expect(senderMock).not.toHaveBeenCalled();
expect(state.events).toHaveLength(0);
});
@@ -131,7 +137,7 @@ describe("runWebHeartbeatOnce", () => {
sender,
replyResolver,
});
expect(sender).toHaveBeenCalledWith("+123", HEARTBEAT_TOKEN, { verbose: false });
expect(senderMock).toHaveBeenCalledWith("+123", HEARTBEAT_TOKEN, { verbose: false });
expect(state.events).toEqual(
expect.arrayContaining([expect.objectContaining({ status: "ok-empty", silent: false })]),
);
@@ -147,13 +153,13 @@ describe("runWebHeartbeatOnce", () => {
dryRun: true,
});
expect(replyResolver).toHaveBeenCalledTimes(1);
const ctx = replyResolver.mock.calls[0]?.[0];
const ctx = replyResolverMock.mock.calls[0]?.[0];
expect(ctx?.Body).toContain("Ops check");
expect(ctx?.Body).toContain("Current time: 2026-02-15T00:00:00Z (mock)");
});
it("treats heartbeat token-only replies as ok-token and preserves session updatedAt", async () => {
replyResolver.mockResolvedValue({ text: HEARTBEAT_TOKEN });
replyResolverMock.mockResolvedValue({ text: HEARTBEAT_TOKEN });
const { runWebHeartbeatOnce } = await getModules();
await runWebHeartbeatOnce({
cfg: { agents: { defaults: {} }, session: {} } as never,
@@ -162,7 +168,7 @@ describe("runWebHeartbeatOnce", () => {
replyResolver,
});
expect(state.store.k?.updatedAt).toBe(123);
expect(sender).toHaveBeenCalledWith("+123", HEARTBEAT_TOKEN, { verbose: false });
expect(senderMock).toHaveBeenCalledWith("+123", HEARTBEAT_TOKEN, { verbose: false });
expect(state.events).toEqual(
expect.arrayContaining([expect.objectContaining({ status: "ok-token", silent: false })]),
);
@@ -170,7 +176,7 @@ describe("runWebHeartbeatOnce", () => {
it("skips sending alerts when showAlerts is disabled but still emits a skipped event", async () => {
state.visibility = { showAlerts: false, showOk: true, useIndicator: true };
replyResolver.mockResolvedValue({ text: "ALERT" });
replyResolverMock.mockResolvedValue({ text: "ALERT" });
const { runWebHeartbeatOnce } = await getModules();
await runWebHeartbeatOnce({
cfg: { agents: { defaults: {} }, session: {} } as never,
@@ -178,7 +184,7 @@ describe("runWebHeartbeatOnce", () => {
sender,
replyResolver,
});
expect(sender).not.toHaveBeenCalled();
expect(senderMock).not.toHaveBeenCalled();
expect(state.events).toEqual(
expect.arrayContaining([
expect.objectContaining({ status: "skipped", reason: "alerts-disabled", preview: "ALERT" }),
@@ -187,8 +193,8 @@ describe("runWebHeartbeatOnce", () => {
});
it("emits failed events when sending throws and rethrows the error", async () => {
replyResolver.mockResolvedValue({ text: "ALERT" });
sender.mockRejectedValueOnce(new Error("nope"));
replyResolverMock.mockResolvedValue({ text: "ALERT" });
senderMock.mockRejectedValueOnce(new Error("nope"));
const { runWebHeartbeatOnce } = await getModules();
await expect(
runWebHeartbeatOnce({