From e3b5e28cfff233aff3a9414a4e41ffd08e5bdd79 Mon Sep 17 00:00:00 2001 From: Shakker Date: Mon, 11 May 2026 08:52:53 +0100 Subject: [PATCH] test: tighten mattermost slash state assertions --- .../src/mattermost/slash-state.test.ts | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/extensions/mattermost/src/mattermost/slash-state.test.ts b/extensions/mattermost/src/mattermost/slash-state.test.ts index 26d4432d966..1fba9d70abf 100644 --- a/extensions/mattermost/src/mattermost/slash-state.test.ts +++ b/extensions/mattermost/src/mattermost/slash-state.test.ts @@ -58,7 +58,13 @@ describe("slash-state token routing", () => { }); const match = resolveSlashHandlerForToken("tok-a"); - expect(match).toMatchObject({ kind: "single", source: "token", accountIds: ["a1"] }); + expect(match.kind).toBe("single"); + if (match.kind !== "single") { + throw new Error("expected single match"); + } + expect(match.source).toBe("token"); + expect(match.accountIds).toEqual(["a1"]); + expect(typeof match.handler).toBe("function"); }); it("returns ambiguous when same token exists in multiple accounts", () => { @@ -99,7 +105,13 @@ describe("slash-state token routing", () => { command: "/oc_status", }); - expect(match).toMatchObject({ kind: "single", source: "command", accountIds: ["a1"] }); + expect(match.kind).toBe("single"); + if (match.kind !== "single") { + throw new Error("expected single match"); + } + expect(match.source).toBe("command"); + expect(match.accountIds).toEqual(["a1"]); + expect(typeof match.handler).toBe("function"); }); it("returns ambiguous when registered team and command match multiple accounts", () => {