Commit 166aa8a
test: fix child-process-pipe-dataflow
Make sure all the `wc` process stdout data is received before checking
its validity.
Fixes: #25988
PR-URL: #36366
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 6a6b3af commit 166aa8a
1 file changed
Lines changed: 6 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -61,8 +61,13 @@ const MB = KB * KB; | |||
| 61 | 61 | })); | |
| 62 | 62 | }); | |
| 63 | 63 | ||
| 64 | + let wcBuf = ''; | ||
| 64 | 65 | wc.stdout.on('data', common.mustCall((data) => { | |
| 66 | + wcBuf += data; | ||
| 67 | + })); | ||
| 68 | + | ||
| 69 | + wc.on('close', common.mustCall(() => { | ||
| 65 | 70 | // Grep always adds one extra byte at the end. | |
| 66 | - assert.strictEqual(data.toString().trim(), (MB + 1).toString()); | ||
| 71 | + assert.strictEqual(wcBuf.trim(), (MB + 1).toString()); | ||
| 67 | 72 | })); | |
| 68 | 73 | } | |
0 commit comments