dilim/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-08-05 01:21:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: tighten slack outbound adapter assertions

Browse Source
This commit is contained in:
Shakker
2026-05-11 03:25:00 +01:00
parent b4b4503ff3
commit 7430475044
1 changed files with 47 additions and 56 deletions
Download Patch File Download Diff File Expand all files Collapse all files

103
extensions/slack/src/outbound-adapter.test.ts
View File

@@ -50,40 +50,35 @@ describe("slackOutbound", () => {
});
expect(sendMessageSlackMock).toHaveBeenCalledTimes(3);
expect(sendMessageSlackMock).toHaveBeenNthCalledWith(
1,
"C123",
"",
expect.objectContaining({
cfg,
mediaUrl: "https://example.com/1.png",
mediaLocalRoots: ["/tmp/workspace"],
}),
);
expect(sendMessageSlackMock).toHaveBeenNthCalledWith(
2,
"C123",
"",
expect.objectContaining({
cfg,
mediaUrl: "https://example.com/2.png",
mediaLocalRoots: ["/tmp/workspace"],
}),
);
expect(sendMessageSlackMock).toHaveBeenNthCalledWith(
3,
"C123",
"final text",
expect.objectContaining({
cfg,
blocks: [
{
type: "section",
text: { type: "mrkdwn", text: "Block body" },
},
],
}),
);
expect(sendMessageSlackMock).toHaveBeenNthCalledWith(1, "C123", "", {
cfg,
threadTs: undefined,
accountId: "default",
mediaUrl: "https://example.com/1.png",
mediaAccess: undefined,
mediaLocalRoots: ["/tmp/workspace"],
mediaReadFile: undefined,
});
expect(sendMessageSlackMock).toHaveBeenNthCalledWith(2, "C123", "", {
cfg,
threadTs: undefined,
accountId: "default",
mediaUrl: "https://example.com/2.png",
mediaAccess: undefined,
mediaLocalRoots: ["/tmp/workspace"],
mediaReadFile: undefined,
});
expect(sendMessageSlackMock).toHaveBeenNthCalledWith(3, "C123", "final text", {
cfg,
threadTs: undefined,
accountId: "default",
blocks: [
{
type: "section",
text: { type: "mrkdwn", text: "Block body" },
},
],
});
expect(result).toEqual({ channel: "slack", messageId: "m-final" });
});
@@ -105,14 +100,12 @@ describe("slackOutbound", () => {
accountId: "default",
});
expect(sendMessageSlackMock).toHaveBeenCalledWith(
"C123",
"fallback text",
expect.objectContaining({
cfg,
blocks: [{ type: "divider" }],
}),
);
expect(sendMessageSlackMock).toHaveBeenCalledWith("C123", "fallback text", {
cfg,
threadTs: undefined,
accountId: "default",
blocks: [{ type: "divider" }],
});
expect(result).toEqual({ channel: "slack", messageId: "m-blocks" });
});
@@ -136,13 +129,12 @@ describe("slackOutbound", () => {
accountId: "default",
});
expect(sendMessageSlackMock).toHaveBeenCalledWith(
"C123",
"fallback text",
expect.objectContaining({
threadTs: "1712345678.123456",
}),
);
expect(sendMessageSlackMock).toHaveBeenCalledWith("C123", "fallback text", {
cfg,
threadTs: "1712345678.123456",
accountId: "default",
blocks: [{ type: "divider" }],
});
});
it("does not thread payloads without a valid Slack thread timestamp", async () => {
@@ -165,12 +157,11 @@ describe("slackOutbound", () => {
accountId: "default",
});
expect(sendMessageSlackMock).toHaveBeenCalledWith(
"C123",
"fallback text",
expect.objectContaining({
threadTs: undefined,
}),
);
expect(sendMessageSlackMock).toHaveBeenCalledWith("C123", "fallback text", {
cfg,
threadTs: undefined,
accountId: "default",
blocks: [{ type: "divider" }],
});
});
});