Skip to content

Commit c161604

Browse files
matsuda-koushiMylesBorins
matsuda-koushi
authored andcommitted
test: improve coverage on removeListeners functions
PR-URL: #11140 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Italo A. Casas <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
1 parent 34614af commit c161604

File tree

2 files changed

+19
-0
lines changed

2 files changed

+19
-0
lines changed

test/parallel/test-event-emitter-remove-all-listeners.js

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,3 +77,14 @@ function listener() {}
7777
ee.removeAllListeners('baz');
7878
assert.strictEqual(ee.listeners('baz').length, 0);
7979
}
80+
81+
{
82+
const ee = new events.EventEmitter();
83+
assert.deepStrictEqual(ee, ee.removeAllListeners());
84+
}
85+
86+
{
87+
const ee = new events.EventEmitter();
88+
ee._events = undefined;
89+
assert.strictEqual(ee, ee.removeAllListeners());
90+
}

test/parallel/test-event-emitter-remove-listeners.js

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -122,3 +122,11 @@ assert.throws(() => {
122122

123123
ee.removeListener('foo', null);
124124
}, /^TypeError: "listener" argument must be a function$/);
125+
126+
{
127+
const ee = new EventEmitter();
128+
const listener = () => {};
129+
ee._events = undefined;
130+
const e = ee.removeListener('foo', listener);
131+
assert.strictEqual(e, ee);
132+
}

0 commit comments

Comments
 (0)