update test specs for api parser

Author: manwaring

package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "@manwaring/lambda-wrapper",
   "description": "A lambda handler wrapper to abstract common functionality and provide useful defaults",
-  "version": "3.5.0",
+  "version": "3.5.1",
   "scripts": {
     "publish-please-dry-run": "publish-please --dry-run",
     "publish-please": "publish-please",
@@ -22,18 +22,18 @@
     "@babel/preset-typescript": "^7.9.0",
     "@types/aws-lambda": "^8.10.51",
     "@types/cfn-response": "^1.0.3",
-    "@types/jest": "^25.2.1",
-    "@types/node": "^13.13.5",
+    "@types/jest": "^25.2.2",
+    "@types/node": "^14.0.1",
     "aws-lambda": "^1.0.6",
-    "aws-sdk": "^2.672.0",
+    "aws-sdk": "^2.678.0",
     "babel-jest": "^26.0.1",
     "codecov": "^3.6.5",
     "husky": "^4.2.5",
     "jest": "^26.0.1",
     "publish-please": "^5.5.1",
     "rimraf": "^3.0.2",
-    "serverless-plugin-test-helper": "^2.1.3",
-    "typescript": "^3.8.3"
+    "serverless-plugin-test-helper": "^2.3.0",
+    "typescript": "^3.9.2"
   },
   "husky": {
     "hooks": {

package.json

@@ -59,7 +59,7 @@ describe('Request parsing', () => {
       body: JSON.stringify({ hello: 'world' }),
       pathParameters: { proxy: 'not today' },
       queryStringParameters: { name: 'a test' },
-      headers: { 'content-type': 'application/json', 'Test-Request': 'true' }
+      headers: { 'content-type': 'application/json', 'Test-Request': 'true' },
     });
     const { body, path, query, auth, headers, testRequest } = new Request(event).getProperties();
 
@@ -68,7 +68,7 @@ describe('Request parsing', () => {
     expect(query).toEqual({ name: 'a test' });
     expect(headers['content-type']).toEqual('application/json');
     expect(testRequest).toEqual(true);
-    expect(auth).toBeFalsy();
+    expect(auth).toBeTruthy();
   });
 
   it("Get's falsy fields when optional parameters not used", () => {
@@ -82,7 +82,7 @@ describe('Request parsing', () => {
     expect(body).toBeFalsy();
     expect(path).toBeFalsy();
     expect(query).toBeFalsy();
-    expect(auth).toBeFalsy();
+    expect(auth).toBeTruthy();
     expect(headers).toBeFalsy();
     expect(testRequest).toBeFalsy();
   });

src/api/parser.test.ts

@@ -47,7 +47,7 @@ describe('API wrapper', () => {
       expect(query).toEqual({ name: 'a test' });
       expect(headers['content-type']).toEqual('application/json');
       expect(testRequest).toEqual(true);
-      expect(auth).toBeFalsy();
+      expect(auth).toBeTruthy();
       expect(success).toBeInstanceOf(Function);
       expect(notFound).toBeInstanceOf(Function);
       expect(notAuthorized).toBeInstanceOf(Function);
@@ -85,7 +85,7 @@ describe('API wrapper', () => {
       expect(query).toEqual({ name: 'a test' });
       expect(headers['content-type']).toEqual('application/json');
       expect(testRequest).toEqual(true);
-      expect(auth).toBeFalsy();
+      expect(auth).toBeTruthy();
       expect(success).toBeInstanceOf(Function);
       expect(notFound).toBeInstanceOf(Function);
       expect(notAuthorized).toBeInstanceOf(Function);

src/api/wrapper.test.ts

@@ -10,5 +10,9 @@ export function success(message: any): void {
 
 export function error(error: any): void {
   metrics.error(error);
-  throw new Error(error);
+  if (error instanceof Error) {
+    throw error;
+  } else {
+    throw new Error(error);
+  }
 }