惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
U
Unit 42
GbyAI
GbyAI
M
MIT News - Artificial intelligence
美团技术团队
罗磊的独立博客
雷峰网
雷峰网
量子位
博客园 - 【当耐特】
Last Week in AI
Last Week in AI
D
Docker
小众软件
小众软件
S
SegmentFault 最新的问题
Blog — PlanetScale
Blog — PlanetScale
阮一峰的网络日志
阮一峰的网络日志
宝玉的分享
宝玉的分享
T
Tailwind CSS Blog
WordPress大学
WordPress大学
V
V2EX
博客园_首页
腾讯CDC
The Cloudflare Blog
A
About on SuperTechFans
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
test: dedupe telegram delivery mock calls · openclaw/open...
steipete · 2026-05-12 · via Recent Commits to openclaw:main

@@ -121,6 +121,16 @@ function mockCallArg(mock: ReturnType<typeof vi.fn>, callIndex: number, argIndex

121121

return call[argIndex];

122122

}

123123124+

function firstMockCallArg(mock: ReturnType<typeof vi.fn>, argIndex: number) {

125+

return mockCallArg(mock, 0, argIndex);

126+

}

127+128+

function firstSendText(mock: ReturnType<typeof vi.fn>) {

129+

const text = firstMockCallArg(mock, 1);

130+

expect(text).toBeTypeOf("string");

131+

return text as string;

132+

}

133+124134

