added fix and test for non exist path in swagger

Author: Manor

src/inputValidationError.js

@@ -7,7 +7,7 @@
  * @extends {Error}
  */
 class InputValidationError extends Error {
-    constructor(errors, path, method, options) {
+    constructor(errors, path, method, options = {}) {
         super('Input validation error');
 
         if (options.beautifyErrors && options.firstError) {

src/middleware.js

@@ -95,7 +95,7 @@ function _validateRequest(requestOptions) {
 
 function _validateBody(body, path, method) {
     return new Promise(function (resolve, reject) {
-        if (schemas[path][method].body && !schemas[path][method].body.validate(body)) {
+        if (schemas[path] && schemas[path][method] && schemas[path][method].body && !schemas[path][method].body.validate(body)) {
             return reject(schemas[path][method].body.errors);
         }
         return resolve();
@@ -104,7 +104,7 @@ function _validateBody(body, path, method) {
 
 function _validateParams(headers, pathParams, query, files, path, method) {
     return new Promise(function (resolve, reject) {
-        if (schemas[path][method].parameters && !schemas[path][method].parameters({ query: query, headers: headers, path: pathParams, files: files })) {
+        if (schemas[path] && schemas[path][method] && schemas[path][method].parameters && !schemas[path][method].parameters({ query: query, headers: headers, path: pathParams, files: files })) {
             return reject(schemas[path][method].parameters.errors);
         }
 

test/openapi3/openapi3-test.js

@@ -118,7 +118,20 @@ describe('input-validation middleware tests', function () {
                     done();
                 });
         });
-
+        it('when path does not exist in swagger - should not execute validation on request', function (done) {
+            request(app)
+                .post('/non-exist-path-in-swagger')
+                .send({})
+                .expect(200, function (err, res) {
+                    if (err) {
+                        throw err;
+                    }
+                    expect(res.body).to.eql({
+                        'result': 'OK'
+                    });
+                    done();
+                });
+        });
         describe('discriminator-pet', function () {
             it('missing discriminator field', function (done) {
                 request(app)

test/openapi3/test-server-pet.js

@@ -39,6 +39,9 @@ module.exports = function (options) {
             app.post('/pet-discriminator-on-child', inputValidation.validate, function (req, res, next) {
                 res.json({ result: 'OK' });
             });
+            app.post('/non-exist-path-in-swagger',inputValidation.validate, function (req, res, next) {
+                res.json({ result: 'OK' });
+            });
 
             app.use(function (err, req, res, next) {
                 if (err instanceof inputValidation.InputValidationError) {