Refactor: Add personalizationRepo and notificationRepo mocks in route tests

src/routes/admin.content.routes.test.ts

@@ -60,6 +60,8 @@ vi.mock('../services/db/index.db', () => ({
   flyerRepo: mockedDb.flyerRepo,
   recipeRepo: mockedDb.recipeRepo,
   userRepo: mockedDb.userRepo,
+  personalizationRepo: {},
+  notificationRepo: {},
 }));
 
 // Mock other dependencies

src/routes/admin.jobs.routes.test.ts

@@ -45,6 +45,8 @@ vi.mock('../services/db/index.db', () => ({
   flyerRepo: {},
   recipeRepo: {},
   userRepo: {},
+  personalizationRepo: {},
+  notificationRepo: {},
 }));
 vi.mock('../services/geocodingService.server');
 vi.mock('node:fs/promises');

src/routes/admin.monitoring.routes.test.ts

@@ -35,6 +35,8 @@ vi.mock('../services/db/index.db', () => ({
   flyerRepo: {},
   recipeRepo: {},
   userRepo: {},
+  personalizationRepo: {},
+  notificationRepo: {},
 }));
 
 // Mock the queue service to control worker statuses

src/routes/admin.stats.routes.test.ts

@@ -25,6 +25,8 @@ vi.mock('../services/db/index.db', () => ({
   flyerRepo: {},
   recipeRepo: {},
   userRepo: {},
+  personalizationRepo: {},
+  notificationRepo: {},
 }));
 
 // Mock other dependencies

src/routes/admin.system.routes.test.ts

@@ -20,12 +20,17 @@ vi.mock('../services/db/index.db', () => ({
   recipeRepo: {},
   userRepo: {},
   personalizationRepo: {},
+  notificationRepo: {},
 }));
 vi.mock('../services/db/flyer.db');
 vi.mock('../services/db/recipe.db');
 vi.mock('../services/db/user.db');
 vi.mock('node:fs/promises');
-vi.mock('../services/backgroundJobService');
+vi.mock('../services/backgroundJobService', () => ({
+  backgroundJobService: {
+    runDailyDealCheck: vi.fn(),
+  },
+}));
 vi.mock('../services/queueService.server');
 vi.mock('@bull-board/api');
 vi.mock('@bull-board/api/bullMQAdapter');

src/routes/admin.users.routes.test.ts

@@ -6,9 +6,18 @@ import adminRouter from './admin.routes'; // This was a duplicate, fixed.
 import { createMockUserProfile, createMockAdminUserView } from '../tests/utils/mockFactories';
 import { UserProfile, Profile } from '../types';
 import { NotFoundError } from '../services/db/errors.db';
-import { mockLogger } from '../tests/utils/mockLogger';
 import { createTestApp } from '../tests/utils/createTestApp';
 
+const { mockLogger } = vi.hoisted(() => ({
+  mockLogger: {
+    info: vi.fn(),
+    debug: vi.fn(),
+    warn: vi.fn(),
+    error: vi.fn(),
+    child: vi.fn().mockReturnThis(),
+  },
+}));
+
 vi.mock('../services/db/index.db', () => ({
   adminRepo: {
     getAllUsers: vi.fn(),
@@ -20,6 +29,8 @@ vi.mock('../services/db/index.db', () => ({
   },
   flyerRepo: {},
   recipeRepo: {},
+  personalizationRepo: {},
+  notificationRepo: {},
 }));
 
 // Mock other dependencies that are not directly tested but are part of the adminRouter setup

src/utils/pdfConverter.test.ts

@@ -4,7 +4,6 @@
  */
 
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import type { PDFDocumentLoadingTask } from 'pdfjs-dist';
 import { convertPdfToImageFiles } from './pdfConverter';
 
 // Mock the entire pdfjs-dist library