Skip to content

Commit c8a269f

Browse files
committed
[Tests] parse: remove useless tests; add coverage
1 parent 1e04701 commit c8a269f

File tree

2 files changed

+11
-26
lines changed

2 files changed

+11
-26
lines changed

test/parse.js

+10-26
Original file line numberDiff line numberDiff line change
@@ -39,35 +39,19 @@ test('parse()', function (t) {
3939
st.end();
4040
});
4141

42-
t.test('arrayFormat: brackets allows only explicit arrays', function (st) {
43-
st.deepEqual(qs.parse('a[]=b&a[]=c', { arrayFormat: 'brackets' }), { a: ['b', 'c'] });
44-
st.deepEqual(qs.parse('a[0]=b&a[1]=c', { arrayFormat: 'brackets' }), { a: ['b', 'c'] });
45-
st.deepEqual(qs.parse('a=b,c', { arrayFormat: 'brackets' }), { a: 'b,c' });
46-
st.deepEqual(qs.parse('a=b&a=c', { arrayFormat: 'brackets' }), { a: ['b', 'c'] });
47-
st.end();
48-
});
49-
50-
t.test('arrayFormat: indices allows only indexed arrays', function (st) {
51-
st.deepEqual(qs.parse('a[]=b&a[]=c', { arrayFormat: 'indices' }), { a: ['b', 'c'] });
52-
st.deepEqual(qs.parse('a[0]=b&a[1]=c', { arrayFormat: 'indices' }), { a: ['b', 'c'] });
53-
st.deepEqual(qs.parse('a=b,c', { arrayFormat: 'indices' }), { a: 'b,c' });
54-
st.deepEqual(qs.parse('a=b&a=c', { arrayFormat: 'indices' }), { a: ['b', 'c'] });
55-
st.end();
56-
});
57-
58-
t.test('arrayFormat: comma allows only comma-separated arrays', function (st) {
59-
st.deepEqual(qs.parse('a[]=b&a[]=c', { arrayFormat: 'comma' }), { a: ['b', 'c'] });
60-
st.deepEqual(qs.parse('a[0]=b&a[1]=c', { arrayFormat: 'comma' }), { a: ['b', 'c'] });
61-
st.deepEqual(qs.parse('a=b,c', { arrayFormat: 'comma' }), { a: 'b,c' });
62-
st.deepEqual(qs.parse('a=b&a=c', { arrayFormat: 'comma' }), { a: ['b', 'c'] });
42+
t.test('comma: false', function (st) {
43+
st.deepEqual(qs.parse('a[]=b&a[]=c'), { a: ['b', 'c'] });
44+
st.deepEqual(qs.parse('a[0]=b&a[1]=c'), { a: ['b', 'c'] });
45+
st.deepEqual(qs.parse('a=b,c'), { a: 'b,c' });
46+
st.deepEqual(qs.parse('a=b&a=c'), { a: ['b', 'c'] });
6347
st.end();
6448
});
6549

66-
t.test('arrayFormat: repeat allows only repeated values', function (st) {
67-
st.deepEqual(qs.parse('a[]=b&a[]=c', { arrayFormat: 'repeat' }), { a: ['b', 'c'] });
68-
st.deepEqual(qs.parse('a[0]=b&a[1]=c', { arrayFormat: 'repeat' }), { a: ['b', 'c'] });
69-
st.deepEqual(qs.parse('a=b,c', { arrayFormat: 'repeat' }), { a: 'b,c' });
70-
st.deepEqual(qs.parse('a=b&a=c', { arrayFormat: 'repeat' }), { a: ['b', 'c'] });
50+
t.test('comma: true', function (st) {
51+
st.deepEqual(qs.parse('a[]=b&a[]=c', { comma: true }), { a: ['b', 'c'] });
52+
st.deepEqual(qs.parse('a[0]=b&a[1]=c', { comma: true }), { a: ['b', 'c'] });
53+
st.deepEqual(qs.parse('a=b,c', { comma: true }), { a: ['b', 'c'] });
54+
st.deepEqual(qs.parse('a=b&a=c', { comma: true }), { a: ['b', 'c'] });
7155
st.end();
7256
});
7357

test/stringify.js

+1
Original file line numberDiff line numberDiff line change
@@ -889,6 +889,7 @@ test('stringify()', function (t) {
889889
qs.stringify({ a: 1, b: new Date(), c: true, d: [1] }, {
890890
encoder: function (str) {
891891
st.match(typeof str, /^(?:string|number|boolean)$/);
892+
return '';
892893
}
893894
});
894895

0 commit comments

Comments
 (0)