stream: add test for multiple .push(null)

PR-URL: #29645 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Sep 24, 2019 · e078e48 · e078e48
1 parent fed05cc
commit e078e48
Showing 1 changed file with 13 additions and 0 deletions.
diff --git a/test/parallel/test-stream-readable-ended.js b/test/parallel/test-stream-readable-ended.js
@@ -31,3 +31,16 @@ const assert = require('assert');
     assert.strictEqual(readable.readableEnded, false);
   }));
 }
+
+// Verifies no `error` triggered on multiple .push(null) invocations
+{
+  const readable = new Readable();
+
+  readable.on('readable', () => { readable.read(); });
+  readable.on('error', common.mustNotCall());
+  readable.on('end', common.mustCall());
+
+  readable.push('a');
+  readable.push(null);
+  readable.push(null);
+}