test: deflake http2-pipe-named-pipe

Wait for all data to be read before sending the response and closing the client. Fixes: #40277 PR-URL: #40842 Reviewed-By: Robert Nagy <[email protected]> Reviewed-By: Adrian Estrada <[email protected]>
nodejs · Feb 1, 2022 · 4978689 · 4978689
1 parent f17789a
commit 4978689
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/test/parallel/test-http2-pipe-named-pipe.js b/test/parallel/test-http2-pipe-named-pipe.js
@@ -22,12 +22,15 @@ const server = http2.createServer();
 server.on('stream', common.mustCall((stream) => {
   const dest = stream.pipe(fs.createWriteStream(fn));
 
-  dest.on('finish', () => {
+  stream.on('end', common.mustCall(() => {
+    stream.respond();
+    stream.end();
+  }));
+
+  dest.on('finish', common.mustCall(() => {
     assert.strictEqual(fs.readFileSync(loc).length,
                        fs.readFileSync(fn).length);
-  });
-  stream.respond();
-  stream.end();
+  }));
 }));
 
 server.listen(common.PIPE, common.mustCall(() => {