@vellumai/assistant 0.5.13 → 0.5.14

package/src/__tests__/app-git-history.test.ts

@@ -1,28 +1,20 @@
-import { existsSync, mkdirSync, readFileSync, rmSync } from "node:fs";
+import { existsSync, readFileSync, rmSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
-import { afterEach, beforeEach, describe, expect, mock, test } from "bun:test";
+import { afterEach, beforeEach, describe, expect, test } from "bun:test";
 
 import { _resetGitServiceRegistry } from "../workspace/git-service.js";
 
-// Mock getDataDir to use a temp directory
 let testDataDir: string;
 
-mock.module("../util/platform.js", () => ({
-  getDataDir: () => testDataDir,
-  getProjectDir: () => testDataDir,
-}));
-
-// Re-import after mocking so modules use our temp dir
-const { createApp, updateApp, getAppDirPath } =
-  await import("../memory/app-store.js");
-const {
-  getAppHistory,
+import {
+  commitAppTurnChanges,
   getAppDiff,
   getAppFileAtVersion,
+  getAppHistory,
   restoreAppVersion,
-  commitAppTurnChanges,
-} = await import("../memory/app-git-service.js");
+} from "../memory/app-git-service.js";
+import { createApp, getAppDirPath, updateApp } from "../memory/app-store.js";
 
 describe("App Git History", () => {
   beforeEach(() => {
@@ -32,7 +24,7 @@ describe("App Git History", () => {
         .toString(36)
         .slice(2)}`,
     );
-    mkdirSync(join(testDataDir, "apps"), { recursive: true });
+    process.env.VELLUM_WORKSPACE_DIR = testDataDir;
     _resetGitServiceRegistry();
   });
 
@@ -55,7 +47,7 @@ describe("App Git History", () => {
 
     const history = await getAppHistory(app.id);
     expect(history.length).toBeGreaterThanOrEqual(2);
-    expect(history[0].message).toContain("Turn 2");
+    expect(history[0].message).toContain("update ");
     expect(history[0].commitHash).toMatch(/^[0-9a-f]+$/);
     expect(history[0].timestamp).toBeGreaterThan(0);
   });

package/src/__tests__/app-git-service.test.ts

@@ -1,29 +1,21 @@
 import { execFileSync } from "node:child_process";
-import { existsSync, mkdirSync, readFileSync, rmSync } from "node:fs";
+import { existsSync, readFileSync, rmSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
-import { afterEach, beforeEach, describe, expect, mock, test } from "bun:test";
+import { afterEach, beforeEach, describe, expect, test } from "bun:test";
 
 import { commitAppTurnChanges } from "../memory/app-git-service.js";
-import { _resetGitServiceRegistry } from "../workspace/git-service.js";
-
-// Mock getDataDir to use a temp directory
-let testDataDir: string;
-
-mock.module("../util/platform.js", () => ({
-  getDataDir: () => testDataDir,
-  getProjectDir: () => testDataDir,
-}));
-
-// Re-import app-store after mocking so it uses our temp dir
-const {
+import {
   createApp,
-  updateApp,
   deleteApp,
-  writeAppFile,
   editAppFile,
   getAppsDir,
-} = await import("../memory/app-store.js");
+  updateApp,
+  writeAppFile,
+} from "../memory/app-store.js";
+import { _resetGitServiceRegistry } from "../workspace/git-service.js";
+
+let testDataDir: string;
 
 describe("App Git Service", () => {
   beforeEach(() => {
@@ -31,7 +23,7 @@ describe("App Git Service", () => {
       tmpdir(),
       `vellum-app-git-test-${Date.now()}-${Math.random().toString(36).slice(2)}`,
     );
-    mkdirSync(join(testDataDir, "apps"), { recursive: true });
+    process.env.VELLUM_WORKSPACE_DIR = testDataDir;
     _resetGitServiceRegistry();
   });
 
@@ -100,7 +92,9 @@ describe("App Git Service", () => {
     expect(commits.length).toBeLessThanOrEqual(2);
     // The turn commit message should appear (or files are in the initial commit)
     expect(
-      commits.some((c) => c.includes("Turn 1") || c.includes("Initial commit")),
+      commits.some(
+        (c) => c.includes("update ") || c.includes("Initial commit"),
+      ),
     ).toBe(true);
   });
 
@@ -141,6 +135,6 @@ describe("App Git Service", () => {
 
     const appsDir = getAppsDir();
     const commits = getGitLog(appsDir);
-    expect(commits[0]).toContain("Turn 2: app changes");
+    expect(commits[0]).toContain("update ");
   });
 });

package/src/__tests__/app-store-dir-names.test.ts

@@ -7,32 +7,22 @@ import {
 } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
-import { afterEach, beforeEach, describe, expect, mock, test } from "bun:test";
-
-// ---------------------------------------------------------------------------
-// Mock getDataDir to use a temp directory
-// ---------------------------------------------------------------------------
+import { afterEach, beforeEach, describe, expect, test } from "bun:test";
 
 let testDataDir: string;
 
-mock.module("../util/platform.js", () => ({
-  getDataDir: () => testDataDir,
-  getProjectDir: () => testDataDir,
-}));
-
-// Re-import app-store after mocking so it uses our temp dir
-const {
-  slugify,
+import {
+  createApp,
+  deleteApp,
   generateAppDirName,
-  validateDirName,
-  resolveAppDir,
+  getApp,
   getAppDirPath,
   getAppsDir,
-  createApp,
-  getApp,
+  resolveAppDir,
+  slugify,
   updateApp,
-  deleteApp,
-} = await import("../memory/app-store.js");
+  validateDirName,
+} from "../memory/app-store.js";
 
 // ---------------------------------------------------------------------------
 // Helpers
@@ -59,7 +49,7 @@ function makeAppParams(name: string) {
 
 beforeEach(() => {
   testDataDir = freshTempDir();
-  mkdirSync(join(testDataDir, "apps"), { recursive: true });
+  process.env.VELLUM_WORKSPACE_DIR = testDataDir;
 });
 
 afterEach(() => {

package/src/__tests__/approval-cascade.test.ts

@@ -5,10 +5,7 @@
  * allow_conversation, always_allow, always_deny), other pending confirmations in
  * the same conversation that match are auto-resolved.
  */
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
+import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 import { Minimatch } from "minimatch";
 
@@ -22,8 +19,6 @@ import type { ConfirmationStateChanged } from "../daemon/message-types/messages.
 import type { Message, ProviderResponse } from "../providers/types.js";
 import type { ConfirmationDetails } from "../runtime/pending-interactions.js";
 
-const testDir = mkdtempSync(join(tmpdir(), "approval-cascade-test-"));
-
 // ---------------------------------------------------------------------------
 // Mocks — must precede Conversation import
 // ---------------------------------------------------------------------------
@@ -49,10 +44,6 @@ mock.module("../util/logger.js", () => ({
   getLogger: () => makeLoggerStub(),
 }));
 
-mock.module("../util/platform.js", () => ({
-  getDataDir: () => testDir,
-}));
-
 mock.module("../memory/guardian-action-store.js", () => ({
   getGuardianActionRequest: () => null,
   resolveGuardianActionRequest: () => {},
@@ -265,7 +256,7 @@ function makeConversation(
     "system prompt",
     4096,
     sendToClient ?? (() => {}),
-    testDir,
+    process.env.VELLUM_WORKSPACE_DIR!,
   );
 }
 
@@ -325,14 +316,6 @@ function makeConfirmationDetails(patterns: string[]): ConfirmationDetails {
   };
 }
 
-afterAll(() => {
-  try {
-    rmSync(testDir, { recursive: true, force: true });
-  } catch {
-    /* best effort */
-  }
-});
-
 beforeEach(() => {
   pendingInteractions.clear();
 });

package/src/__tests__/approval-primitive.test.ts

@@ -1,20 +1,4 @@
-import { mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
-
-const testDir = mkdtempSync(join(tmpdir(), "approval-primitive-test-"));
-
-mock.module("../util/platform.js", () => ({
-  getDataDir: () => testDir,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => join(testDir, "test.db"),
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
+import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
@@ -29,7 +13,7 @@ import {
   mintGrantFromDecision,
   type MintGrantParams,
 } from "../approvals/approval-primitive.js";
-import { getDb, initializeDb, resetDb } from "../memory/db.js";
+import { getDb, initializeDb } from "../memory/db.js";
 import { scopedApprovalGrants } from "../memory/schema.js";
 import { computeToolApprovalDigest } from "../security/tool-approval-digest.js";
 
@@ -40,15 +24,6 @@ function clearTables(): void {
   db.delete(scopedApprovalGrants).run();
 }
 
-afterAll(() => {
-  resetDb();
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
-});
-
 // ---------------------------------------------------------------------------
 // Helper to build mint params with sensible defaults
 // ---------------------------------------------------------------------------

package/src/__tests__/approval-routes-http.test.ts

@@ -4,30 +4,11 @@
  * Tests POST /v1/confirm, POST /v1/secret, and POST /v1/trust-rules
  * through RuntimeHttpServer with pending-interactions tracking.
  */
-import { mkdtempSync, realpathSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
+import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 import type { Conversation } from "../daemon/conversation.js";
 import type { ServerMessage } from "../daemon/message-protocol.js";
 
-const testDir = realpathSync(
-  mkdtempSync(join(tmpdir(), "approval-routes-http-test-")),
-);
-
-mock.module("../util/platform.js", () => ({
-  getRootDir: () => testDir,
-  getDataDir: () => testDir,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => join(testDir, "test.db"),
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
-
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
     new Proxy({} as Record<string, unknown>, {
@@ -86,7 +67,7 @@ mock.module("../permissions/trust-store.js", () => ({
   getRules: () => [],
 }));
 
-import { getDb, initializeDb, resetDb } from "../memory/db.js";
+import { getDb, initializeDb } from "../memory/db.js";
 import { AssistantEventHub } from "../runtime/assistant-event-hub.js";
 import { RuntimeHttpServer } from "../runtime/http-server.js";
 import * as pendingInteractions from "../runtime/pending-interactions.js";
@@ -248,15 +229,6 @@ describe("standalone approval endpoints — HTTP layer", () => {
     eventHub = new AssistantEventHub();
   });
 
-  afterAll(() => {
-    resetDb();
-    try {
-      rmSync(testDir, { recursive: true, force: true });
-    } catch {
-      /* best effort */
-    }
-  });
-
   async function startServer(
     conversationFactory: () => Conversation,
   ): Promise<void> {

package/src/__tests__/assistant-events-sse-hardening.test.ts

@@ -8,24 +8,7 @@
  *   - Subscription cleanup on request abort.
  *   - Subscription cleanup on reader cancel.
  */
-import { mkdtempSync, realpathSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
-
-const testDir = realpathSync(mkdtempSync(join(tmpdir(), "sse-hardening-")));
-
-mock.module("../util/platform.js", () => ({
-  getRootDir: () => testDir,
-  getDataDir: () => testDir,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => join(testDir, "test.db"),
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-}));
+import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
@@ -46,21 +29,12 @@ mock.module("../config/loader.js", () => ({
   }),
 }));
 
-import { getDb, initializeDb, resetDb } from "../memory/db.js";
+import { getDb, initializeDb } from "../memory/db.js";
 import { AssistantEventHub } from "../runtime/assistant-event-hub.js";
 import { handleSubscribeAssistantEvents } from "../runtime/routes/events-routes.js";
 
 initializeDb();
 
-afterAll(() => {
-  resetDb();
-  try {
-    rmSync(testDir, { recursive: true, force: true });
-  } catch {
-    /* best effort */
-  }
-});
-
 // ── Helpers ───────────────────────────────────────────────────────────────────
 
 function clearTables() {

package/src/__tests__/assistant-feature-flags-integration.test.ts

@@ -8,14 +8,7 @@
  *   - Protected feature-flags.json is the sole override mechanism
  *   - Undeclared keys default to enabled
  */
-import {
-  existsSync,
-  mkdirSync,
-  readFileSync,
-  rmSync,
-  writeFileSync,
-} from "node:fs";
-import { tmpdir } from "node:os";
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
 import { afterEach, beforeEach, describe, expect, mock, test } from "bun:test";
 
@@ -23,10 +16,7 @@ import { afterEach, beforeEach, describe, expect, mock, test } from "bun:test";
 // Test-scoped temp directory and config state
 // ---------------------------------------------------------------------------
 
-const TEST_DIR = join(
-  tmpdir(),
-  `vellum-asst-flags-test-${crypto.randomUUID()}`,
-);
+const TEST_DIR = process.env.VELLUM_WORKSPACE_DIR!;
 
 let currentConfig: Record<string, unknown> = {
   services: {
@@ -48,35 +38,6 @@ const DECLARED_FLAG_ID = "contacts";
 const DECLARED_FLAG_KEY = DECLARED_FLAG_ID;
 const DECLARED_SKILL_ID = "contacts";
 
-// eslint-disable-next-line @typescript-eslint/no-require-imports
-const realPlatform = require("../util/platform.js");
-mock.module("../util/platform.js", () => ({
-  ...realPlatform,
-  getRootDir: () => TEST_DIR,
-  getDataDir: () => TEST_DIR,
-  getWorkspaceDir: () => TEST_DIR,
-  getWorkspaceConfigPath: () => join(TEST_DIR, "config.json"),
-  getWorkspaceSkillsDir: () => join(TEST_DIR, "skills"),
-  getWorkspaceHooksDir: () => join(TEST_DIR, "hooks"),
-  getWorkspacePromptPath: (file: string) => join(TEST_DIR, file),
-  ensureDataDir: () => {},
-  getPidPath: () => join(TEST_DIR, "vellum.pid"),
-  getDbPath: () => join(TEST_DIR, "data", "assistant.db"),
-  getLogPath: () => join(TEST_DIR, "logs", "vellum.log"),
-  getHistoryPath: () => join(TEST_DIR, "history"),
-  getHooksDir: () => join(TEST_DIR, "hooks"),
-
-  getSandboxRootDir: () => join(TEST_DIR, "sandbox"),
-  getSandboxWorkingDir: () => TEST_DIR,
-  getInterfacesDir: () => join(TEST_DIR, "interfaces"),
-  isMacOS: () => false,
-  isLinux: () => false,
-  isWindows: () => false,
-  getPlatformName: () => "linux",
-  getClipboardCommand: () => null,
-  readSessionToken: () => null,
-}));
-
 const noopLogger = new Proxy({} as Record<string, unknown>, {
   get: (_target, prop) => (prop === "child" ? () => noopLogger : () => {}),
 });
@@ -128,7 +89,6 @@ const { skillFlagKey } = await import("../config/skill-state.js");
 // ---------------------------------------------------------------------------
 
 beforeEach(() => {
-  mkdirSync(TEST_DIR, { recursive: true });
   _setOverridesForTesting({});
   currentConfig = {
     services: {
@@ -149,9 +109,6 @@ beforeEach(() => {
 
 afterEach(() => {
   _setOverridesForTesting({});
-  if (existsSync(TEST_DIR)) {
-    rmSync(TEST_DIR, { recursive: true, force: true });
-  }
 });
 
 // ---------------------------------------------------------------------------

package/src/__tests__/attachments-store.test.ts

@@ -1,23 +1,6 @@
-import { existsSync, mkdirSync, mkdtempSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
+import { existsSync, mkdirSync, rmSync } from "node:fs";
 import { join } from "node:path";
-import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
-
-const testDir = mkdtempSync(join(tmpdir(), "attach-store-test-"));
-
-mock.module("../util/platform.js", () => ({
-  getDataDir: () => testDir,
-  isMacOS: () => process.platform === "darwin",
-  isLinux: () => process.platform === "linux",
-  isWindows: () => process.platform === "win32",
-  getPidPath: () => join(testDir, "test.pid"),
-  getDbPath: () => join(testDir, "test.db"),
-  getLogPath: () => join(testDir, "test.log"),
-  ensureDataDir: () => {},
-  getRootDir: () => testDir,
-  getWorkspaceDir: () => join(testDir, "workspace"),
-  getConversationsDir: () => join(testDir, "workspace", "conversations"),
-}));
+import { beforeEach, describe, expect, mock, test } from "bun:test";
 
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
@@ -54,19 +37,11 @@ import {
 } from "../memory/attachments-store.js";
 import { addMessage, createConversation } from "../memory/conversation-crud.js";
 import { getConversationDirPath } from "../memory/conversation-disk-view.js";
-import { getDb, initializeDb, rawGet, rawRun, resetDb } from "../memory/db.js";
+import { getDb, initializeDb, rawGet, rawRun } from "../memory/db.js";
+import { getConversationsDir } from "../util/platform.js";
 
 initializeDb();
 
-afterAll(() => {
-  resetDb();
-  try {
-    rmSync(testDir, { recursive: true });
-  } catch {
-    /* best effort */
-  }
-});
-
 function resetTables() {
   const db = getDb();
   db.run("DELETE FROM message_attachments");
@@ -84,9 +59,7 @@ function getLegacyConversationDirPath(
   createdAt: number,
 ): string {
   return join(
-    testDir,
-    "workspace",
-    "conversations",
+    getConversationsDir(),
     `${conversationId}_${getConversationTimestamp(createdAt)}`,
   );
 }

package/src/__tests__/audit-log-rotation.test.ts

@@ -1,20 +1,10 @@
 import { randomBytes } from "node:crypto";
-import { existsSync, mkdirSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { Database } from "bun:sqlite";
 import { beforeAll, beforeEach, describe, expect, mock, test } from "bun:test";
 
 // ---------------------------------------------------------------------------
-// Test setup — temp directory and mock modules
+// Test setup — mock modules
 // ---------------------------------------------------------------------------
 
-const TEST_DIR = join(
-  tmpdir(),
-  `vellum-rotation-test-${randomBytes(4).toString("hex")}`,
-);
-const DB_PATH = join(TEST_DIR, "assistant.db");
-
 mock.module("../util/logger.js", () => ({
   getLogger: () =>
     new Proxy({} as Record<string, unknown>, {
@@ -22,19 +12,8 @@ mock.module("../util/logger.js", () => ({
     }),
 }));
 
-mock.module("../util/platform.js", () => ({
-  getDataDir: () => TEST_DIR,
-  getDbPath: () => DB_PATH,
-  getLogPath: () => join(TEST_DIR, "logs", "vellum.log"),
-  ensureDataDir: () => {
-    if (!existsSync(TEST_DIR)) mkdirSync(TEST_DIR, { recursive: true });
-  },
-  isMacOS: () => false,
-  isLinux: () => false,
-  isWindows: () => false,
-}));
-
 import { initializeDb, resetDb } from "../memory/db.js";
+import { getSqlite } from "../memory/db-connection.js";
 import {
   getRecentInvocations,
   rotateToolInvocations,
@@ -46,7 +25,7 @@ import {
 
 function addInvocation(ageMs: number): void {
   // Insert directly with a specific timestamp in the past
-  const db = new Database(DB_PATH);
+  const db = getSqlite();
   const id = randomBytes(8).toString("hex");
   const createdAt = Date.now() - ageMs;
   db.prepare(
@@ -63,13 +42,10 @@ function addInvocation(ageMs: number): void {
     100,
     createdAt,
   );
-  db.close();
 }
 
 function clearTable(): void {
-  const db = new Database(DB_PATH);
-  db.run("DELETE FROM tool_invocations");
-  db.close();
+  getSqlite().run("DELETE FROM tool_invocations");
 }
 
 const ONE_DAY_MS = 24 * 60 * 60 * 1000;
@@ -80,25 +56,18 @@ const ONE_DAY_MS = 24 * 60 * 60 * 1000;
 
 describe("audit log rotation", () => {
   beforeAll(() => {
-    mkdirSync(join(TEST_DIR, "logs"), { recursive: true });
     resetDb();
     initializeDb();
     // Insert a conversations row so FK-enforced ORM inserts succeed
-    const db = new Database(DB_PATH);
-    db.run(
+    getSqlite().run(
       `INSERT INTO conversations (id, title, created_at, updated_at) VALUES ('conv-1', 'test', ${Date.now()}, ${Date.now()})`,
     );
-    db.close();
   });
 
   beforeEach(() => {
     clearTable();
   });
 
-  // TEST_DIR is in os.tmpdir() — let the OS handle cleanup.
-  // Deleting it here would trigger ENOENT in other test files because
-  // the platform.js mock leaks getDataDir() → TEST_DIR globally.
-
   test("returns 0 when retentionDays is 0 (retain forever)", () => {
     addInvocation(100 * ONE_DAY_MS); // 100 days old
     const deleted = rotateToolInvocations(0);

package/src/__tests__/avatar-e2e.test.ts

@@ -5,7 +5,6 @@ import { afterEach, beforeEach, describe, expect, mock, test } from "bun:test";
 // ---------------------------------------------------------------------------
 
 let mockGeminiKey: string | undefined = "test-gemini-key";
-let mockWorkspaceDir = "/tmp/test-workspace-e2e";
 
 const mkdirSyncFn = mock(() => {});
 const writeFileSyncFn = mock(() => {});
@@ -51,10 +50,6 @@ mock.module("../security/secure-keys.js", () => ({
     provider === "gemini" ? mockGeminiKey : undefined,
 }));
 
-mock.module("../util/platform.js", () => ({
-  getWorkspaceDir: () => mockWorkspaceDir,
-}));
-
 mock.module("pino", () => {
   function createLogger() {
     return {
@@ -140,8 +135,7 @@ function geminiContentResponse() {
   };
 }
 
-const expectedAvatarPath =
-  "/tmp/test-workspace-e2e/data/avatar/avatar-image.png";
+const expectedAvatarPath = `${process.env.VELLUM_WORKSPACE_DIR}/data/avatar/avatar-image.png`;
 
 // ---------------------------------------------------------------------------
 // Tests
@@ -153,8 +147,6 @@ describe("avatar E2E integration", () => {
 
   beforeEach(() => {
     mockGeminiKey = "test-gemini-key";
-    mockWorkspaceDir = "/tmp/test-workspace-e2e";
-
     mkdirSyncFn.mockClear();
     writeFileSyncFn.mockClear();
     renameSyncFn.mockClear();