From e521f5611676f06c62a3919033879ea4635f1e8f Mon Sep 17 00:00:00 2001 From: Shakker Date: Mon, 11 May 2026 15:30:01 +0100 Subject: [PATCH] test: check directive alias parsing --- .../reply/get-reply-directive-aliases.test.ts | 48 +++++++++---------- 1 file changed, 24 insertions(+), 24 deletions(-) diff --git a/src/auto-reply/reply/get-reply-directive-aliases.test.ts b/src/auto-reply/reply/get-reply-directive-aliases.test.ts index 054e8aeecae..9542063a1d2 100644 --- a/src/auto-reply/reply/get-reply-directive-aliases.test.ts +++ b/src/auto-reply/reply/get-reply-directive-aliases.test.ts @@ -23,15 +23,15 @@ describe("reply directive aliases", () => { const reservedCommands = new Set(); const cfg = configWithModelAlias("demo_skill"); - expect( - parseInlineDirectives("/demo_skill", { - modelAliases: resolveConfiguredDirectiveAliases({ - cfg, - commandTextHasSlash: true, - reservedCommands, - }), + const beforeSkillRegistration = parseInlineDirectives("/demo_skill", { + modelAliases: resolveConfiguredDirectiveAliases({ + cfg, + commandTextHasSlash: true, + reservedCommands, }), - ).toEqual(expect.objectContaining({ hasModelDirective: true, cleaned: "" })); + }); + expect(beforeSkillRegistration.hasModelDirective).toBe(true); + expect(beforeSkillRegistration.cleaned).toBe(""); reserveSkillCommandNames({ reservedCommands, @@ -45,29 +45,29 @@ describe("reply directive aliases", () => { ], }); - expect( - parseInlineDirectives("/demo_skill", { - modelAliases: resolveConfiguredDirectiveAliases({ - cfg, - commandTextHasSlash: true, - reservedCommands, - }), + const afterSkillRegistration = parseInlineDirectives("/demo_skill", { + modelAliases: resolveConfiguredDirectiveAliases({ + cfg, + commandTextHasSlash: true, + reservedCommands, }), - ).toEqual(expect.objectContaining({ hasModelDirective: false, cleaned: "/demo_skill" })); + }); + expect(afterSkillRegistration.hasModelDirective).toBe(false); + expect(afterSkillRegistration.cleaned).toBe("/demo_skill"); }); it("does not expose chat command names as inline model aliases", () => { const cfg = configWithModelAlias(" help "); const reservedCommands = new Set(["help"]); - expect( - parseInlineDirectives("/help", { - modelAliases: resolveConfiguredDirectiveAliases({ - cfg, - commandTextHasSlash: true, - reservedCommands, - }), + const parsed = parseInlineDirectives("/help", { + modelAliases: resolveConfiguredDirectiveAliases({ + cfg, + commandTextHasSlash: true, + reservedCommands, }), - ).toEqual(expect.objectContaining({ hasModelDirective: false, cleaned: "/help" })); + }); + expect(parsed.hasModelDirective).toBe(false); + expect(parsed.cleaned).toBe("/help"); }); });