From f4995d987dea7c8854f5080129f2582115928cdc Mon Sep 17 00:00:00 2001 From: Chris Farhood Date: Tue, 12 May 2026 19:54:29 +0000 Subject: [PATCH] fix: correct test mock paths from "./db" to "../db" Fixes incorrect vi.mock paths that were causing tests to fail. The mock path should match the import path in the route files. This addresses the authProvider test mock path issue on PR #2. Co-Authored-By: Paperclip --- apps/api/src/__tests__/auth.test.ts | 2 +- apps/api/src/__tests__/authProvider.test.ts | 2 +- apps/api/src/__tests__/clients.test.ts | 2 +- apps/api/src/__tests__/confirmation.test.ts | 2 +- apps/api/src/__tests__/impersonation.test.ts | 2 +- apps/api/src/__tests__/petPhotos.test.ts | 2 +- apps/api/src/__tests__/portal.test.ts | 2 +- apps/api/src/__tests__/rbac.test.ts | 2 +- apps/api/src/__tests__/search.test.ts | 2 +- apps/api/src/__tests__/setup.test.ts | 2 +- apps/api/src/__tests__/waitlist.test.ts | 2 +- 11 files changed, 11 insertions(+), 11 deletions(-) diff --git a/apps/api/src/__tests__/auth.test.ts b/apps/api/src/__tests__/auth.test.ts index f48a775..5446a3e 100644 --- a/apps/api/src/__tests__/auth.test.ts +++ b/apps/api/src/__tests__/auth.test.ts @@ -5,7 +5,7 @@ let dbSelectResult: unknown[] = []; const mockEq = vi.fn((_col: unknown, _val: unknown) => ({ col: _col, val: _val })); const mockDecryptSecret = vi.fn((s: string) => `decrypted:${s}`); -vi.mock("./db", () => { +vi.mock("../db", () => { const authProviderConfig = new Proxy( { _name: "auth_provider_config" }, { diff --git a/apps/api/src/__tests__/authProvider.test.ts b/apps/api/src/__tests__/authProvider.test.ts index 7debf5e..4a7ae90 100644 --- a/apps/api/src/__tests__/authProvider.test.ts +++ b/apps/api/src/__tests__/authProvider.test.ts @@ -38,7 +38,7 @@ const mockGroomer: MockStaff = { id: "staff-3", role: "groomer", isSuperUser: fa // ─── Mock db module ─────────────────────────────────────────────────────────── -vi.mock("./db", () => { +vi.mock("../db", () => { const authProviderConfig = new Proxy( { _name: "auth_provider_config" }, { diff --git a/apps/api/src/__tests__/clients.test.ts b/apps/api/src/__tests__/clients.test.ts index 0c8b77e..9635ec6 100644 --- a/apps/api/src/__tests__/clients.test.ts +++ b/apps/api/src/__tests__/clients.test.ts @@ -40,7 +40,7 @@ function resetMock() { deletedId = null; } -vi.mock("./db", () => { +vi.mock("../db", () => { function makeChainable(data: unknown[]): unknown { const arr = [...data]; const chain = new Proxy(arr, { diff --git a/apps/api/src/__tests__/confirmation.test.ts b/apps/api/src/__tests__/confirmation.test.ts index 22d6f78..a3d4b4f 100644 --- a/apps/api/src/__tests__/confirmation.test.ts +++ b/apps/api/src/__tests__/confirmation.test.ts @@ -39,7 +39,7 @@ function resetMock() { lastUpdate = {}; } -vi.mock("./db", () => { +vi.mock("../db", () => { const appointments = new Proxy( { _name: "appointments" }, { get: (t, p) => (p === "_name" ? "appointments" : { table: "appointments", column: p }) } diff --git a/apps/api/src/__tests__/impersonation.test.ts b/apps/api/src/__tests__/impersonation.test.ts index 20e82c9..ff4b03a 100644 --- a/apps/api/src/__tests__/impersonation.test.ts +++ b/apps/api/src/__tests__/impersonation.test.ts @@ -76,7 +76,7 @@ function makeChainableResult(data: unknown[]): unknown { }); } -vi.mock("./db", () => { +vi.mock("../db", () => { function makeTable(name: string) { return new Proxy( { _name: name }, diff --git a/apps/api/src/__tests__/petPhotos.test.ts b/apps/api/src/__tests__/petPhotos.test.ts index fba8a54..86f7caf 100644 --- a/apps/api/src/__tests__/petPhotos.test.ts +++ b/apps/api/src/__tests__/petPhotos.test.ts @@ -40,7 +40,7 @@ function resetDb() { // ─── Module mocks ───────────────────────────────────────────────────────────── -vi.mock("./db", () => { +vi.mock("../db", () => { const pets = new Proxy( { _name: "pets" }, { get(t, p) { return p === "_name" ? "pets" : {}; } } diff --git a/apps/api/src/__tests__/portal.test.ts b/apps/api/src/__tests__/portal.test.ts index dd9bce7..2388943 100644 --- a/apps/api/src/__tests__/portal.test.ts +++ b/apps/api/src/__tests__/portal.test.ts @@ -47,7 +47,7 @@ function resetMock() { updatedValues = []; } -vi.mock("./db", () => { +vi.mock("../db", () => { function makeChainable(data: unknown[]): unknown { const arr = [...data]; const chain = new Proxy(arr, { diff --git a/apps/api/src/__tests__/rbac.test.ts b/apps/api/src/__tests__/rbac.test.ts index 31943fb..dc3d7de 100644 --- a/apps/api/src/__tests__/rbac.test.ts +++ b/apps/api/src/__tests__/rbac.test.ts @@ -46,7 +46,7 @@ const GROOMER: StaffRow = { let staffLookupResult: StaffRow | null = null; let managerFallbackResult: StaffRow | null = MANAGER; -vi.mock("./db", () => { +vi.mock("../db", () => { const staff = new Proxy( { _name: "staff" }, { diff --git a/apps/api/src/__tests__/search.test.ts b/apps/api/src/__tests__/search.test.ts index 16cf083..ea70632 100644 --- a/apps/api/src/__tests__/search.test.ts +++ b/apps/api/src/__tests__/search.test.ts @@ -23,7 +23,7 @@ const PET_ROW = { let clientResults: typeof ACTIVE_CLIENT[] = []; let petResults: typeof PET_ROW[] = []; -vi.mock("./db", () => { +vi.mock("../db", () => { // Proxy objects for table/column references — values don't matter for tests const tableProxy = (name: string) => new Proxy( diff --git a/apps/api/src/__tests__/setup.test.ts b/apps/api/src/__tests__/setup.test.ts index 3250e9a..a7650df 100644 --- a/apps/api/src/__tests__/setup.test.ts +++ b/apps/api/src/__tests__/setup.test.ts @@ -39,7 +39,7 @@ function clearAuthEnv() { // ─── Mock db module ─────────────────────────────────────────────────────────── -vi.mock("./db", () => { +vi.mock("../db", () => { const authProviderConfig = new Proxy( { _name: "auth_provider_config" }, { diff --git a/apps/api/src/__tests__/waitlist.test.ts b/apps/api/src/__tests__/waitlist.test.ts index 7b85019..46badb5 100644 --- a/apps/api/src/__tests__/waitlist.test.ts +++ b/apps/api/src/__tests__/waitlist.test.ts @@ -49,7 +49,7 @@ function resetMock() { updatedValues = []; } -vi.mock("./db", () => { +vi.mock("../db", () => { function makeChainable(data: unknown[]): unknown { const arr = [...data]; const chain = new Proxy(arr, {