stream: always defer readable in EOF when sync

Fix a regression introduced by
#18515 that broke
the dicer module tests.

See: #18515

PR-URL: #18615
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>

Author: mcollina

lib/_stream_readable.js

@@ -488,9 +488,9 @@ function onEofChunk(stream, state) {
   }
   state.ended = true;
 
-  if (state.sync && state.length) {
-    // if we are sync and have data in the buffer, wait until next tick
-    // to emit the data. otherwise we risk emitting data in the flow()
+  if (state.sync) {
+    // if we are sync, wait until next tick to emit the data.
+    // Otherwise we risk emitting data in the flow()
     // the readable code triggers during a read() call
     emitReadable(stream);
   } else {

test/parallel/test-stream-readable-object-multi-push-async.js

@@ -160,7 +160,7 @@ const BATCH = 10;
   });
 
   readable.on('end', common.mustCall(() => {
-    assert.strictEqual(nextTickPassed, false);
+    assert.strictEqual(nextTickPassed, true);
   }));
 }