test: do not use the same EventEmitter instance

Prevent multiple listeners for the `'error'` event to be added to the same `EventEmitter` instance. PR-URL: nodejs#35560 Refs: nodejs#35557 (comment) Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Gerhard Stöbich <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Robert Nagy <[email protected]> Reviewed-By: Rich Trott <[email protected]>
lpinca · Oct 12, 2020 · c739f37 · c739f37
1 parent c759281
commit c739f37
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/test/parallel/test-stream2-readable-wrap-error.js b/test/parallel/test-stream2-readable-wrap-error.js
@@ -5,12 +5,14 @@ const assert = require('assert');
 const Readable = require('_stream_readable');
 const EE = require('events').EventEmitter;
 
-const oldStream = new EE();
-oldStream.pause = () => {};
-oldStream.resume = () => {};
+class LegacyStream extends EE {
+  pause() {}
+  resume() {}
+}
 
 {
   const err = new Error();
+  const oldStream = new LegacyStream();
   const r = new Readable({ autoDestroy: true })
     .wrap(oldStream)
     .on('error', common.mustCall(() => {
@@ -23,6 +25,7 @@ oldStream.resume = () => {};
 
 {
   const err = new Error();
+  const oldStream = new LegacyStream();
   const r = new Readable({ autoDestroy: false })
     .wrap(oldStream)
     .on('error', common.mustCall(() => {