fixed logger tests

This commit is contained in:
Bojan Kucera 2025-06-07 11:04:35 -04:00
parent e76489e8f3
commit 6b4d4ea29f
3 changed files with 23 additions and 13 deletions

View file

@ -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 { Logger, getLogger } from '../src';
import { loggerTestHelpers } from './setup'; import { loggerTestHelpers } from './setup';
describe('Logger Basic Tests', () => { describe('Advanced Logger Tests', () => {
let logger: Logger; let logger: Logger;
beforeEach(() => { beforeEach(() => {
logger = loggerTestHelpers.createTestLogger('basic-test'); loggerTestHelpers.clearCapturedLogs();
logger = loggerTestHelpers.createTestLogger('advanced-test');
});
afterEach(() => {
loggerTestHelpers.clearCapturedLogs(); loggerTestHelpers.clearCapturedLogs();
}); });
@ -32,14 +36,12 @@ describe('Logger Basic Tests', () => {
expect(logs.length).toBe(1); expect(logs.length).toBe(1);
expect(logs[0].msg).toBe('Test message'); expect(logs[0].msg).toBe('Test message');
expect(logs[0].level).toBe('info'); expect(logs[0].level).toBe('info');
}); }); it('should include service name in logs', () => {
it('should include service name in logs', () => {
logger.info('Service test'); logger.info('Service test');
const logs = loggerTestHelpers.getCapturedLogs(); const logs = loggerTestHelpers.getCapturedLogs();
expect(logs.length).toBe(1); 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', () => { it('should handle multiple log levels', () => {

View file

@ -1,18 +1,22 @@
/** /**
* Logger Utility Tests * Basic Logger Tests
* *
* Tests for the core logger functionality and utilities. * 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 { Logger, getLogger, createLogger } from '../src';
import { loggerTestHelpers } from './setup'; import { loggerTestHelpers } from './setup';
describe('Logger Utilities', () => { describe('Basic Logger Tests', () => {
let logger: Logger; let logger: Logger;
beforeEach(() => { beforeEach(() => {
loggerTestHelpers.clearCapturedLogs();
logger = loggerTestHelpers.createTestLogger('utils-test'); logger = loggerTestHelpers.createTestLogger('utils-test');
});
afterEach(() => {
loggerTestHelpers.clearCapturedLogs(); loggerTestHelpers.clearCapturedLogs();
}); });

View file

@ -4,7 +4,7 @@
* Tests the core functionality of the simplified @stock-bot/logger package. * 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 { import {
Logger, Logger,
createLogger, createLogger,
@ -17,7 +17,11 @@ describe('Logger Integration Tests', () => {
beforeEach(() => { beforeEach(() => {
// Create a new test logger before each test // Create a new test logger before each test
loggerTestHelpers.clearCapturedLogs();
logger = loggerTestHelpers.createTestLogger('integration-test'); logger = loggerTestHelpers.createTestLogger('integration-test');
});
afterEach(() => {
loggerTestHelpers.clearCapturedLogs(); loggerTestHelpers.clearCapturedLogs();
}); });