From ee8e1a13b60a6adfdc691b2a9b57289188397641 Mon Sep 17 00:00:00 2001 From: nanalelfe Date: Thu, 21 Jul 2016 02:29:31 -0400 Subject: need pull --- .../express-validator/test/checkQueryTest.js | 111 +++++++++++++++++++++ 1 file changed, 111 insertions(+) create mode 100644 node_modules/express-validator/test/checkQueryTest.js (limited to 'node_modules/express-validator/test/checkQueryTest.js') diff --git a/node_modules/express-validator/test/checkQueryTest.js b/node_modules/express-validator/test/checkQueryTest.js new file mode 100644 index 0000000..b3cee22 --- /dev/null +++ b/node_modules/express-validator/test/checkQueryTest.js @@ -0,0 +1,111 @@ +var chai = require('chai'); +var expect = chai.expect; +var request = require('supertest'); + +var app; +var errorMessage = 'Parameter is not an integer'; + +// There are three ways to pass parameters to express: +// - as part of the URL +// - as GET parameter in the querystring +// - as POST parameter in the body +// These test show that req.checkQuery are only interested in req.query values, all other +// parameters will be ignored. + +function validation(req, res) { + req.checkQuery('testparam', errorMessage).notEmpty().isInt(); + + var errors = req.validationErrors(); + if (errors) { + return res.send(errors); + } + res.send({ testparam: req.query.testparam }); +} + +function fail(body, length) { + expect(body).to.have.length(length); + expect(body[0]).to.have.property('msg', errorMessage); +} + +function pass(body) { + expect(body).to.have.property('testparam', '42'); +} + +function getRoute(path, test, length, done) { + request(app) + .get(path) + .end(function(err, res) { + test(res.body, length); + done(); + }); +} + +function postRoute(path, data, test, length, done) { + request(app) + .post(path) + .send(data) + .end(function(err, res) { + test(res.body, length); + done(); + }); +} + +// This before() is required in each set of tests in +// order to use a new validation function in each file +before(function() { + delete require.cache[require.resolve('./helpers/app')]; + app = require('./helpers/app')(validation); +}); + +describe('#checkQuery()', function() { + describe('GET tests', function() { + it('should return two errors when query is missing, and unrelated param is present', function(done) { + getRoute('/test', fail, 2, done); + }); + + it('should return two errors when query is missing', function(done) { + getRoute('/', fail, 2, done); + }); + + it('should return a success when query validates and unrelated query is present', function(done) { + getRoute('/42?testparam=42', pass, null, done); + }); + + it('should return one error when query does not validate as int and unrelated query is present', function(done) { + getRoute('/test?testparam=blah', fail, 1, done); + }); + }); + + describe('POST tests', function() { + it('should return two errors when query is missing, and unrelated param is present', function(done) { + postRoute('/test', null, fail, 2, done); + }); + + it('should return two errors when query is missing', function(done) { + postRoute('/', null, fail, 2, done); + }); + + it('should return a success when query validates and unrelated query is present', function(done) { + postRoute('/42?testparam=42', null, pass, null, done); + }); + + it('should return one error when query does not validate as int and unrelated query is present', function(done) { + postRoute('/test?testparam=blah', null, fail, 1, done); + }); + + // POST only + + it('should return a success when query validates and unrelated param/body is present', function(done) { + postRoute('/test?testparam=42', { testparam: 'posttest' }, pass, null, done); + }); + + it('should return one error when query does not validate as int and unrelated param/body is present', function(done) { + postRoute('/test?testparam=blah', { testparam: '42' }, fail, 1, done); + }); + + it('should return two errors when query is missing and unrelated body is present', function(done) { + postRoute('/', { testparam: '42' }, fail, 2, done); + }); + }); + +}); \ No newline at end of file -- cgit v1.2.3