From e9c4b1f83dc58653abeef9fbfc41dcafaf990401 Mon Sep 17 00:00:00 2001 From: Javier Enrique Mendoza Gomez Date: Mon, 10 Jan 2022 19:21:48 +0000 Subject: [PATCH 1/2] test(logger): added unit test for URIError --- package-lock.json | 1 + .../formatter/PowertoolLogFormatter.test.ts | 28 +++++++++++++++++++ 2 files changed, 29 insertions(+) diff --git a/package-lock.json b/package-lock.json index 2a42c66138..eb95b10c05 100644 --- a/package-lock.json +++ b/package-lock.json @@ -5,6 +5,7 @@ "requires": true, "packages": { "": { + "name": "aws-lambda-powertools-typescript", "version": "0.0.1", "license": "MIT", "workspaces": [ diff --git a/packages/logger/tests/unit/formatter/PowertoolLogFormatter.test.ts b/packages/logger/tests/unit/formatter/PowertoolLogFormatter.test.ts index 0f7ae288e3..6e82fd41de 100644 --- a/packages/logger/tests/unit/formatter/PowertoolLogFormatter.test.ts +++ b/packages/logger/tests/unit/formatter/PowertoolLogFormatter.test.ts @@ -266,6 +266,34 @@ describe('Class: PowertoolLogFormatter', () => { expect(shouldThrow).toThrowError(expect.any(TypeError)); }); + test('When an error of type URIError is passed, it returns an object with expected structure and values', () => { + + // Prepare + const formatter = new PowertoolLogFormatter(); + const shouldThrow = (): void => { + // eslint-disable-next-line @typescript-eslint/ban-ts-comment + // @ts-ignore + decodeURIComponent('%'); + }; + + // Act + try { + shouldThrow(); + } catch (error) { + // Assess + expect(error).toBeInstanceOf(URIError); + const formattedURIError = formatter.formatError(error); + expect(formattedURIError).toEqual({ + location: expect.stringMatching(/PowertoolLogFormatter.test.ts:[0-9]+/), + message: 'URI malformed', + name: 'URIError', + stack: expect.stringMatching(/PowertoolLogFormatter.test.ts:[0-9]+:[0-9]+/), + }); + } + + expect(shouldThrow).toThrowError(expect.any(URIError)); + }); + }); describe('Method: formatTimestamp', () => { From 9fe2a6aa6d993ef6bba9c7ab936fab9d56822066 Mon Sep 17 00:00:00 2001 From: Javier Enrique Mendoza Gomez Date: Wed, 12 Jan 2022 18:08:36 +0000 Subject: [PATCH 2/2] chore(npm): Revert name addition --- package-lock.json | 1 - 1 file changed, 1 deletion(-) diff --git a/package-lock.json b/package-lock.json index 429941e310..aa38cbc19f 100644 --- a/package-lock.json +++ b/package-lock.json @@ -5,7 +5,6 @@ "requires": true, "packages": { "": { - "name": "aws-lambda-powertools-typescript", "version": "0.0.1", "license": "MIT", "workspaces": [