From 54f2c8e9395615c8af552a4be971ee37b40f50be Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Wed, 1 Apr 2026 03:17:10 +0100 Subject: [PATCH] test: merge mattermost setup registration checks --- extensions/mattermost/src/setup.test.ts | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/extensions/mattermost/src/setup.test.ts b/extensions/mattermost/src/setup.test.ts index ea40ca89678..99e9c45a977 100644 --- a/extensions/mattermost/src/setup.test.ts +++ b/extensions/mattermost/src/setup.test.ts @@ -206,18 +206,18 @@ describe("mattermost setup", () => { }); }); - it("skips slash callback registration in setup-only mode", () => { + it.each([ + { name: "skips slash callback registration in setup-only mode", mode: "setup-only" as const }, + { name: "registers slash callback routes in full mode", mode: "full" as const }, + ])("$name", ({ mode }) => { const registerHttpRoute = vi.fn(); - plugin.register(createApi("setup-only", registerHttpRoute)); + plugin.register(createApi(mode, registerHttpRoute)); - expect(registerHttpRoute).not.toHaveBeenCalled(); - }); - - it("registers slash callback routes in full mode", () => { - const registerHttpRoute = vi.fn(); - - plugin.register(createApi("full", registerHttpRoute)); + if (mode === "setup-only") { + expect(registerHttpRoute).not.toHaveBeenCalled(); + return; + } expect(registerHttpRoute).toHaveBeenCalledTimes(1); expect(registerHttpRoute).toHaveBeenCalledWith(