fix: Fix logger used for tests to always be 'noop'

Cleans up logging output during tests
FoxxMD · Feb 19, 2024 · ec897b1 · ec897b1
1 parent 69e15b3
commit ec897b1
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git 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
@@ -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
@@ -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);
     }