fs: fix missing 'error' event in (Read|Write)Stream#destroy

fs.ReadStream / fs.WriteStream destroy([error]) function
should emit 'error' event if `error` is set.

PR-URL: #19735
Fixes: #19727
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

Author: range3

lib/fs.js

@@ -2128,7 +2128,7 @@ ReadStream.prototype._destroy = function(err, cb) {
     return;
   }
 
-  closeFsStream(this, cb);
+  closeFsStream(this, cb, err);
   this.fd = null;
 };
 

test/parallel/test-fs-stream-destroy-emit-error.js

@@ -0,0 +1,20 @@
+'use strict';
+const common = require('../common');
+const assert = require('assert');
+const fs = require('fs');
+
+const tmpdir = require('../common/tmpdir');
+tmpdir.refresh();
+
+test(fs.createReadStream(__filename));
+test(fs.createWriteStream(`${tmpdir.path}/dummy`));
+
+function test(stream) {
+  const err = new Error('DESTROYED');
+  stream.on('open', function() {
+    stream.destroy(err);
+  });
+  stream.on('error', common.mustCall(function(err_) {
+    assert.strictEqual(err_, err);
+  }));
+}