function createSendMessageHarness(messageId = 4) {

125135

const runtime = createRuntime();

126136

const sendMessage = vi.fn().mockResolvedValue({

@@ -229,7 +239,7 @@ describe("deliverReplies", () => {

229239230240

expect(runtime.error).toHaveBeenCalledTimes(1);

231241

expect(sendMessage).toHaveBeenCalledTimes(1);

232-

expect(sendMessage.mock.calls.at(0)?.[1]).toBe("hello");

242+

expect(firstMockCallArg(sendMessage, 1)).toBe("hello");

233243

});

234244235245

it("mirrors delivered replies once after successful sends", async () => {

@@ -276,8 +286,8 @@ describe("deliverReplies", () => {

276286

bot,

277287

});

278288279-

expect(sendMessage.mock.calls.at(0)?.[0]).toBe("123");

280-

expect(sendMessage.mock.calls.at(0)?.[1]).toBe("Plugin bind approval required");

289+

expect(firstMockCallArg(sendMessage, 0)).toBe("123");

290+

expect(firstMockCallArg(sendMessage, 1)).toBe("Plugin bind approval required");

281291

expectRecordFields(mockCallArg(sendMessage, 0, 2), {

282292

reply_markup: {

283293

inline_keyboard: [

@@ -309,8 +319,8 @@ describe("deliverReplies", () => {

309319

});

310320311321

expect(runtime.error).not.toHaveBeenCalled();

312-

expect(sendMessage.mock.calls.at(0)?.[0]).toBe("123");

313-

expect(sendMessage.mock.calls.at(0)?.[1]).toContain("Retry");

322+

expect(firstMockCallArg(sendMessage, 0)).toBe("123");

323+

expect(firstMockCallArg(sendMessage, 1)).toContain("Retry");

314324

expectRecordFields(mockCallArg(sendMessage, 0, 2), {

315325

reply_markup: {

316326

inline_keyboard: [[{ text: "Retry", callback_data: "cmd:retry" }]],

@@ -392,8 +402,8 @@ describe("deliverReplies", () => {

392402

silent: true,

393403

});

394404395-

expect(sendMessage.mock.calls.at(0)?.[0]).toBe("123");

396-

expect(typeof sendMessage.mock.calls.at(0)?.[1]).toBe("string");

405+

expect(firstMockCallArg(sendMessage, 0)).toBe("123");

406+

firstSendText(sendMessage);

397407

expectRecordFields(mockCallArg(sendMessage, 0, 2), { disable_notification: true });

398408

});

399409

@@ -455,9 +465,9 @@ describe("deliverReplies", () => {

455465

});

456466457467

expect(sendMessage).toHaveBeenCalledTimes(1);

458-

expect(sendMessage.mock.calls.at(0)?.[1]).toBeTypeOf("string");

459-

expect(sendMessage.mock.calls.at(0)?.[1]).not.toBe("");

460-

expect(sendMessage.mock.calls.at(0)?.[1]?.trim()).not.toBe("NO_REPLY");

468+

const text = firstSendText(sendMessage);

469+

expect(text).not.toBe("");

470+

expect(text.trim()).not.toBe("NO_REPLY");

461471

});

462472463473

it("uses the policy session key for exact NO_REPLY policy", async () => {

@@ -474,9 +484,9 @@ describe("deliverReplies", () => {

474484

});

475485476486

expect(sendMessage).toHaveBeenCalledTimes(1);

477-

expect(sendMessage.mock.calls.at(0)?.[1]).toBeTypeOf("string");

478-

expect(sendMessage.mock.calls.at(0)?.[1]).not.toBe("");

479-

expect(sendMessage.mock.calls.at(0)?.[1]?.trim()).not.toBe("NO_REPLY");

487+

const text = firstSendText(sendMessage);

488+

expect(text).not.toBe("");

489+

expect(text.trim()).not.toBe("NO_REPLY");

480490

});

481491482492

it("suppresses exact NO_REPLY for group Telegram sessions", async () => {

@@ -631,8 +641,8 @@ describe("deliverReplies", () => {

631641

bot,

632642

});

633643634-

expect(sendPhoto.mock.calls.at(0)?.[0]).toBe("123");

635-

if (sendPhoto.mock.calls.at(0)?.[1] === undefined) {

644+

expect(firstMockCallArg(sendPhoto, 0)).toBe("123");

645+

if (firstMockCallArg(sendPhoto, 1) === undefined) {

636646

throw new Error("Expected Telegram photo media");

637647

}

638648

expectRecordFields(mockCallArg(sendPhoto, 0, 2), {

@@ -659,8 +669,8 @@ describe("deliverReplies", () => {

659669

});

660670661671

expect(probeVideoDimensions).toHaveBeenCalledWith(Buffer.from("video"));

662-

expect(sendVideo.mock.calls.at(0)?.[0]).toBe("123");

663-

if (sendVideo.mock.calls.at(0)?.[1] === undefined) {

672+

expect(firstMockCallArg(sendVideo, 0)).toBe("123");

673+

if (firstMockCallArg(sendVideo, 1) === undefined) {

664674

throw new Error("Expected Telegram video media");

665675

}

666676

expectRecordFields(mockCallArg(sendVideo, 0, 2), {

@@ -688,8 +698,8 @@ describe("deliverReplies", () => {

688698

});

689699690700

expect(probeVideoDimensions).not.toHaveBeenCalled();

691-

expect(sendAnimation.mock.calls.at(0)?.[0]).toBe("123");

692-

if (sendAnimation.mock.calls.at(0)?.[1] === undefined) {

701+

expect(firstMockCallArg(sendAnimation, 0)).toBe("123");

702+

if (firstMockCallArg(sendAnimation, 1) === undefined) {

693703

throw new Error("Expected Telegram animation media");

694704

}

695705

const options = mockCallArg(sendAnimation, 0, 2) as Record<string, unknown>;

@@ -735,8 +745,8 @@ describe("deliverReplies", () => {

735745

linkPreview: false,

736746

});

737747738-

expect(sendMessage.mock.calls.at(0)?.[0]).toBe("123");

739-

expect(typeof sendMessage.mock.calls.at(0)?.[1]).toBe("string");

748+

expect(firstMockCallArg(sendMessage, 0)).toBe("123");

749+

firstSendText(sendMessage);

740750

expectRecordFields(mockCallArg(sendMessage, 0, 2), {

741751

link_preview_options: { is_disabled: true },

742752

});

@@ -752,8 +762,8 @@ describe("deliverReplies", () => {

752762

thread: { id: 42, scope: "dm" },

753763

});

754764755-

expect(sendMessage.mock.calls.at(0)?.[0]).toBe("123");

756-

expect(typeof sendMessage.mock.calls.at(0)?.[1]).toBe("string");

765+

expect(firstMockCallArg(sendMessage, 0)).toBe("123");

766+

firstSendText(sendMessage);

757767

expectRecordFields(mockCallArg(sendMessage, 0, 2), { message_thread_id: 42 });

758768

});

759769

@@ -867,8 +877,8 @@ describe("deliverReplies", () => {

867877

linkPreview: true,

868878

});

869879870-

expect(sendMessage.mock.calls.at(0)?.[0]).toBe("123");

871-

expect(typeof sendMessage.mock.calls.at(0)?.[1]).toBe("string");

880+

expect(firstMockCallArg(sendMessage, 0)).toBe("123");

881+

firstSendText(sendMessage);

872882

expect(mockCallArg(sendMessage, 0, 2)).not.toHaveProperty("link_preview_options");

873883

});

874884

@@ -897,8 +907,8 @@ describe("deliverReplies", () => {

897907

});

898908899909

expect(sendMessage).toHaveBeenCalledTimes(1);

900-

expect(sendMessage.mock.calls.at(0)?.[0]).toBe("123");

901-

expect(sendMessage.mock.calls.at(0)?.[1]).toBe(">");

910+

expect(firstMockCallArg(sendMessage, 0)).toBe("123");

911+

expect(firstMockCallArg(sendMessage, 1)).toBe(">");

902912

expectRecordFields(mockCallArg(sendMessage, 0, 2), { message_thread_id: 42 });

903913

});

904914

@@ -940,8 +950,8 @@ describe("deliverReplies", () => {

940950

replyQuoteEntities: [{ type: "bold", offset: 0, length: 6 }],

941951

});

942952943-

expect(sendMessage.mock.calls.at(0)?.[0]).toBe("123");

944-

expect(typeof sendMessage.mock.calls.at(0)?.[1]).toBe("string");

953+

expect(firstMockCallArg(sendMessage, 0)).toBe("123");

954+

firstSendText(sendMessage);

945955

expectRecordFields(mockCallArg(sendMessage, 0, 2), {

946956

reply_parameters: {

947957

message_id: 500,

@@ -1052,8 +1062,8 @@ describe("deliverReplies", () => {

10521062

replyQuoteText: "quoted text",

10531063

});

105410641055-

expect(sendMessage.mock.calls.at(0)?.[0]).toBe("123");

1056-

expect(typeof sendMessage.mock.calls.at(0)?.[1]).toBe("string");

1065+

expect(firstMockCallArg(sendMessage, 0)).toBe("123");

1066+

firstSendText(sendMessage);

10571067

expectRecordFields(mockCallArg(sendMessage, 0, 2), {

10581068

reply_to_message_id: 501,

10591069

allow_sending_without_reply: true,

@@ -1098,8 +1108,8 @@ describe("deliverReplies", () => {

10981108

replyQuoteText: "quoted text",

10991109

});

110011101101-

expect(sendMessage.mock.calls.at(0)?.[0]).toBe("123");

1102-

expect(typeof sendMessage.mock.calls.at(0)?.[1]).toBe("string");

1111+

expect(firstMockCallArg(sendMessage, 0)).toBe("123");

1112+

firstSendText(sendMessage);

11031113

expectRecordFields(mockCallArg(sendMessage, 0, 2), {

11041114

reply_to_message_id: 501,

11051115

allow_sending_without_reply: true,

@@ -1130,9 +1140,9 @@ describe("deliverReplies", () => {

11301140

expect(sendVoice).toHaveBeenCalledTimes(1);

11311141

// Fallback to text succeeded

11321142

expect(sendMessage).toHaveBeenCalledTimes(1);

1133-

expect(sendMessage.mock.calls.at(0)?.[0]).toBe("123");

1134-

expect(sendMessage.mock.calls.at(0)?.[1]).toContain("Hello there");

1135-

if (sendMessage.mock.calls.at(0)?.[2] === undefined) {

1143+

expect(firstMockCallArg(sendMessage, 0)).toBe("123");

1144+

expect(firstMockCallArg(sendMessage, 1)).toContain("Hello there");

1145+

if (firstMockCallArg(sendMessage, 2) === undefined) {

11361146

throw new Error("Expected Telegram fallback text options");

11371147

}

11381148

});

@@ -1158,8 +1168,8 @@ describe("deliverReplies", () => {

11581168

});

1159116911601170

expect(sendVoice).toHaveBeenCalledTimes(1);

1161-

expect(sendMessage.mock.calls.at(0)?.[0]).toBe("123");

1162-

expect(sendMessage.mock.calls.at(0)?.[1]).toContain("Hello there");

1171+

expect(firstMockCallArg(sendMessage, 0)).toBe("123");

1172+

expect(firstMockCallArg(sendMessage, 1)).toContain("Hello there");

11631173

expectRecordFields(mockCallArg(sendMessage, 0, 2), { disable_notification: true });

11641174

});

11651175