build: upgrade typescript-related package versions

Author: onebytegone

package-lock.json

@@ -31,12 +31,12 @@
       "@silvermine/chai-strictly-equal": "1.1.1",
       "@silvermine/eslint-config": "3.1.0-beta.0",
       "@silvermine/standardization": "2.0.0",
-      "@silvermine/typescript-config": "0.9.0",
+      "@silvermine/typescript-config": "1.0.0",
       "@types/aws-lambda": "8.10.17",
       "@types/chai": "4.1.7",
       "@types/cookie": "0.3.2",
       "@types/mocha": "5.2.5",
-      "@types/node": "8.10.36",
+      "@types/node": "20.12.2",
       "@types/qs": "6.5.1",
       "@types/sinon": "5.0.5",
       "@types/underscore": "1.8.9",
@@ -56,7 +56,7 @@
       "source-map-support": "0.5.9",
       "standard-version": "git+https://github.com/jthomerson/standard-version.git#fix-305-header-repeat",
       "ts-node": "7.0.1",
-      "typescript": "3.2.2"
+      "typescript": "4.7.4"
    },
    "dependencies": {
       "@silvermine/toolbox": "0.1.0",

package.json

@@ -1,7 +1,7 @@
 import { Callback, Context } from 'aws-lambda';
 import Router from './Router';
 import { RequestEvent, HandlerContext } from './request-response-types';
-import { StringUnknownMap, Writable } from '@silvermine/toolbox';
+import { isUndefined, StringUnknownMap, Writable } from '@silvermine/toolbox';
 import { Request, Response } from '.';
 import _ from 'underscore';
 import { isErrorWithStatusCode } from './interfaces';
@@ -102,28 +102,26 @@ export default class Application extends Router {
    }
 
    private _createHandlerContext(context: Context): HandlerContext {
-      // keys should exist on both `HandlerContext` and `Context`
-      const keys: (keyof HandlerContext & keyof Context)[] = [
-         'functionName', 'functionVersion', 'invokedFunctionArn', 'memoryLimitInMB',
-         'awsRequestId', 'logGroupName', 'logStreamName', 'identity', 'clientContext',
-         'getRemainingTimeInMillis',
-      ];
-
-      let handlerContext: Writable<HandlerContext>;
-
-      handlerContext = _.reduce(keys, (memo, key) => {
-         let contextValue = context[key];
+      const newContext: Writable<HandlerContext> = {
+         functionName: context.functionName,
+         functionVersion: context.functionVersion,
+         invokedFunctionArn: context.invokedFunctionArn,
+         memoryLimitInMB: context.memoryLimitInMB,
+         awsRequestId: context.awsRequestId,
+         logGroupName: context.logGroupName,
+         logStreamName: context.logStreamName,
+         getRemainingTimeInMillis: context.getRemainingTimeInMillis,
+      };
+
+      if (!isUndefined(context.identity)) {
+         newContext.identity = Object.freeze({ ...context.identity });
+      }
 
-         if (typeof contextValue === 'object' && contextValue) {
-            // Freeze sub-objects
-            memo[key] = Object.freeze(_.extend({}, contextValue));
-         } else if (typeof contextValue !== 'undefined') {
-            memo[key] = contextValue;
-         }
-         return memo;
-      }, {} as Writable<HandlerContext>);
+      if (!isUndefined(context.clientContext)) {
+         newContext.clientContext = Object.freeze({ ...context.clientContext });
+      }
 
-      return Object.freeze(handlerContext);
+      return Object.freeze(newContext);
    }
 
 }

src/Application.ts

@@ -11,7 +11,7 @@ import {
    apiGatewayRequestRawQuery,
    albMultiValHeadersRawQuery,
 } from './samples';
-import { isKeyValueStringObject } from '@silvermine/toolbox';
+import { isKeyValueStringObject, Optional } from '@silvermine/toolbox';
 import ConsoleLogger from '../src/logging/ConsoleLogger';
 import sinon from 'sinon';
 import { DebugLogObject } from '../src/logging/logging-types';
@@ -44,13 +44,16 @@ describe('Request', () => {
          expect(new Request(app, albRequest(), handlerContext()).method).to.strictlyEqual('GET');
          expect(new Request(app, _.extend({}, albRequest(), { httpMethod: 'get' }), handlerContext()).method).to.strictlyEqual('GET');
          expect(new Request(app, _.extend({}, albRequest(), { httpMethod: 'PoSt' }), handlerContext()).method).to.strictlyEqual('POST');
+      });
 
-         // make sure that undefined values don't break it:
-         let evt2: RequestEvent = albRequest();
+      // TODO: Why is this test here? It was added during initial implementation but none
+      // of the types agree with this test.
+      it('an undefined `httpMethod` doesn\'t break the setting of `method`', () => {
+         let evt2: Optional<RequestEvent, 'httpMethod'> = albRequest();
 
          delete evt2.httpMethod;
          expect(evt2.httpMethod).to.strictlyEqual(undefined);
-         expect(new Request(app, evt2, handlerContext()).method).to.strictlyEqual('');
+         expect(new Request(app, evt2 as RequestEvent, handlerContext()).method).to.strictlyEqual('');
       });
 
       it('sets URL related fields correctly, when created from an event', () => {

tests/Request.test.ts

@@ -8,7 +8,7 @@ import {
 } from './samples';
 import { spy, SinonSpy, assert } from 'sinon';
 import { Application, Request, Response, Router } from '../src';
-import { RequestEvent } from '../src/request-response-types';
+import { RequestEvent, ResponseResult } from '../src/request-response-types';
 import { NextCallback, IRoute, IRouter, ErrorWithStatusCode } from '../src/interfaces';
 import { expect } from 'chai';
 import { StringArrayOfStringsMap, StringMap, KeyValueStringObject } from '@silvermine/toolbox';
@@ -369,20 +369,21 @@ describe('integration tests', () => {
 
                app.run(evt, handlerContext(), cb);
 
-               const expectedCallbackValue = {
+               const expectedCallbackValue: ResponseResult = {
                   statusCode: code,
                   statusDescription: desc,
                   body: expectedBody,
                   isBase64Encoded: false,
-                  headers: {
-                     'X-Did-Run-All-Hello-World': 'true',
-                  } as StringMap,
                   multiValueHeaders: {
                      'X-Did-Run-All-Hello-World': [ 'true' ],
-                  } as StringArrayOfStringsMap,
+                  },
+               };
+
+               expectedCallbackValue.headers = {
+                  'X-Did-Run-All-Hello-World': 'true',
+                  [hdrName]: hdrVal,
                };
 
-               expectedCallbackValue.headers[hdrName] = hdrVal;
                expectedCallbackValue.multiValueHeaders[hdrName] = [ hdrVal ];
 
                if (contentType) {