From 8e1a9addc1f21b6ccf4bc35619f3f4424e176e1e Mon Sep 17 00:00:00 2001 From: Kacper Date: Fri, 2 Jan 2026 20:54:39 +0100 Subject: [PATCH] fix: update logger test expectations to include log level prefixes - Modified test cases in logger.test.ts to assert that log messages include appropriate log level prefixes (INFO, ERROR, WARN, DEBUG). - Ensured consistency in log output formatting across various logging methods, enhancing clarity in test results. --- libs/utils/tests/logger.test.ts | 28 +++++++++++++++------------- 1 file changed, 15 insertions(+), 13 deletions(-) diff --git a/libs/utils/tests/logger.test.ts b/libs/utils/tests/logger.test.ts index 4c9a9326..f058786b 100644 --- a/libs/utils/tests/logger.test.ts +++ b/libs/utils/tests/logger.test.ts @@ -35,7 +35,7 @@ describe('logger.ts', () => { logger.info('test message'); - expect(console.log).toHaveBeenCalledWith('[TestContext]', 'test message'); + expect(console.log).toHaveBeenCalledWith('INFO [TestContext]', 'test message'); }); it('should handle multiple arguments', () => { @@ -44,7 +44,7 @@ describe('logger.ts', () => { logger.info('message', { data: 123 }, [1, 2, 3]); - expect(console.log).toHaveBeenCalledWith('[Test]', 'message', { data: 123 }, [1, 2, 3]); + expect(console.log).toHaveBeenCalledWith('INFO [Test]', 'message', { data: 123 }, [1, 2, 3]); }); }); @@ -113,7 +113,9 @@ describe('logger.ts', () => { logger.error('error occurred', { code: 500 }); - expect(console.error).toHaveBeenCalledWith('[ErrorTest]', 'error occurred', { code: 500 }); + expect(console.error).toHaveBeenCalledWith('ERROR [ErrorTest]', 'error occurred', { + code: 500, + }); }); it('should not log when level is below ERROR', () => { @@ -133,7 +135,7 @@ describe('logger.ts', () => { logger.warn('warning message'); - expect(console.warn).toHaveBeenCalledWith('[WarnTest]', 'warning message'); + expect(console.warn).toHaveBeenCalledWith('WARN [WarnTest]', 'warning message'); }); it('should not log when level is below WARN', () => { @@ -153,7 +155,7 @@ describe('logger.ts', () => { logger.info('info message'); - expect(console.log).toHaveBeenCalledWith('[InfoTest]', 'info message'); + expect(console.log).toHaveBeenCalledWith('INFO [InfoTest]', 'info message'); }); it('should not log when level is below INFO', () => { @@ -173,7 +175,7 @@ describe('logger.ts', () => { logger.debug('debug details', { trace: '...' }); - expect(console.log).toHaveBeenCalledWith('[DebugTest]', '[DEBUG]', 'debug details', { + expect(console.log).toHaveBeenCalledWith('DEBUG [DebugTest]', 'debug details', { trace: '...', }); }); @@ -216,7 +218,7 @@ describe('logger.ts', () => { setLogLevel(LogLevel.INFO); logger.info('should log'); - expect(console.log).toHaveBeenCalledWith('[Test]', 'should log'); + expect(console.log).toHaveBeenCalledWith('INFO [Test]', 'should log'); }); }); @@ -229,8 +231,8 @@ describe('logger.ts', () => { logger1.info('from service 1'); logger2.info('from service 2'); - expect(console.log).toHaveBeenNthCalledWith(1, '[Service1]', 'from service 1'); - expect(console.log).toHaveBeenNthCalledWith(2, '[Service2]', 'from service 2'); + expect(console.log).toHaveBeenNthCalledWith(1, 'INFO [Service1]', 'from service 1'); + expect(console.log).toHaveBeenNthCalledWith(2, 'INFO [Service2]', 'from service 2'); }); it('should share log level setting', () => { @@ -253,7 +255,7 @@ describe('logger.ts', () => { logger.info('message'); - expect(console.log).toHaveBeenCalledWith('[]', 'message'); + expect(console.log).toHaveBeenCalledWith('INFO []', 'message'); }); it('should handle context with special characters', () => { @@ -262,7 +264,7 @@ describe('logger.ts', () => { logger.info('message'); - expect(console.log).toHaveBeenCalledWith('[Test-Service_v2.0]', 'message'); + expect(console.log).toHaveBeenCalledWith('INFO [Test-Service_v2.0]', 'message'); }); it('should handle no arguments to log methods', () => { @@ -271,7 +273,7 @@ describe('logger.ts', () => { logger.info(); - expect(console.log).toHaveBeenCalledWith('[Test]'); + expect(console.log).toHaveBeenCalledWith('INFO [Test]'); }); it('should handle complex object arguments', () => { @@ -286,7 +288,7 @@ describe('logger.ts', () => { logger.info('complex', complexObj); - expect(console.log).toHaveBeenCalledWith('[Test]', 'complex', complexObj); + expect(console.log).toHaveBeenCalledWith('INFO [Test]', 'complex', complexObj); }); }); });