Skip to content

Commit 11115c0

Browse files
russokjMylesBorins
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 9f13b5f commit 11115c0

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
'use strict';
22
const common = require('../common');
3-
var assert = require('assert');
4-
var events = require('events');
5-
var e = new events.EventEmitter();
3+
const assert = require('assert');
4+
const events = require('events');
5+
const e = new events.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 must be a positive number$/);
1515

1616
assert.throws(function() {
1717
e.setMaxListeners(-1);
18-
});
18+
}, /^TypeError: n must be a positive number$/);
1919

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

2424
e.emit('maxListeners');

0 commit comments

Comments
 (0)