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]>
nodejs · Feb 1, 2017 · 11115c0 · 11115c0
1 parent 9f13b5f
commit 11115c0
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/test/parallel/test-event-emitter-max-listeners.js b/test/parallel/test-event-emitter-max-listeners.js
@@ -1,8 +1,8 @@
 'use strict';
 const common = require('../common');
-var assert = require('assert');
-var events = require('events');
-var e = new events.EventEmitter();
+const assert = require('assert');
+const events = require('events');
+const e = new events.EventEmitter();
 
 e.on('maxListeners', common.mustCall(function() {}));
 
@@ -11,14 +11,14 @@ e.setMaxListeners(42);
 
 assert.throws(function() {
   e.setMaxListeners(NaN);
-});
+}, /^TypeError: n must be a positive number$/);
 
 assert.throws(function() {
   e.setMaxListeners(-1);
-});
+}, /^TypeError: n must be a positive number$/);
 
 assert.throws(function() {
   e.setMaxListeners('and even this');
-});
+}, /^TypeError: n must be a positive number$/);
 
 e.emit('maxListeners');