-
-
Notifications
You must be signed in to change notification settings - Fork 23k
Expand file tree
/
Copy pathreq.query.js
More file actions
117 lines (93 loc) · 3.08 KB
/
req.query.js
File metadata and controls
117 lines (93 loc) · 3.08 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
'use strict'
var assert = require('node:assert')
var express = require('../')
, request = require('supertest');
describe('req', function(){
describe('.query', function(){
it('should default to {}', function(done){
var app = createApp();
request(app)
.get('/')
.expect(200, '{}', done);
});
it('should default to parse simple keys', function (done) {
var app = createApp();
request(app)
.get('/?user[name]=tj')
.expect(200, '{"user[name]":"tj"}', done);
});
describe('when "query parser" is extended', function () {
it('should parse complex keys', function (done) {
var app = createApp('extended');
request(app)
.get('/?foo[0][bar]=baz&foo[0][fizz]=buzz&foo[]=done!')
.expect(200, '{"foo":[{"bar":"baz","fizz":"buzz"},"done!"]}', done);
});
it('should parse parameters with dots', function (done) {
var app = createApp('extended');
request(app)
.get('/?user.name=tj')
.expect(200, '{"user.name":"tj"}', done);
});
it('should parse repeated keys over 20 values as an array', function (done) {
var app = createApp('extended');
var query = Array.from({ length: 21 }, function (_, i) {
return 'tenancyIds=' + (i + 1)
}).join('&')
request(app)
.get('/?' + query)
.expect(200, '{"tenancyIds":["1","2","3","4","5","6","7","8","9","10","11","12","13","14","15","16","17","18","19","20","21"]}', done);
});
});
describe('when "query parser" is simple', function () {
it('should not parse complex keys', function (done) {
var app = createApp('simple');
request(app)
.get('/?user%5Bname%5D=tj')
.expect(200, '{"user[name]":"tj"}', done);
});
});
describe('when "query parser" is a function', function () {
it('should parse using function', function (done) {
var app = createApp(function (str) {
return {'length': (str || '').length};
});
request(app)
.get('/?user%5Bname%5D=tj')
.expect(200, '{"length":17}', done);
});
});
describe('when "query parser" disabled', function () {
it('should not parse query', function (done) {
var app = createApp(false);
request(app)
.get('/?user%5Bname%5D=tj')
.expect(200, '{}', done);
});
});
describe('when "query parser" enabled', function () {
it('should not parse complex keys', function (done) {
var app = createApp(true);
request(app)
.get('/?user%5Bname%5D=tj')
.expect(200, '{"user[name]":"tj"}', done);
});
});
describe('when "query parser" an unknown value', function () {
it('should throw', function () {
assert.throws(createApp.bind(null, 'bogus'),
/unknown value.*query parser/)
});
});
})
})
function createApp(setting) {
var app = express();
if (setting !== undefined) {
app.set('query parser', setting);
}
app.use(function (req, res) {
res.send(req.query);
});
return app;
}