Merge pull request #138 from arithmetric/support_node16

Support Node.js 16 runtime for AWS Lambda

Author: arithmetric

.github/workflows/lint-test.yaml

@@ -6,6 +6,7 @@ on:
     branches:
       - master
       - github_actions
+      - support_node16
   pull_request:
 
 jobs:
@@ -15,7 +16,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        nodejs_version: [ "8.10", 10, 12 ]
+        nodejs_version: [ "8.10", 10, 12, 14, 16, 18 ]
 
     steps:
       - name: Checkout

.travis.yml

@@ -1,11 +1,11 @@
 language: node_js
 node_js:
-  - "8.10"
-  - 10
   - 12
+  - 14
+  - 16
 matrix:
   allow_failures:
-  - node_js: "8.10"
+  - node_js: "12"
 script:
   - npm run lint
   - npm test

README.md

@@ -59,7 +59,7 @@ the email forwarding mapping from original destinations to new destination.
 2. In AWS Lambda, add a new function and skip selecting a blueprint.
 
  - Name the function "SesForwarder" and optionally give it a description. Ensure
- Runtime is set to Node.js 12.x. (Node.js 10.x is also supported.)
+ Runtime is set to Node.js 18.x or Node.js 16.x.
 
  - For the Lambda function code, either copy and paste the contents of
  `index.js` into the inline code editor or zip the contents of the repository

index.js

@@ -377,8 +377,9 @@ exports.handler = function(event, context, callback, overrides) {
 Promise.series = function(promises, initValue) {
   return promises.reduce(function(chain, promise) {
     if (typeof promise !== 'function') {
-      return Promise.reject(new Error("Error: Invalid promise item: " +
-        promise));
+      return chain.then(() => {
+        throw new Error("Error: Invalid promise item: " + promise);
+      });
     }
     return chain.then(promise);
   }, Promise.resolve(initValue));

package.json

@@ -9,7 +9,7 @@
     "test": "nyc --statements 100 -- mocha -- --check-leaks --timeout 3000"
   },
   "dependencies": {
-    "aws-sdk": "~2.631.0"
+    "aws-sdk": "~2.1083.0"
   },
   "devDependencies": {
     "coveralls": "^3.0.7",

test/handler.js

@@ -61,7 +61,8 @@ describe('index.js', function() {
       var event = {};
       var context = {};
       var callback = function(err) {
-        done(err ? null : true);
+        if (err) assert.ok(true, "callback function received error");
+        done();
       };
       var overrides = {
         steps: [