@mapbox/cloudfriend 9.2.0 → 9.2.1

package/jest.config.js

@@ -0,0 +1,14 @@
+'use strict';
+
+module.exports = {
+  testEnvironment: 'node',
+  testMatch: ['**/test/*.test.js'],
+  collectCoverageFrom: [
+    'index.js',
+    'lib/**/*.js',
+    'bin/**/*.js',
+    '!**/node_modules/**'
+  ],
+  coverageDirectory: 'coverage',
+  verbose: true
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mapbox/cloudfriend",
-  "version": "9.2.0",
+  "version": "9.2.1",
   "description": "Helper functions for assembling CloudFormation templates in JavaScript",
   "main": "index.js",
   "engines": {
@@ -8,11 +8,10 @@
   },
   "scripts": {
     "pretest": "npm run lint && npm run shortcuts-api-doc",
-    "lint": "eslint index.js test lib bin cloudformation",
+    "lint": "eslint index.js test lib bin",
     "lint:fix": "npm run lint -- --fix",
-    "test": "nyc tape test/*.test.js | tap-spec",
-    "test:update": "UPDATE=1 npm run test",
-    "coverage": "nyc --reporter html tape test/*.test.js && opener coverage/index.html",
+    "test": "jest --coverage",
+    "test:update": "jest --updateSnapshot",
     "build-ci-template": "bin/build-template.js cloudformation/ci.template.js > cloudformation/ci.template.json",
     "shortcuts-api-doc": "jsdoc2md lib/shortcuts/*.js > lib/shortcuts/api.md"
   },
@@ -35,14 +34,11 @@
   },
   "homepage": "https://github.com/mapbox/cloudfriend#readme",
   "devDependencies": {
-    "@mapbox/eslint-config-mapbox": "^1.2.1",
-    "eslint": "^4.18.2",
+    "@mapbox/eslint-config-mapbox": "^5.0.1",
+    "eslint": "^8.57.0",
     "eslint-plugin-node": "^6.0.1",
-    "jsdoc-to-markdown": "^7.1.1",
-    "nyc": "^14.1.0",
-    "opener": "^1.4.1",
-    "tap-spec": "^5.0.0",
-    "tape": "^4.6.0"
+    "jest": "^29.7.0",
+    "jsdoc-to-markdown": "^7.1.1"
   },
   "dependencies": {
     "@aws-sdk/client-cloudformation": "^3.848.0",
@@ -50,6 +46,12 @@
     "redent": "^2.0.0"
   },
   "eslintConfig": {
-    "extends": "@mapbox/eslint-config-mapbox"
+    "extends": "@mapbox/eslint-config-mapbox",
+    "env": {
+      "jest": true
+    },
+    "ignorePatterns": [
+      "test/fixtures/malformed.json"
+    ]
   }
 }

package/readme.md

@@ -1,6 +1,6 @@
 # cloudfriend
 
-[![Build Status](https://travis-ci.com/mapbox/cloudfriend.svg?branch=master)](https://travis-ci.com/mapbox/cloudfriend)
+[![mapbox/cloudfriend](https://github.com/mapbox/cloudfriend/actions/workflows/test.yml/badge.svg)](https://github.com/mapbox/cloudfriend/actions/workflows/test.yml)
 
 Helper functions for assembling CloudFormation templates in JavaScript.
 

package/test/bin.test.js

@@ -1,22 +1,26 @@
 'use strict';
 
 const path = require('path');
-const exec = require('child_process').exec;
-const test = require('tape');
+const { execFile } = require('child_process');
+const { promisify } = require('util');
 
-test('bin/build-template', (t) => {
+const execFileAsync = promisify(execFile);
+
+describe('bin/build-template', () => {
   const script = path.normalize(__dirname + '/../bin/build-template.js');
   const template = path.normalize(__dirname + '/fixtures/sync-args.js');
-  t.test('outputs expected', (q) => {
-    exec([script, template, '--this', 'that', '-r', 'cn-north-1'].join(' '), (err, stdout, stderr) => {
-      q.error(err);
-      q.error(stderr);
-      const result = JSON.parse(stdout);
-      q.equal('that', result.this);
-      q.equal('cn-north-1', result.region);
-      q.end();
-    });
+
+  test('outputs expected', async () => {
+    const { stdout, stderr } = await execFileAsync(script, [
+      template,
+      '--this',
+      'that',
+      '-r',
+      'cn-north-1'
+    ]);
+    expect(stderr).toBeFalsy();
+    const result = JSON.parse(stdout);
+    expect(result.this).toBe('that');
+    expect(result.region).toBe('cn-north-1');
   });
-  t.end();
 });
-

package/test/fixtures/shortcuts/cross-account-role-defaults.json

@@ -30,4 +30,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/cross-account-role-no-defaults.json

@@ -63,4 +63,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/event-lambda-defaults.json

@@ -199,4 +199,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/event-lambda-full.json

@@ -199,4 +199,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/firehose-defaults.json

@@ -197,4 +197,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/firehose-with-stream.json

@@ -231,4 +231,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/glue-database-defaults.json

@@ -22,4 +22,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/glue-database-no-defaults.json

@@ -34,4 +34,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/glue-json-table-defaults.json

@@ -45,4 +45,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/glue-json-table-no-defaults.json

@@ -69,4 +69,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/glue-orc-table-defaults.json

@@ -45,4 +45,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/glue-orc-table-no-defaults.json

@@ -69,4 +69,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/glue-parquet-table-defaults.json

@@ -49,4 +49,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/glue-parquet-table-no-defaults.json

@@ -69,4 +69,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/glue-table-defaults.json

@@ -37,4 +37,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/glue-table-no-defaults.json

@@ -69,4 +69,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/glue-view-defaults.json

@@ -60,4 +60,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/glue-view-no-defaults.json

@@ -88,4 +88,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/hookshot-github-secret-ref.json

@@ -434,4 +434,4 @@
       }
     }
   }
-}
+}

package/test/fixtures/shortcuts/hookshot-github-secret-string.json

@@ -426,4 +426,4 @@
       "Value": "abc123"
     }
   }
-}
+}

