fix(utils/log): Refactor ConsoleLogger to improve title formatting

equinor · Aug 27, 2024 · 177c7d6 · 177c7d6
1 parent 779adf6
commit 177c7d6
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 5 deletions.
diff --git a/packages/utils/log/src/ConsoleLogger.ts b/packages/utils/log/src/ConsoleLogger.ts
@@ -66,8 +66,13 @@ export class ConsoleLogger extends Logger {
     }
 
     protected _formatTitle(_lvl: LogLevel): string {
-        const title = [this.title, this.subtitle].filter((x) => !!x).join(' - ');
-        return chalk.magenta(title);
+        const title = chalk.magenta([this.title, this.subtitle].filter((x) => !!x).join(' - '));
+        switch (_lvl) {
+            case LogLevel.Warning:
+            case LogLevel.Error:
+                return chalk.bold(title);
+        }
+        return title;
     }
 
     /**

diff --git a/packages/utils/log/tests/ConsoleLogger.test.ts b/packages/utils/log/tests/ConsoleLogger.test.ts
@@ -11,7 +11,8 @@ describe('Console logger', () => {
         logger.level = LogLevel.Debug;
         logger.debug('This is a debug message');
         expect(spy).toHaveBeenCalledWith(
-            chalk.dim(chalk.magenta('MainLogger'), 'This is a debug message'),
+            chalk(chalk.magenta('MainLogger')),
+            'This is a debug message',
         );
     });
     it('should log info messages', () => {
@@ -28,7 +29,8 @@ describe('Console logger', () => {
         logger.level = LogLevel.Warning;
         logger.warn('This is a warning message');
         expect(spy).toHaveBeenCalledWith(
-            chalk.bold(chalk.magenta('MainLogger'), 'This is a warning message'),
+            chalk.bold.magenta('MainLogger'),
+            'This is a warning message',
         );
     });
 
@@ -37,7 +39,10 @@ describe('Console logger', () => {
         const logger = new ConsoleLogger('MainLogger');
         logger.level = LogLevel.Error;
         logger.error('This is an error message');
-        expect(spy).toHaveBeenCalledWith(chalk.magenta('MainLogger'), 'This is an error message');
+        expect(spy).toHaveBeenCalledWith(
+            chalk.bold.magenta('MainLogger'),
+            'This is an error message',
+        );
     });
 
     it('should log multiple messages', () => {