test: fixed flaky test-http-readable-data-event

Fixes: #19905

PR-URL: #19931
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

Author: mcollina

test/parallel/test-http-readable-data-event.js

@@ -6,16 +6,21 @@ const http = require('http');
 const helloWorld = 'Hello World!';
 const helloAgainLater = 'Hello again later!';
 
+let next = null;
+
 const server = http.createServer((req, res) => {
   res.writeHead(200, {
     'Content-Length': '' + (helloWorld.length + helloAgainLater.length)
   });
-  res.write(helloWorld);
 
   // we need to make sure the data is flushed
-  setTimeout(() => {
+  // before writing again
+  next = () => {
     res.end(helloAgainLater);
-  }, common.platformTimeout(10));
+    next = () => {};
+  };
+
+  res.write(helloWorld);
 }).listen(0, function() {
   const opts = {
     hostname: 'localhost',
@@ -27,14 +32,15 @@ const server = http.createServer((req, res) => {
   const expectedRead = [helloWorld, null, helloAgainLater, null];
 
   const req = http.request(opts, (res) => {
-    res.on('error', common.mustNotCall);
+    res.on('error', common.mustNotCall());
 
     res.on('readable', common.mustCall(() => {
       let data;
 
       do {
         data = res.read();
         assert.strictEqual(data, expectedRead.shift());
+        next();
       } while (data !== null);
     }, 2));