impulse-api 2.0.2 → 2.1.0

package/.eslintrc.json

@@ -0,0 +1,9 @@
+{
+    "parserOptions": {
+        "ecmaVersion": 2017
+    },
+
+    "env": {
+        "es6": true
+    }
+}

package/package.json

@@ -11,12 +11,12 @@
   "scripts": {
     "lint": "eslint --ignore-path .gitignore .",
     "test": "nyc --reporter=lcov --reporter=text-summary mocha",
-    "integration-server": "node ./test/integration/test-server.js"
+    "test-server": "node ./test/integration/test-server.js"
   },
-  "version": "2.0.2",
+  "version": "2.1.0",
   "dependencies": {
-    "async": "1.5.2",
-    "aws-sdk": "2.343.0",
+    "async": "*2.6.4",
+    "aws-sdk": "^2.1141.0",
     "cors": "2.8.4",
     "express": "4.16.4",
     "express-fileupload": "^1.1.9",
@@ -30,13 +30,11 @@
     "restify": "^8.5.1"
   },
   "devDependencies": {
-    "chai": "4.2.0",
-    "eslint": "3.12.2",
-    "eslint-config-airbnb-base": "9.0.0",
-    "eslint-plugin-import": "2.2.0",
-    "istanbul": "0.4.5",
+    "eslint": "8.16.0",
+    "eslint-config-airbnb-base": "15.0.0",
+    "eslint-plugin-import": "2.26.0",
     "mocha": "7.1.1",
-    "nyc": "^15.0.1",
+    "nyc": "15.1.0",
     "pre-commit": "1.1.3",
     "sinon": "4.4.6"
   },

package/test/integration/routes/sample.js

@@ -0,0 +1,19 @@
+exports.test = {
+    name: 'test-get-person',
+    description: 'retrieve a person by name and age',
+    method: 'GET',
+    endpoint: '/test-get-person',
+    version: 'v1',
+    inputs: {
+        name: {
+            required: true,
+        },
+        age: {
+            required: true,
+        },
+    },
+    requestBodyOverride: true,
+    run: (services, inputs, next) => {
+        next(inputs);
+    }
+}

package/test/server-test.js

@@ -116,12 +116,15 @@ describe('server-test', () => {
                 env: 'test',
             });
             try {
-                Api.buildParameters({}, {
-                    'test-param': {
-                        required: true
-                    }
-                });
+                Api.buildParameters({
+                        'test-param': {
+                            required: true
+                        },
+                    },
+                    {}
+                );
             } catch (e) {
+                console.log(e)
                 assert.contains(e.message, 'Missing parameter [test-param]');
             }
         });
@@ -221,52 +224,6 @@ describe('server-test', () => {
                 assert.ifError(e);
             }
         });
-        it('should not run the formatter if the param is not required and not provided', () => {
-            const Api = new Server({
-                name: 'test-Server',
-                routeDir: './test-routes',
-                port: 4000,
-                env: 'test',
-            });
-            try {
-                Api.buildParameters({
-                    param1: undefined
-                }, {
-                    param1: {
-                        required: false,
-                        formatter: () => {
-                            // The formatter should not run, so this error should not be thrown.
-                            throw new Error('param was formatted');
-                        }
-                    }
-                });
-            } catch (e) {
-                assert.ifError(e);
-            }
-        });
-        it('should not run the validator if the parameter is not required and not provided', () => {
-            const Api = new Server({
-                name: 'test-Server',
-                routeDir: './test-routes',
-                port: 4000,
-                env: 'test',
-            });
-            try {
-                Api.buildParameters({
-                    param1: undefined
-                }, {
-                    param1: {
-                        required: false,
-                        validate: () => {
-                            // The validator should not run, so this error should not be thrown.
-                            throw new Error('param was validated');
-                        }
-                    }
-                });
-            } catch (e) {
-                assert.ifError(e);
-            }
-        });
     });
 
     describe('#loadRoutes', () => {