chore(logger): lint fix

Author: saragerion

Diff for: packages/logger/src/Logger.ts

@@ -178,20 +178,6 @@ class Logger extends Utility implements ClassThatLogs {
     merge(this.persistentLogAttributes, attributes);
   }
 
-  /**
-   * It removes attributes based on provided keys to all log items generated by this Logger instance.
-   *
-   * @param {string[]} keys
-   * @returns {void}
-   */
-  public removePersistentLogAttributes(keys: string[]): void {
-    keys.forEach((key) => {
-      if(this.persistentLogAttributes && key in this.persistentLogAttributes) {
-        delete this.persistentLogAttributes[key];
-      }
-    })
-  }
-
   /**
    * Alias for addPersistentLogAttributes.
    *
@@ -202,16 +188,6 @@ class Logger extends Utility implements ClassThatLogs {
     this.addPersistentLogAttributes(attributes);
   }
 
-  /**
-   * Alias for removePersistentLogAttributes.
-   *
-   * @param {string[]} keys
-   * @returns {void}
-   */
-  public removeKeys(keys: string[]): void {
-    this.removePersistentLogAttributes(keys);
-  }
-
   /**
    * It creates a separate Logger instance, identical to the current one
    * It's possible to overwrite the new instance options by passing them.
@@ -298,6 +274,30 @@ class Logger extends Utility implements ClassThatLogs {
     this.setLogsSampled();
   }
 
+  /**
+   * Alias for removePersistentLogAttributes.
+   *
+   * @param {string[]} keys
+   * @returns {void}
+   */
+  public removeKeys(keys: string[]): void {
+    this.removePersistentLogAttributes(keys);
+  }
+
+  /**
+   * It removes attributes based on provided keys to all log items generated by this Logger instance.
+   *
+   * @param {string[]} keys
+   * @returns {void}
+   */
+  public removePersistentLogAttributes(keys: string[]): void {
+    keys.forEach((key) => {
+      if (this.persistentLogAttributes && key in this.persistentLogAttributes) {
+        delete this.persistentLogAttributes[key];
+      }
+    });
+  }
+
   /**
    * It sets the user-provided sample rate value.
    *

Diff for: packages/logger/tests/e2e/basicFeatures.middy.test.FunctionCode.ts

@@ -24,7 +24,7 @@ const testFunction = async (event: APIGatewayProxyEvent, context: Context): Prom
   // Test feature 2: Context data
   // Test feature 3: Add and remove persistent additional log keys and value
   // Test feature 4: X-Ray Trace ID injection
-  logger.removeKeys([ REMOVABLE_KEY ]);
+  logger.removeKeys([REMOVABLE_KEY]);
   logger.debug('##### This should not appear');
   logger.info('This is an INFO log with context and persistent key');
 

Diff for: packages/logger/tests/e2e/basicFeatures.middy.test.ts

@@ -43,8 +43,8 @@ const lambdaFunctionCodeFile = 'basicFeatures.middy.test.FunctionCode.ts';
 // Text to be used by Logger in the Lambda function
 const PERSISTENT_KEY = 'persistentKey';
 const PERSISTENT_VALUE = `a persistent value that will be put in every log ${uuid}`;
-const REMOVABLE_KEY = "removableKey";
-const REMOVABLE_VALUE = "a persistent value that will be removed and not displayed in any log ${uuid}";
+const REMOVABLE_KEY = 'removableKey';
+const REMOVABLE_VALUE = 'a persistent value that will be removed and not displayed in any log ${uuid}';
 const SINGLE_LOG_ITEM_KEY = `keyForSingleLogItem${uuid}`;
 const SINGLE_LOG_ITEM_VALUE = `a value for a single log item${uuid}`;
 const ERROR_MSG = `error-${uuid}`;

Diff for: packages/logger/tests/unit/Logger.test.ts

@@ -720,7 +720,7 @@ describe('Class: Logger', () => {
       });
 
       // Act
-      logger.removeKeys(["aws_account_id", "aws_region"]);
+      logger.removeKeys([ 'aws_account_id', 'aws_region' ]);
 
       // Assess
       expect(logger).toEqual(expect.objectContaining({
@@ -748,7 +748,7 @@ describe('Class: Logger', () => {
       const loggerBeforeKeysAreRemoved = { ...logger };
 
       // Act
-      logger.removeKeys(["not_existing_key"]);
+      logger.removeKeys(['not_existing_key']);
 
       // Assess
       expect(logger).toEqual(loggerBeforeKeysAreRemoved);
@@ -780,8 +780,8 @@ describe('Class: Logger', () => {
     });
 
     // Act
-    logger.removeKeys(["aws_account_id", "aws_region"]);
-    logger.removeKeys(["aws_account_id", "aws_region"]);
+    logger.removeKeys([ 'aws_account_id', 'aws_region' ]);
+    logger.removeKeys([ 'aws_account_id', 'aws_region' ]);
 
     // Assess
     expect(logger).toEqual(expect.objectContaining({