From ec897b185cf886c75a9422647e0f5c4b34c09563 Mon Sep 17 00:00:00 2001
From: FoxxMD <FoxxMD@users.noreply.github.com>
Date: Mon, 19 Feb 2024 10:50:10 -0500
Subject: [PATCH] fix: Fix logger used for tests to always be 'noop'

Cleans up logging output during tests
---
 src/backend/tests/jellyfin/jellyfin.test.ts  | 2 +-
 src/backend/tests/player/player.test.ts      | 2 +-
 src/backend/tests/scrobbler/TestScrobbler.ts | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/backend/tests/jellyfin/jellyfin.test.ts b/src/backend/tests/jellyfin/jellyfin.test.ts
index 41e93712..c704a28d 100644
--- a/src/backend/tests/jellyfin/jellyfin.test.ts
+++ b/src/backend/tests/jellyfin/jellyfin.test.ts
@@ -28,7 +28,7 @@ describe('Jellyfin Payload Parsing', function () {
 
     describe('Correctly detects events as valid/invalid', function () {
 
-        const jfSource = new JellyfinSource('Test', {data: {}}, {localUrl: 'test', configDir: 'test', logger: getLogger({}, 'Test')}, new EventEmitter());
+        const jfSource = new JellyfinSource('Test', {data: {}}, {localUrl: 'test', configDir: 'test', logger: getLogger({}, 'noop')}, new EventEmitter());
 
         it('Should parse PlayProgress with Audio ItemType as valid event', async function () {
             const fixture = dataAsFixture(samplePayload[0]);
diff --git a/src/backend/tests/player/player.test.ts b/src/backend/tests/player/player.test.ts
index 27f8aade..c00cdb70 100644
--- a/src/backend/tests/player/player.test.ts
+++ b/src/backend/tests/player/player.test.ts
@@ -8,7 +8,7 @@ import { playObjDataMatch } from "../../utils.js";
 import dayjs from "dayjs";
 import clone from "clone";
 
-const logger = getLogger({});
+const logger = getLogger({}, 'noop');
 
 const newPlay = generatePlay({duration: 300});
 
diff --git a/src/backend/tests/scrobbler/TestScrobbler.ts b/src/backend/tests/scrobbler/TestScrobbler.ts
index 5acfe4be..355dae61 100644
--- a/src/backend/tests/scrobbler/TestScrobbler.ts
+++ b/src/backend/tests/scrobbler/TestScrobbler.ts
@@ -8,7 +8,7 @@ import request from "superagent";
 export class TestScrobbler extends AbstractScrobbleClient {
 
     constructor() {
-        const logger = getLogger({});
+        const logger = getLogger({}, 'noop');
         const notifier = new Notifiers(new EventEmitter(), new EventEmitter(), new EventEmitter());
         super('test', 'Test', {name: 'test'}, notifier, new EventEmitter(), logger);
     }