package/test/fixtures/shortcuts/hookshot-github.json

@@ -436,4 +436,4 @@
       }
     }
   }
-}
+}

package/test/fixtures/shortcuts/hookshot-passthrough-access-log-format.json

@@ -432,4 +432,4 @@
       }
     }
   }
-}
+}

package/test/fixtures/shortcuts/hookshot-passthrough-alarms.json

@@ -416,4 +416,4 @@
       }
     }
   }
-}
+}

package/test/fixtures/shortcuts/hookshot-passthrough-enhanced-logging.json

@@ -414,4 +414,4 @@
       }
     }
   }
-}
+}

package/test/fixtures/shortcuts/hookshot-passthrough-full-blown-logging.json

@@ -414,4 +414,4 @@
       }
     }
   }
-}
+}

package/test/fixtures/shortcuts/hookshot-passthrough-logging.json

@@ -414,4 +414,4 @@
       }
     }
   }
-}
+}

package/test/fixtures/shortcuts/hookshot-passthrough.json

@@ -414,4 +414,4 @@
       }
     }
   }
-}
+}

package/test/fixtures/shortcuts/lambda-defaults.json

@@ -133,4 +133,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/lambda-docker.json

@@ -131,4 +131,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/lambda-full.json

@@ -183,4 +183,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/lambda-provided-role.json

@@ -133,4 +133,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/lambda-zipfile.json

@@ -132,4 +132,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/log-subscription-lambda-defaults.json

@@ -169,4 +169,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/log-subscription-lambda-no-defaults.json

@@ -169,4 +169,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/queue-defaults.json

@@ -104,4 +104,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/queue-external-topic-ref.json

@@ -99,4 +99,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/queue-external-topic.json

@@ -92,4 +92,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/queue-fifo-queuename.json

@@ -49,4 +49,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/queue-fifo.json

@@ -53,4 +53,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/queue-full.json

@@ -118,4 +118,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/queue-lambda-zero.json

@@ -174,4 +174,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/queue-lambda.json

@@ -174,4 +174,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/role-defaults.json

@@ -24,4 +24,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/role-no-defaults.json

@@ -63,4 +63,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/scheduled-lambda-defaults.json

@@ -208,4 +208,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/scheduled-lambda-full.json

@@ -166,4 +166,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/service-role-defaults.json

@@ -26,4 +26,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/service-role-no-defaults.json

@@ -59,4 +59,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/service-role-no-url-suffix.json

@@ -26,4 +26,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/service-role-url-suffix-with-replacement.json

@@ -26,4 +26,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/service-role-url-suffix.json

@@ -26,4 +26,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/stream-lambda-defaults.json

@@ -179,4 +179,4 @@
     }
   },
   "Outputs": {}
-}
+}

package/test/fixtures/shortcuts/stream-lambda-no-defaults.json

@@ -187,4 +187,4 @@
     }
   },
   "Outputs": {}
-}
+}