chore(all): npm libraries bump (#215)

* chore(all): npm libraries bump

* build(logger,tracer): added Error typings to test

* fix(logger,metrics): handle errors as unknown types

* fix: move coverage creation in GA - PR event

* fix: create lcov.info file in GA - PR build event

* fix: check for lcov file to exist in PR build

Author: saragerion

.github/workflows/on-pull-request.yml

@@ -24,8 +24,12 @@ jobs:
       - name: Collate Coverage Reports
         if: ${{ github.actor != 'dependabot[bot]' }}
         run: |
-          mkdir -p coverage
           for d in ./packages/*/ ; do
+              mkdir -p coverage
+              if [[ ! -f coverage/lcov.info ]]
+              then
+                  continue
+              fi
               filename="$d""coverage/lcov.info"
               targetSource="SF:""$d""src"
               sed "s|SF:src|$targetSource|g" $filename >> coverage/lcov.info

npm-shrinkwrap.json

@@ -162,9 +162,9 @@ class Logger implements ClassThatLogs {
     if (typeof input !== 'string') {
       logItem.addAttributes(input);
     }
-    extraInput.forEach((item: Error | LogAttributes) => {
+    extraInput.forEach((item: Error | LogAttributes | unknown) => {
       const attributes = (item instanceof Error) ? { error: item } : item;
-      logItem.addAttributes(attributes);
+      logItem.addAttributes(<LogAttributes>attributes);
     });
 
     return logItem;

packages/logger/npm-shrinkwrap.json

@@ -109,7 +109,8 @@ describe('Class: PowertoolLogFormatter', () => {
         shouldThrow();
       } catch (error) {
         // Assess
-        const formattedError = formatter.formatError(error);
+        expect(error).toBeInstanceOf(Error);
+        const formattedError = formatter.formatError(<Error>error);
         expect(formattedError).toEqual({
           location: expect.stringMatching(/PowertoolLogFormatter.test.ts:[0-9]+$/),
           message: 'Ouch!',
@@ -136,7 +137,8 @@ describe('Class: PowertoolLogFormatter', () => {
         shouldThrow();
       } catch (error) {
         // Assess
-        const formattedReferenceError = formatter.formatError(error);
+        expect(error).toBeInstanceOf(Error);
+        const formattedReferenceError = formatter.formatError(<Error>error);
         expect(formattedReferenceError).toEqual({
           location: expect.stringMatching(/PowertoolLogFormatter.test.ts:[0-9]+$/),
           message: 'doesNotExist is not defined',
@@ -164,7 +166,8 @@ describe('Class: PowertoolLogFormatter', () => {
         shouldThrow();
       } catch (error) {
         // Assess
-        const formattedAssertionError = formatter.formatError(error);
+        expect(error).toBeInstanceOf(AssertionError);
+        const formattedAssertionError = formatter.formatError(<AssertionError>error);
         expect(formattedAssertionError).toEqual({
           location: expect.stringMatching(/PowertoolLogFormatter.test.ts:[0-9]+/),
           message: expect.stringMatching(/Expected values to be strictly equal/),
@@ -192,7 +195,8 @@ describe('Class: PowertoolLogFormatter', () => {
         shouldThrow();
       } catch (error) {
         // Assess
-        const formattedRangeError = formatter.formatError(error);
+        expect(error).toBeInstanceOf(RangeError);
+        const formattedRangeError = formatter.formatError(<RangeError>error);
         expect(formattedRangeError).toEqual({
           location: expect.stringMatching(/PowertoolLogFormatter.test.ts:[0-9]+/),
           message: 'The argument must be between 10 and 20',
@@ -219,7 +223,8 @@ describe('Class: PowertoolLogFormatter', () => {
         shouldThrow();
       } catch (error) {
         // Assess
-        const formattedSyntaxError = formatter.formatError(error);
+        expect(error).toBeInstanceOf(SyntaxError);
+        const formattedSyntaxError = formatter.formatError(<SyntaxError>error);
         expect(formattedSyntaxError).toEqual({
           location: expect.stringMatching(/PowertoolLogFormatter.test.ts:[0-9]+/),
           message: 'Unexpected identifier',
@@ -247,7 +252,8 @@ describe('Class: PowertoolLogFormatter', () => {
         shouldThrow();
       } catch (error) {
         // Assess
-        const formattedTypeError = formatter.formatError(error);
+        expect(error).toBeInstanceOf(Error);
+        const formattedTypeError = formatter.formatError(<Error>error);
         expect(formattedTypeError).toEqual({
           location: expect.stringMatching(/PowertoolLogFormatter.test.ts:[0-9]+/),
           message: 'Cannot read property \'foo\' of null',

packages/logger/src/Logger.ts

@@ -50,7 +50,7 @@ type UnformattedAttributes = {
 };
 
 type LogItemMessage = string | LogAttributesWithMessage;
-type LogItemExtraInput = Array<Error | LogAttributes>;
+type LogItemExtraInput = Array<Error | LogAttributes | unknown>;
 
 type HandlerMethodDecorator = (target: LambdaInterface, propertyKey: string | symbol, descriptor: TypedPropertyDescriptor<Handler>) => TypedPropertyDescriptor<Handler> | void;