Skip to content

Commit 60f6955

Browse files
russokjevanlucas
authored andcommitted
test: add second argument to assert.throws()
The assert.throws() calls in test-event-emitter-max-listeners.js should include a constructor or RegExp as a second argument. PR-URL: #9987 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 2e14c5c commit 60f6955

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed
Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
'use strict';
22
constcommon=require('../common');
3-
varassert=require('assert');
4-
varevents=require('events');
5-
vare=newevents.EventEmitter();
3+
constassert=require('assert');
4+
constevents=require('events');
5+
conste=newevents.EventEmitter();
66

77
e.on('maxListeners',common.mustCall(function(){}));
88

@@ -11,14 +11,14 @@ e.setMaxListeners(42);
1111

1212
assert.throws(function(){
1313
e.setMaxListeners(NaN);
14-
});
14+
},/^TypeError:"n"argumentmustbeapositivenumber$/);
1515

1616
assert.throws(function(){
1717
e.setMaxListeners(-1);
18-
});
18+
},/^TypeError:"n"argumentmustbeapositivenumber$/);
1919

2020
assert.throws(function(){
2121
e.setMaxListeners('and even this');
22-
});
22+
},/^TypeError:"n"argumentmustbeapositivenumber$/);
2323

2424
e.emit('maxListeners');

0 commit comments

Comments
(0)