stream: do not error async iterators on destroy(null)

Fixes: https://github.com/nodejs/node/issues/23890

PR-URL: https://github.com/nodejs/node/pull/23901
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

1 files changed, 18 insertions, 5 deletions

diff --git a/test/parallel/test-stream-readable-async-iterators.js b/test/parallel/test-stream-readable-async-iterators.js
index a9431fab81..83540de9de 100644
--- a/test/parallel/test-stream-readable-async-iterators.js
+++ b/test/parallel/test-stream-readable-async-iterators.js
@@ -335,11 +335,8 @@ async function tests() {
 
     readable.destroy();
 
-    try {
-      await readable[Symbol.asyncIterator]().next();
-    } catch (e) {
-      assert.strictEqual(e.code, 'ERR_STREAM_PREMATURE_CLOSE');
-    }
+    const { done } = await readable[Symbol.asyncIterator]().next();
+    assert.strictEqual(done, true);
   })();
 
   await (async function() {
@@ -380,6 +377,22 @@ async function tests() {
     for await (const b of r) {
     }
   })();
+
+  await (async () => {
+    console.log('destroy mid-stream does not error');
+    const r = new Readable({
+      objectMode: true,
+      read() {
+        this.push('asdf');
+        this.push('hehe');
+      }
+    });
+
+    // eslint-disable-next-line no-unused-vars
+    for await (const a of r) {
+      r.destroy(null);
+    }
+  })();
 }
 
 // to avoid missing some tests if a promise does not resolve