From f0a5974eaf787474919872301ae32cdc326d6c1d Mon Sep 17 00:00:00 2001
From: Alex Holliday <ajhollid@gmail.com>
Date: Tue, 23 Jun 2026 11:31:35 -0700
Subject: [PATCH] update tests

---
 .../unit/services/maintenanceWindowService.test.ts   |  6 +++---
 server/test/unit/services/monitorService.test.ts     |  4 ++--
 server/test/unit/services/userService.test.ts        | 12 ++++++------
 3 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/server/test/unit/services/maintenanceWindowService.test.ts b/server/test/unit/services/maintenanceWindowService.test.ts
index 5c7387da8..9547eaac8 100644
--- a/server/test/unit/services/maintenanceWindowService.test.ts
+++ b/server/test/unit/services/maintenanceWindowService.test.ts
@@ -3,7 +3,7 @@ import { MaintenanceWindowService } from "../../../src/domain/maintenance-window
 import type { IMaintenanceWindowsRepository } from "../../../src/domain/maintenance-windows/maintenance-window.repository.interface.ts";
 import type { IMonitorsRepository } from "../../../src/domain/monitors/monitor.repository.interface.ts";
 import type { IJobsRepository } from "../../../src/domain/jobs/job.repository.interface.ts";
-import type { IWorker } from "../../../src/worker/worker.interface.ts";
+import type { IJobScheduler } from "../../../src/worker/worker.interface.ts";
 import type { MaintenanceWindow } from "../../../src/domain/maintenance-windows/maintenance-window.type.ts";
 
 // ── Helpers ──────────────────────────────────────────────────────────────────
@@ -37,7 +37,7 @@ const createJobsRepo = () =>
 const createWorker = () =>
 	({
 		wake: jest.fn(),
-	}) as unknown as jest.Mocked<IWorker>;
+	}) as unknown as jest.Mocked<IJobScheduler>;
 
 const createService = (overrides?: {
 	monitorsRepository?: ReturnType<typeof createMonitorsRepo>;
@@ -49,7 +49,7 @@ const createService = (overrides?: {
 	const maintenanceWindowsRepository = overrides?.maintenanceWindowsRepository ?? createMaintenanceWindowsRepo();
 	const jobsRepository = overrides?.jobsRepository ?? createJobsRepo();
 	const worker = overrides?.worker ?? createWorker();
-	const service = new MaintenanceWindowService({ monitorsRepository, maintenanceWindowsRepository, jobsRepository, worker });
+	const service = new MaintenanceWindowService({ monitorsRepository, maintenanceWindowsRepository, jobsRepository, scheduler: worker });
 	return { service, monitorsRepository, maintenanceWindowsRepository, jobsRepository, worker };
 };
 
diff --git a/server/test/unit/services/monitorService.test.ts b/server/test/unit/services/monitorService.test.ts
index d64007ebe..65d6923a6 100644
--- a/server/test/unit/services/monitorService.test.ts
+++ b/server/test/unit/services/monitorService.test.ts
@@ -98,7 +98,7 @@ const createService = (
 	const jobQueue = overrides.jobQueue ?? createJobQueueMock();
 	const logger = overrides.logger ?? createMockLogger();
 	const service = new MonitorService({
-		worker: jobQueue as any,
+		scheduler: jobQueue as any,
 		logger: logger as any,
 		games: (overrides.games ?? { cs2: { name: "Counter-Strike 2" } }) as any,
 		monitorsRepository: overrides.monitorsRepository ?? createMonitorsRepositoryMock(),
@@ -1535,7 +1535,7 @@ describe("MonitorService", () => {
 
 			const logger = createMockLogger();
 			const service = new MonitorService({
-				worker: jobQueue as any,
+				scheduler: jobQueue as any,
 				logger: logger as any,
 				games: {} as any,
 				monitorsRepository,
diff --git a/server/test/unit/services/userService.test.ts b/server/test/unit/services/userService.test.ts
index 7b2fb380b..061003e7f 100644
--- a/server/test/unit/services/userService.test.ts
+++ b/server/test/unit/services/userService.test.ts
@@ -65,7 +65,7 @@ const createService = (overrides?: Record<string, unknown>) => {
 	const settingsService = {
 		getSettings: jest.fn().mockReturnValue(makeAppSettings()),
 	};
-	const worker = {
+	const scheduler = {
 		deleteJob: jest.fn().mockResolvedValue(undefined),
 	};
 	const jwtMock = {
@@ -85,7 +85,7 @@ const createService = (overrides?: Record<string, unknown>) => {
 		monitorsRepository,
 		emailService,
 		settingsService,
-		worker,
+		scheduler,
 		jwt: jwtMock,
 		crypto: cryptoMock,
 		...overrides,
@@ -475,23 +475,23 @@ describe("UserService", () => {
 
 	describe("deleteUser", () => {
 		it("deletes user and cleans up monitor jobs for superadmin", async () => {
-			const { service, usersRepository, worker, monitorsRepository } = createService();
+			const { service, usersRepository, scheduler, monitorsRepository } = createService();
 
 			await service.deleteUser({ userId: "user-1", teamId: "team-1", roles: ["superadmin"] });
 
 			expect(monitorsRepository.findByTeamId).toHaveBeenCalledWith("team-1", {});
-			expect(worker.deleteJob).toHaveBeenCalledTimes(2);
+			expect(scheduler.deleteJob).toHaveBeenCalledTimes(2);
 			expect(usersRepository.deleteById).toHaveBeenCalledWith("user-1");
 		});
 
 		it("handles null monitors for superadmin gracefully", async () => {
-			const { service, usersRepository, worker } = createService({
+			const { service, usersRepository, scheduler } = createService({
 				monitorsRepository: { findByTeamId: jest.fn().mockResolvedValue(null) },
 			});
 
 			await service.deleteUser({ userId: "user-1", teamId: "team-1", roles: ["superadmin"] });
 
-			expect(worker.deleteJob).not.toHaveBeenCalled();
+			expect(scheduler.deleteJob).not.toHaveBeenCalled();
 			expect(usersRepository.deleteById).toHaveBeenCalledWith("user-1");
 		});