Skip to content

Commit 4aaf407

Browse files
bnoordhuistargos
authored andcommitted
test: show child stderr output in largepages test
The test starts child processes. A recent change is suspected of causing flaky crashes on one of the alpine buildbots but we can't know for sure because the test hides the child's stderr. Refs: #31547 (comment) PR-URL: #31612 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
1 parent e8985c2 commit 4aaf407

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

test/parallel/test-startup-large-pages.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,8 @@ const { spawnSync } = require('child_process');
88

99
{
1010
const child = spawnSync(process.execPath,
11-
[ '--use-largepages=on', '-p', '42' ]);
11+
[ '--use-largepages=on', '-p', '42' ],
12+
{ stdio: ['inherit', 'pipe', 'inherit'] });
1213
const stdout = child.stdout.toString().match(/\S+/g);
1314
assert.strictEqual(child.status, 0);
1415
assert.strictEqual(child.signal, null);

0 commit comments

Comments
 (0)