Skip to content

Commit 74d5ab5

Browse files
committed
tests: simplify verfy option throw test
1 parent d2cdae7 commit 74d5ab5

File tree

4 files changed

+16
-48
lines changed

4 files changed

+16
-48
lines changed

test/json.js

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -312,18 +312,10 @@ describe('bodyParser.json()', function(){
312312
})
313313
})
314314

315-
describe('with verify option', function(){
316-
it('should assert value if function', function(){
317-
var err;
318-
319-
try {
320-
var server = createServer({ verify: 'lol' })
321-
} catch (e) {
322-
err = e;
323-
}
324-
325-
assert.ok(err);
326-
assert.equal(err.name, 'TypeError');
315+
describe('with verify option', function () {
316+
it('should assert value if function', function () {
317+
assert.throws(createServer.bind(null, { verify: 'lol' }),
318+
/TypeError: option verify must be function/)
327319
})
328320

329321
it('should error from verify', function(done){

test/raw.js

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -228,18 +228,10 @@ describe('bodyParser.raw()', function(){
228228
})
229229
})
230230

231-
describe('with verify option', function(){
232-
it('should assert value is function', function(){
233-
var err;
234-
235-
try {
236-
var server = createServer({ verify: 'lol' })
237-
} catch (e) {
238-
err = e;
239-
}
240-
241-
assert.ok(err);
242-
assert.equal(err.name, 'TypeError');
231+
describe('with verify option', function () {
232+
it('should assert value is function', function () {
233+
assert.throws(createServer.bind(null, { verify: 'lol' }),
234+
/TypeError: option verify must be function/)
243235
})
244236

245237
it('should error from verify', function(done){

test/text.js

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -256,18 +256,10 @@ describe('bodyParser.text()', function(){
256256
})
257257
})
258258

259-
describe('with verify option', function(){
260-
it('should assert value is function', function(){
261-
var err;
262-
263-
try {
264-
var server = createServer({ verify: 'lol' })
265-
} catch (e) {
266-
err = e;
267-
}
268-
269-
assert.ok(err);
270-
assert.equal(err.name, 'TypeError');
259+
describe('with verify option', function () {
260+
it('should assert value is function', function () {
261+
assert.throws(createServer.bind(null, { verify: 'lol' }),
262+
/TypeError: option verify must be function/)
271263
})
272264

273265
it('should error from verify', function(done){

test/urlencoded.js

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -480,18 +480,10 @@ describe('bodyParser.urlencoded()', function(){
480480
})
481481
})
482482

483-
describe('with verify option', function(){
484-
it('should assert value if function', function(){
485-
var err;
486-
487-
try {
488-
var server = createServer({ verify: 'lol' })
489-
} catch (e) {
490-
err = e;
491-
}
492-
493-
assert.ok(err);
494-
assert.equal(err.name, 'TypeError');
483+
describe('with verify option', function () {
484+
it('should assert value if function', function () {
485+
assert.throws(createServer.bind(null, { verify: 'lol' }),
486+
/TypeError: option verify must be function/)
495487
})
496488

497489
it('should error from verify', function(done){

0 commit comments

Comments
 (0)