github/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-05-08 08:01:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(core): use lightweight clears in stable mock setup

Browse Source
This commit is contained in:
Peter Steinberger
2026-02-22 07:55:00 +00:00
parent ad400afb24
commit 089270e769
3 changed files with 12 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/agents/pi-tool-definition-adapter.after-tool-call.e2e.test.ts
View File

@@ -66,14 +66,14 @@ function expectReadAfterToolCallPayload(result: Awaited<ReturnType<typeof execut
describe("pi tool definition adapter after_tool_call", () => { describe("pi tool definition adapter after_tool_call", () => {
beforeEach(() => { beforeEach(() => {
hookMocks.runner.hasHooks.mockReset(); hookMocks.runner.hasHooks.mockClear();
hookMocks.runner.runAfterToolCall.mockReset(); hookMocks.runner.runAfterToolCall.mockClear();
hookMocks.runner.runAfterToolCall.mockResolvedValue(undefined); hookMocks.runner.runAfterToolCall.mockResolvedValue(undefined);
hookMocks.isToolWrappedWithBeforeToolCallHook.mockReset(); hookMocks.isToolWrappedWithBeforeToolCallHook.mockClear();
hookMocks.isToolWrappedWithBeforeToolCallHook.mockReturnValue(false); hookMocks.isToolWrappedWithBeforeToolCallHook.mockReturnValue(false);
hookMocks.consumeAdjustedParamsForToolCall.mockReset(); hookMocks.consumeAdjustedParamsForToolCall.mockClear();
hookMocks.consumeAdjustedParamsForToolCall.mockReturnValue(undefined); hookMocks.consumeAdjustedParamsForToolCall.mockReturnValue(undefined);
hookMocks.runBeforeToolCallHook.mockReset(); hookMocks.runBeforeToolCallHook.mockClear();
hookMocks.runBeforeToolCallHook.mockImplementation(async ({ params }) => ({ hookMocks.runBeforeToolCallHook.mockImplementation(async ({ params }) => ({
blocked: false, blocked: false,
params, params,

6
src/agents/tools/sessions.e2e.test.ts
View File

@@ -134,7 +134,7 @@ describe("extractAssistantText", () => {
describe("resolveAnnounceTarget", () => { describe("resolveAnnounceTarget", () => {
beforeEach(async () => { beforeEach(async () => {
callGatewayMock.mockReset(); callGatewayMock.mockClear();
await installRegistry(); await installRegistry();
}); });
@@ -179,7 +179,7 @@ describe("resolveAnnounceTarget", () => {
describe("sessions_list gating", () => { describe("sessions_list gating", () => {
beforeEach(() => { beforeEach(() => {
callGatewayMock.mockReset(); callGatewayMock.mockClear();
callGatewayMock.mockResolvedValue({ callGatewayMock.mockResolvedValue({
path: "/tmp/sessions.json", path: "/tmp/sessions.json",
sessions: [ sessions: [
@@ -201,7 +201,7 @@ describe("sessions_list gating", () => {
describe("sessions_send gating", () => { describe("sessions_send gating", () => {
beforeEach(() => { beforeEach(() => {
callGatewayMock.mockReset(); callGatewayMock.mockClear();
}); });
it("blocks cross-agent sends when tools.agentToAgent.enabled is false", async () => { it("blocks cross-agent sends when tools.agentToAgent.enabled is false", async () => {

8
src/memory/qmd-manager.test.ts
View File

@@ -120,9 +120,9 @@ describe("QmdMemoryManager", () => {
beforeEach(async () => { beforeEach(async () => {
spawnMock.mockReset(); spawnMock.mockReset();
spawnMock.mockImplementation(() => createMockChild()); spawnMock.mockImplementation(() => createMockChild());
logWarnMock.mockReset(); logWarnMock.mockClear();
logDebugMock.mockReset(); logDebugMock.mockClear();
logInfoMock.mockReset(); logInfoMock.mockClear();
tmpRoot = path.join(fixtureRoot, `case-${fixtureCount++}`); tmpRoot = path.join(fixtureRoot, `case-${fixtureCount++}`);
await fs.mkdir(tmpRoot); await fs.mkdir(tmpRoot);
workspaceDir = path.join(tmpRoot, "workspace"); workspaceDir = path.join(tmpRoot, "workspace");
@@ -1957,7 +1957,7 @@ describe("QmdMemoryManager", () => {
await fs.rm(customModelsDir, { recursive: true, force: true }); await fs.rm(customModelsDir, { recursive: true, force: true });
await fs.mkdir(defaultModelsDir, { recursive: true }); await fs.mkdir(defaultModelsDir, { recursive: true });
await fs.writeFile(path.join(defaultModelsDir, "model.bin"), "fake-model"); await fs.writeFile(path.join(defaultModelsDir, "model.bin"), "fake-model");
logWarnMock.mockReset(); logWarnMock.mockClear();
await testCase.setup?.(); await testCase.setup?.();
const { manager } = await createManager({ mode: "full" }); const { manager } = await createManager({ mode: "full" });
expect(manager, testCase.name).toBeTruthy(); expect(manager, testCase.name).toBeTruthy();