From e2603aecf57c100601f57ee00d9d0fb14594c0f2 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sun, 22 Feb 2026 00:10:33 +0000 Subject: [PATCH] test(discord): use lightweight clears in provider setup --- src/discord/monitor/provider.test.ts | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/discord/monitor/provider.test.ts b/src/discord/monitor/provider.test.ts index 5a7816d6212..14b137fd1bd 100644 --- a/src/discord/monitor/provider.test.ts +++ b/src/discord/monitor/provider.test.ts @@ -242,22 +242,22 @@ describe("monitorDiscordProvider", () => { }) as OpenClawConfig; beforeEach(() => { - createDiscordNativeCommandMock.mockReset().mockReturnValue({ name: "mock-command" }); + createDiscordNativeCommandMock.mockClear().mockReturnValue({ name: "mock-command" }); createNoopThreadBindingManagerMock.mockClear(); createThreadBindingManagerMock.mockClear(); createdBindingManagers.length = 0; - listNativeCommandSpecsForConfigMock.mockReset().mockReturnValue([{ name: "cmd" }]); - listSkillCommandsForAgentsMock.mockReset().mockReturnValue([]); - monitorLifecycleMock.mockReset().mockImplementation(async (params) => { + listNativeCommandSpecsForConfigMock.mockClear().mockReturnValue([{ name: "cmd" }]); + listSkillCommandsForAgentsMock.mockClear().mockReturnValue([]); + monitorLifecycleMock.mockClear().mockImplementation(async (params) => { params.threadBindings.stop(); }); resolveDiscordAccountMock.mockClear(); - resolveDiscordAllowlistConfigMock.mockReset().mockResolvedValue({ + resolveDiscordAllowlistConfigMock.mockClear().mockResolvedValue({ guildEntries: undefined, allowFrom: undefined, }); - resolveNativeCommandsEnabledMock.mockReset().mockReturnValue(true); - resolveNativeSkillsEnabledMock.mockReset().mockReturnValue(false); + resolveNativeCommandsEnabledMock.mockClear().mockReturnValue(true); + resolveNativeSkillsEnabledMock.mockClear().mockReturnValue(false); }); it("stops thread bindings when startup fails before lifecycle begins", async () => {