Ayuriel/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test: merge discord reaction id resolution cases

Browse Source
This commit is contained in:
Peter Steinberger 2026-03-17 09:45:14 +00:00
parent c4b866855a
commit 50c8569786
1 changed files with 50 additions and 33 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

83
src/channels/plugins/actions/actions.test.ts
View File

@ -516,41 +516,58 @@ describe("handleDiscordMessageAction", () => {
);
});
it("falls back to toolContext.currentMessageId for reactions when messageId is omitted", async () => {
await handleDiscordMessageAction({
action: "react",
params: {
channelId: "123",
emoji: "ok",
},
cfg: {} as OpenClawConfig,
toolContext: { currentMessageId: "9001" },
});
const call = handleDiscordAction.mock.calls.at(-1);
expect(call?.[0]).toEqual(
expect.objectContaining({
action: "react",
channelId: "123",
messageId: "9001",
emoji: "ok",
}),
);
});
it("rejects reactions when neither messageId nor toolContext.currentMessageId is provided", async () => {
await expect(
handleDiscordMessageAction({
action: "react",
params: {
channelId: "123",
emoji: "ok",
it("handles discord reaction messageId resolution", async () => {
const cases = [
{
name: "falls back to toolContext.currentMessageId",
run: async () => {
await handleDiscordMessageAction({
action: "react",
params: {
channelId: "123",
emoji: "ok",
},
cfg: {} as OpenClawConfig,
toolContext: { currentMessageId: "9001" },
});
},
cfg: {} as OpenClawConfig,
}),
).rejects.toThrow(/messageId required/i);
assert: () => {
const call = handleDiscordAction.mock.calls.at(-1);
expect(call?.[0]).toEqual(
expect.objectContaining({
action: "react",
channelId: "123",
messageId: "9001",
emoji: "ok",
}),
);
},
},
{
name: "rejects when no message id source is available",
run: async () => {
await expect(
handleDiscordMessageAction({
action: "react",
params: {
channelId: "123",
emoji: "ok",
},
cfg: {} as OpenClawConfig,
}),
).rejects.toThrow(/messageId required/i);
},
assert: () => {
expect(handleDiscordAction).not.toHaveBeenCalled();
},
},
] as const;
expect(handleDiscordAction).not.toHaveBeenCalled();
for (const testCase of cases) {
handleDiscordAction.mockClear();
await testCase.run();
testCase.assert();
}
});
});