stream: preserve error over AbortError in pipeline

Signed-off-by: marcopiraccini <marco.piraccini@gmail.com>

Author: marcopiraccini

lib/internal/streams/pipeline.js

@@ -227,7 +227,7 @@ function pipelineImpl(streams, callback, opts) {
   }
 
   function finishImpl(err, final) {
-    if (err && (!error || error.code === 'ERR_STREAM_PREMATURE_CLOSE')) {
+    if (err && (!error || error.code === 'ERR_STREAM_PREMATURE_CLOSE' || error.name === 'AbortError')) {
       error = err;
     }
 

test/parallel/test-stream-pipeline.js

@@ -1749,3 +1749,35 @@ tmpdir.refresh();
     assert.deepStrictEqual(err, new Error('booom'));
   }));
 }
+
+{
+  // Errors thrown in Readable.map inside pipeline should not be
+  // swallowed by AbortError when the source is an infinite stream.
+  function createInfiniteReadable() {
+    return new Readable({
+      read() {
+        this.push('data');
+      },
+    });
+  }
+
+  function createObjectTransform() {
+    return new Transform({
+      readableObjectMode: true,
+      transform(chunk, encoding, callback) {
+        this.push({});
+        callback();
+      },
+    });
+  }
+
+  pipelinep(
+    createInfiniteReadable(),
+    createObjectTransform(),
+    (readable) => readable.map(async () => {
+      throw new Error('Boom!');
+    }),
+  ).then(common.mustNotCall(), common.mustCall((err) => {
+    assert.strictEqual(err.message, 'Boom!');
+  }));
+}