stream: finished should complete with read-only Duplex

lib/internal/streams/end-of-stream.js

@@ -76,7 +76,8 @@ function eos(stream, opts, callback) {
     state &&
     state.autoDestroy &&
     state.emitClose &&
-    state.closed === false
+    state.closed === false &&
+    (stream.readable === readable && stream.writable === writable)
   );
 
   let writableFinished = stream.writableFinished ||

test/parallel/test-stream-finished.js

@@ -1,7 +1,7 @@
 'use strict';
 
 const common = require('../common');
-const { Writable, Readable, Transform, finished } = require('stream');
+const { Writable, Readable, Transform, finished, Duplex } = require('stream');
 const assert = require('assert');
 const EE = require('events');
 const fs = require('fs');
@@ -352,3 +352,19 @@ testClosed((opts) => new Writable({ write() {}, ...opts }));
   r.push(null);
   r.destroy();
 }
+
+{
+  const d = new Duplex({
+    final(cb) { }, // Never close writable side for test purpose
+    read() {
+      this.push(null);
+    }
+  });
+
+  d.on('end', common.mustCall());
+
+  finished(d, { readable: true, writable: false }, common.mustCall());
+
+  d.end();
+  d.resume();
+}