unit tests fixin

package-lock.json

@@ -67,7 +67,7 @@
         "tailwindcss": "^4.1.17",
         "testcontainers": "^11.8.1",
         "ts-node-dev": "^2.0.0",
-        "typescript": "~5.9.3",
+        "typescript": "^5.9.3",
         "typescript-eslint": "^8.47.0",
         "vite": "^7.2.4",
         "vitest": "^4.0.13"

package.json

@@ -6,7 +6,7 @@
   "scripts": {
     "dev": "vite",
     "build": "DEBUG=\"tailwindcss:*\" vite build --debug",
-    "preview": "vite preview",    
+    "preview": "vite preview",
     "test": "vitest run --coverage",
     "test:integration": "vitest run --project integration",
     "lint": "eslint . --ext ts,tsx --report-unused-disable-directives --max-warnings 0",
@@ -72,7 +72,7 @@
     "tailwindcss": "^4.1.17",
     "testcontainers": "^11.8.1",
     "ts-node-dev": "^2.0.0",
-    "typescript": "~5.9.3",
+    "typescript": "^5.9.3",
     "typescript-eslint": "^8.47.0",
     "vite": "^7.2.4",
     "vitest": "^4.0.13"

src/tests/setup/mock-db.ts

@@ -19,7 +19,11 @@ vi.mock('pg', async (importOriginal) => {
   return {
     ...pg,
     // Override the Pool constructor. Now, any module that calls `new Pool()`
-    // will receive our singleton `testPool` instance instead of creating a new one.
+    // will receive our singleton `testPool` instance.
-    Pool: vi.fn(() => testPool),
+    // We use `mockImplementation` to return the instance, which is the correct
+    // way to mock a class constructor that should return a specific instance.
+    Pool: vi.fn().mockImplementation(() => {
+      return testPool;
+    }),
   };
 });