From 3eb039c554c0ba7057be243a907d1951af0a794c Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Fri, 13 Mar 2026 22:26:24 +0000 Subject: [PATCH] test: dedupe discord forwarded media assertions --- src/discord/monitor/message-utils.test.ts | 27 +++++------------------ 1 file changed, 5 insertions(+), 22 deletions(-) diff --git a/src/discord/monitor/message-utils.test.ts b/src/discord/monitor/message-utils.test.ts index f4c5be256c1..acb9708ae21 100644 --- a/src/discord/monitor/message-utils.test.ts +++ b/src/discord/monitor/message-utils.test.ts @@ -173,30 +173,13 @@ describe("resolveForwardedMediaList", () => { 512, ); - expect(fetchRemoteMedia).toHaveBeenCalledTimes(1); - const call = fetchRemoteMedia.mock.calls[0]?.[0] as { - url?: string; - filePathHint?: string; - maxBytes?: number; - fetchImpl?: unknown; - ssrfPolicy?: unknown; - }; - expect(call).toMatchObject({ - url: attachment.url, + expectSinglePngDownload({ + result, + expectedUrl: attachment.url, filePathHint: attachment.filename, - maxBytes: 512, - fetchImpl: undefined, + expectedPath: "/tmp/image.png", + placeholder: "", }); - expectDiscordCdnSsrFPolicy(call.ssrfPolicy); - expect(saveMediaBuffer).toHaveBeenCalledTimes(1); - expect(saveMediaBuffer).toHaveBeenCalledWith(expect.any(Buffer), "image/png", "inbound", 512); - expect(result).toEqual([ - { - path: "/tmp/image.png", - contentType: "image/png", - placeholder: "", - }, - ]); }); it("forwards fetchImpl to forwarded attachment downloads", async () => {