From 4281e8764251c9b6319716a3f4cdbbf525b355d1 Mon Sep 17 00:00:00 2001 From: egeoztass Date: Thu, 16 Apr 2026 21:14:47 +0300 Subject: [PATCH] fix: add pushoverProvider to NotificationsService test constructor The existing test was missing the new pushoverProvider argument, causing settingsService to be passed in the wrong position. Co-Authored-By: Claude Opus 4.6 (1M context) --- server/test/unit/services/notificationsService.test.ts | 3 +++ 1 file changed, 3 insertions(+) diff --git a/server/test/unit/services/notificationsService.test.ts b/server/test/unit/services/notificationsService.test.ts index 6c2a40214..00dd644f8 100644 --- a/server/test/unit/services/notificationsService.test.ts +++ b/server/test/unit/services/notificationsService.test.ts @@ -45,6 +45,7 @@ const createService = (overrides?: Record) => { const matrixProvider = createProvider(); const teamsProvider = createProvider(); const telegramProvider = createProvider(); + const pushoverProvider = createProvider(); const settingsService = createSettingsService(); const notificationMessageBuilder = createMessageBuilder(); @@ -60,6 +61,7 @@ const createService = (overrides?: Record) => { matrixProvider, teamsProvider, telegramProvider, + pushoverProvider, settingsService, notificationMessageBuilder, ...overrides, @@ -76,6 +78,7 @@ const createService = (overrides?: Record) => { defaults.matrixProvider as any, defaults.teamsProvider as any, defaults.telegramProvider as any, + defaults.pushoverProvider as any, defaults.settingsService as any, defaults.logger as any, defaults.notificationMessageBuilder as any