1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: increase coverage for lib/events.js

Browse Source 
Adds tests for the case in which listeners() is invoked on a
EventEmitter with no events.

Adds a new test case for the situation in which a class
inherits from the EventEmitter but overrides the constructor
in the EventEmitter so that the _events is never set.

PR-URL: https://github.com/nodejs/node/pull/9865
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
This commit is contained in:
Safia Abdalla 2016-11-30 22:04:28 -06:00 committed by Rich Trott
parent 20fa6e7d07
commit a912b791d4
1 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
test/parallel/test-event-emitter-listeners.js
View File

@ -3,9 +3,12 @@
require('../common'); require('../common');
const assert = require('assert'); const assert = require('assert');
const events = require('events'); const events = require('events');
const util = require('util');
function listener() {} function listener() {}
function listener2() {} function listener2() {}
class TestStream { constructor() { } }
util.inherits(TestStream, events.EventEmitter);
{ {
const ee = new events.EventEmitter(); const ee = new events.EventEmitter();
@ -49,3 +52,14 @@ function listener2() {}
ee.once('foo', listener2); ee.once('foo', listener2);
assert.deepStrictEqual(ee.listeners('foo'), [listener, listener2]); assert.deepStrictEqual(ee.listeners('foo'), [listener, listener2]);
} }
{
const ee = new events.EventEmitter();
ee._events = undefined;
assert.deepStrictEqual(ee.listeners('foo'), []);
}
{
const s = new TestStream();
assert.deepStrictEqual(s.listeners('foo'), []);
}