Add failing unit tests

Author: okoskine

packages/logger/tests/unit/Logger.test.ts

@@ -439,26 +439,42 @@ describe('Class: Logger', () => {
 
   describe('Method: addContext', () => {
 
+    const context1 = {
+      callbackWaitsForEmptyEventLoop: true,
+      functionVersion: '$LATEST',
+      functionName: 'foo-bar-function-with-cold-start',
+      memoryLimitInMB: '128',
+      logGroupName: '/aws/lambda/foo-bar-function-with-cold-start',
+      logStreamName: '2021/03/09/[$LATEST]abcdef123456abcdef123456abcdef123456',
+      invokedFunctionArn: 'arn:aws:lambda:eu-central-1:123456789012:function:foo-bar-function-with-cold-start',
+      awsRequestId: 'c6af9ac6-7b61-11e6-9a41-93e812345678',
+      getRemainingTimeInMillis: () => 1234,
+      done: () => console.log('Done!'),
+      fail: () => console.log('Failed!'),
+      succeed: () => console.log('Succeeded!'),
+    };
+    const context2 = {
+      callbackWaitsForEmptyEventLoop: true,
+      functionVersion: '$LATEST',
+      functionName: 'foo-bar-function-with-cold-start',
+      memoryLimitInMB: '128',
+      logGroupName: '/aws/lambda/foo-bar-function-with-cold-start',
+      logStreamName: '2021/03/09/[$LATEST]abcdef123456abcdef123456abcdef123456',
+      invokedFunctionArn: 'arn:aws:lambda:eu-central-1:123456789012:function:foo-bar-function-with-cold-start',
+      awsRequestId: 'd40c98a9-91c4-478c-a179-433c4b978289',
+      getRemainingTimeInMillis: () => 1234,
+      done: () => console.log('Done!'),
+      fail: () => console.log('Failed!'),
+      succeed: () => console.log('Succeeded!'),
+    };
+
     test('when called during a COLD START invocation, it populates the logger\'s PowertoolLogData object with coldstart set to true', () => {
 
       // Prepare
       const logger = new Logger();
 
       // Act
-      logger.addContext( {
-        callbackWaitsForEmptyEventLoop: true,
-        functionVersion: '$LATEST',
-        functionName: 'foo-bar-function-with-cold-start',
-        memoryLimitInMB: '128',
-        logGroupName: '/aws/lambda/foo-bar-function-with-cold-start',
-        logStreamName: '2021/03/09/[$LATEST]abcdef123456abcdef123456abcdef123456',
-        invokedFunctionArn: 'arn:aws:lambda:eu-central-1:123456789012:function:foo-bar-function-with-cold-start',
-        awsRequestId: 'c6af9ac6-7b61-11e6-9a41-93e812345678',
-        getRemainingTimeInMillis: () => 1234,
-        done: () => console.log('Done!'),
-        fail: () => console.log('Failed!'),
-        succeed: () => console.log('Succeeded!'),
-      });
+      logger.addContext(context1);
 
       // Assess
       expect(logger).toEqual({
@@ -492,6 +508,47 @@ describe('Class: Logger', () => {
       });
     });
 
+    test('when called with another request id, it sets cold start to false', () => {
+
+      // Prepare
+      const logger = new Logger();
+
+      // Act
+      logger.addContext(context1); // first invocation
+      logger.addContext(context2); // second invocation
+
+      // Assess
+      expect(logger).toEqual(
+        expect.objectContaining({
+          powertoolLogData: expect.objectContaining({
+            lambdaContext: expect.objectContaining({
+              coldStart: false
+            })
+          })
+        })
+      );
+    });
+
+    test('when called multiple times, it obeys the newest context values', () => {
+
+      // Prepare
+      const logger = new Logger();
+
+      // Act
+      logger.addContext(context1); // first invocation
+      logger.addContext(context2); // second invocation
+
+      // Assess
+      expect(logger).toEqual(
+        expect.objectContaining({
+          powertoolLogData: expect.objectContaining({
+            lambdaContext: expect.objectContaining({
+              awsRequestId: context2.awsRequestId,
+            })
+          })
+        })
+      );
+    });
   });
 
   describe('Method: appendKeys', () => {
@@ -523,6 +580,47 @@ describe('Class: Logger', () => {
         },
       }));
     });
+
+    test('user-provided attribute object is not mutated', () => {
+
+      // Prepare
+      const logger = new Logger();
+      const attributes1 = { keyOne: 'abc' };
+      const attributes2 = { keyTwo: 'def' };
+
+      // Act
+      logger.appendKeys(attributes1);
+      logger.appendKeys(attributes2);
+
+      // Assess
+      expect(attributes1).toEqual({ keyOne: 'abc' });
+      expect(attributes2).toEqual({ keyTwo: 'def' });
+    });
+
+    test('when called multiple times with same key, the last value overrides earlier values', () => {
+
+      // Prepare
+      const logger = new Logger();
+
+      // Act
+      logger.appendKeys({
+        keyOne: 'abc',
+        duplicateKey: 'one'
+      });
+      logger.appendKeys({
+        keyTwo: 'def',
+        duplicateKey: 'two'
+      });
+
+      // Assess
+      expect(logger).toEqual(expect.objectContaining({
+        persistentLogAttributes: {
+          keyOne: 'abc',
+          keyTwo: 'def',
+          duplicateKey: 'two'
+        }
+      }));
+    });
   });
 
   describe('Method: createChild', () => {