From 1b91cdf459d8ca8d390da0c8aa9df1e09db17e5d Mon Sep 17 00:00:00 2001 From: Shakker Date: Fri, 8 May 2026 17:19:54 +0100 Subject: [PATCH] test: tighten discord acp preflight assertions --- .../message-handler.preflight.acp-bindings.test.ts | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/extensions/discord/src/monitor/message-handler.preflight.acp-bindings.test.ts b/extensions/discord/src/monitor/message-handler.preflight.acp-bindings.test.ts index 874e58f9d2a..00e817c00c4 100644 --- a/extensions/discord/src/monitor/message-handler.preflight.acp-bindings.test.ts +++ b/extensions/discord/src/monitor/message-handler.preflight.acp-bindings.test.ts @@ -278,10 +278,11 @@ describe("preflightDiscordMessage configured ACP bindings", () => { }), ); - expect(result).not.toBeNull(); expect(resolveConfiguredBindingRouteMock).toHaveBeenCalledTimes(1); expect(ensureConfiguredBindingRouteReadyMock).toHaveBeenCalledTimes(1); - expect(result?.boundSessionKey).toBe("agent:codex:acp:binding:discord:default:abc123"); + expect(result).toMatchObject({ + boundSessionKey: "agent:codex:acp:binding:discord:default:abc123", + }); }); it("accepts plain messages in configured ACP-bound channels without a mention", async () => { @@ -309,9 +310,10 @@ describe("preflightDiscordMessage configured ACP bindings", () => { }), ); - expect(result).not.toBeNull(); expect(ensureConfiguredBindingRouteReadyMock).toHaveBeenCalledTimes(1); - expect(result?.boundSessionKey).toBe("agent:codex:acp:binding:discord:default:abc123"); + expect(result).toMatchObject({ + boundSessionKey: "agent:codex:acp:binding:discord:default:abc123", + }); }); it("hydrates empty guild message payloads from REST before ensuring configured ACP bindings", async () => {