From b58695bdb837e07bd99f64d6e79405bc5151c186 Mon Sep 17 00:00:00 2001
From: OpenClaw Subagent <subagent@openclaw.com>
Date: Thu, 19 Mar 2026 11:07:22 +0100
Subject: [PATCH] fix: remove contradictory vi.unmock breaking recover DB
 fallback tests

The test file had both vi.unmock and vi.mock for the same module.
vi.unmock takes precedence in Vitest, so getAllKeys was the real
function instead of a mock, causing mockReturnValueOnce to fail.
---
 src/__tests__/recover-db-fallback.test.ts | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/__tests__/recover-db-fallback.test.ts b/src/__tests__/recover-db-fallback.test.ts
index 4129b37..2d55fa4 100644
--- a/src/__tests__/recover-db-fallback.test.ts
+++ b/src/__tests__/recover-db-fallback.test.ts
@@ -1,7 +1,5 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
 
-// Unmock keys to use real getAllKeys (but we'll mock it ourselves)
-vi.unmock("../services/keys.js");
 // verification.js and email.js are mocked below
 
 vi.mock("../services/db.js", () => ({