refactor(logger): rename args to splat (#119)

artusjs · Jun 28, 2022 · e61aa00 · e61aa00
1 parent 22fed48
commit e61aa00
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 23 deletions.
diff --git a/src/logger/impl.ts b/src/logger/impl.ts
@@ -54,32 +54,32 @@ export default class ArtusLogger extends BaseLogger {
   public log({
     level,
     message,
-    args = []
+    splat = []
   }: LogOptions) {
     if (message instanceof Error) {
       if (level === LoggerLevel.ERROR) {
-        return this.error(message, ...args);
+        return this.error(message, ...splat);
       }
       message = message.stack ?? message.message;
     }
     switch (level) {
       case LoggerLevel.TRACE:
-        this.trace(message, ...args);
+        this.trace(message, ...splat);
         break;
       case LoggerLevel.DEBUG:
-        this.debug(message, ...args);
+        this.debug(message, ...splat);
         break;
       case LoggerLevel.INFO:
-        this.info(message, ...args);
+        this.info(message, ...splat);
         break;
       case LoggerLevel.WARN:
-        this.warn(message, ...args);
+        this.warn(message, ...splat);
         break;
       case LoggerLevel.ERROR:
-        this.error(message, ...args);
+        this.error(message, ...splat);
         break;
       case LoggerLevel.FATAL:
-        this.fatal(message, ...args);
+        this.fatal(message, ...splat);
         break;
     }
   }

diff --git a/src/logger/types.ts b/src/logger/types.ts
@@ -7,7 +7,7 @@ export interface LoggerOptions {
 export interface LogOptions {
   level: LoggerLevel;
   message: string|Error;
-  args?: any[];
+  splat?: any[];
 }
 
 export interface Logger {

diff --git a/test/fixtures/logger/src/test_clazz.ts b/test/fixtures/logger/src/test_clazz.ts
@@ -6,11 +6,11 @@ export default class TestLoggerClazz {
   @Inject()
   private logger!: ArtusLogger;
 
-  public testLog(level: LoggerLevel, message: string|Error, ...args: any[]) {
+  public testLog(level: LoggerLevel, message: string|Error, ...splat: any[]) {
     this.logger.log({
       level,
       message,
-      args,
+      splat,
     });
   }
 

diff --git a/test/logger.test.ts b/test/logger.test.ts
@@ -61,42 +61,42 @@ describe('test/logger.test.ts', () => {
       app.logger.log({
         level: LoggerLevel.TRACE,
         message: 'trace', 
-        args: [ 0, {} ]
+        splat: [ 0, {} ]
       });
       expect(console.trace).toBeCalledTimes(0);
 
       app.logger.log({
         level: LoggerLevel.DEBUG,
         message: 'debug',
-        args: [ 1, {} ]
+        splat: [ 1, {} ]
       });
       expect(console.debug).toBeCalledTimes(0);
 
       app.logger.log({
         level: LoggerLevel.INFO,
         message: 'info',
-        args: [ 2, {} ]
+        splat: [ 2, {} ]
       });
       expect(console.info).toBeCalledTimes(1);
       expect(console.info).toBeCalledWith('info', 2, {});
 
       app.logger.log({
         level: LoggerLevel.WARN,
         message: 'warn',
-        args: [ 3, {} ]
+        splat: [ 3, {} ]
       });
       expect(console.warn).toBeCalledTimes(1);
       expect(console.warn).toBeCalledWith('warn', 3, {});
 
       app.logger.log({
         level: LoggerLevel.ERROR,
         message: 'error',
-        args: [ 4, {} ]
+        splat: [ 4, {} ]
       });
       app.logger.log({
         level: LoggerLevel.ERROR,
         message: err,
-        args: [ 5, {} ]
+        splat: [ 5, {} ]
       });
       expect(console.error).toBeCalledTimes(2);
       expect(console.error).toBeCalledWith('error', 4, {});
@@ -198,44 +198,44 @@ describe('test/logger.test.ts', () => {
       app.logger.log({
         level: LoggerLevel.TRACE,
         message: 'trace', 
-        args: [ 0, {} ]
+        splat: [ 0, {} ]
       });
       expect(console.trace).toBeCalledTimes(1);
       expect(console.trace).toBeCalledWith('trace', 0, {});
 
       app.logger.log({
         level: LoggerLevel.DEBUG,
         message: 'debug',
-        args: [ 1, {} ]
+        splat: [ 1, {} ]
       });
       expect(console.debug).toBeCalledTimes(1);
       expect(console.debug).toBeCalledWith('debug', 1, {});
 
       app.logger.log({
         level: LoggerLevel.INFO,
         message: 'info',
-        args: [ 2, {} ]
+        splat: [ 2, {} ]
       });
       expect(console.info).toBeCalledTimes(1);
       expect(console.info).toBeCalledWith('info', 2, {});
 
       app.logger.log({
         level: LoggerLevel.WARN,
         message: 'warn',
-        args: [ 3, {} ]
+        splat: [ 3, {} ]
       });
       expect(console.warn).toBeCalledTimes(1);
       expect(console.warn).toBeCalledWith('warn', 3, {});
 
       app.logger.log({
         level: LoggerLevel.ERROR,
         message: 'error',
-        args: [ 4, {} ]
+        splat: [ 4, {} ]
       });
       app.logger.log({
         level: LoggerLevel.ERROR,
         message: err,
-        args: [ 5, {} ]
+        splat: [ 5, {} ]
       });
       expect(console.error).toBeCalledTimes(2);
       expect(console.error).toBeCalledWith('error', 4, {});