From 6b4d4ea29f5a37a4b9412829166d04594d4d3982 Mon Sep 17 00:00:00 2001 From: Bojan Kucera Date: Sat, 7 Jun 2025 11:04:35 -0400 Subject: [PATCH] fixed logger tests --- libs/logger/test/advanced.test.ts | 20 +++++++++++--------- libs/logger/test/basic.test.ts | 10 +++++++--- libs/logger/test/integration.test.ts | 6 +++++- 3 files changed, 23 insertions(+), 13 deletions(-) diff --git a/libs/logger/test/advanced.test.ts b/libs/logger/test/advanced.test.ts index 9ed5632..58f5cf5 100644 --- a/libs/logger/test/advanced.test.ts +++ b/libs/logger/test/advanced.test.ts @@ -1,18 +1,22 @@ /** - * Logger Basic Tests + * Advanced Logger Tests * - * Simple tests for the simplified logger functionality. + * Tests for advanced logger functionality. */ -import { describe, it, expect, beforeEach } from 'bun:test'; +import { describe, it, expect, beforeEach, afterEach } from 'bun:test'; import { Logger, getLogger } from '../src'; import { loggerTestHelpers } from './setup'; -describe('Logger Basic Tests', () => { +describe('Advanced Logger Tests', () => { let logger: Logger; beforeEach(() => { - logger = loggerTestHelpers.createTestLogger('basic-test'); + loggerTestHelpers.clearCapturedLogs(); + logger = loggerTestHelpers.createTestLogger('advanced-test'); + }); + + afterEach(() => { loggerTestHelpers.clearCapturedLogs(); }); @@ -32,14 +36,12 @@ describe('Logger Basic Tests', () => { expect(logs.length).toBe(1); expect(logs[0].msg).toBe('Test message'); expect(logs[0].level).toBe('info'); - }); - - it('should include service name in logs', () => { + }); it('should include service name in logs', () => { logger.info('Service test'); const logs = loggerTestHelpers.getCapturedLogs(); expect(logs.length).toBe(1); - expect(logs[0].service).toBe('basic-test'); + expect(logs[0].service).toBe('advanced-test'); }); it('should handle multiple log levels', () => { diff --git a/libs/logger/test/basic.test.ts b/libs/logger/test/basic.test.ts index 8ba68aa..2beeef9 100644 --- a/libs/logger/test/basic.test.ts +++ b/libs/logger/test/basic.test.ts @@ -1,18 +1,22 @@ /** - * Logger Utility Tests + * Basic Logger Tests * * Tests for the core logger functionality and utilities. */ -import { describe, it, expect, beforeEach } from 'bun:test'; +import { describe, it, expect, beforeEach, afterEach } from 'bun:test'; import { Logger, getLogger, createLogger } from '../src'; import { loggerTestHelpers } from './setup'; -describe('Logger Utilities', () => { +describe('Basic Logger Tests', () => { let logger: Logger; beforeEach(() => { + loggerTestHelpers.clearCapturedLogs(); logger = loggerTestHelpers.createTestLogger('utils-test'); + }); + + afterEach(() => { loggerTestHelpers.clearCapturedLogs(); }); diff --git a/libs/logger/test/integration.test.ts b/libs/logger/test/integration.test.ts index 96a0832..8aaccc9 100644 --- a/libs/logger/test/integration.test.ts +++ b/libs/logger/test/integration.test.ts @@ -4,7 +4,7 @@ * Tests the core functionality of the simplified @stock-bot/logger package. */ -import { describe, it, expect, beforeEach } from 'bun:test'; +import { describe, it, expect, beforeEach, afterEach } from 'bun:test'; import { Logger, createLogger, @@ -17,7 +17,11 @@ describe('Logger Integration Tests', () => { beforeEach(() => { // Create a new test logger before each test + loggerTestHelpers.clearCapturedLogs(); logger = loggerTestHelpers.createTestLogger('integration-test'); + }); + + afterEach(() => { loggerTestHelpers.clearCapturedLogs(); });