From 49be50051cb06a9b20346c43e7703734ed05267d Mon Sep 17 00:00:00 2001 From: Ben Noordhuis Date: Sun, 2 Feb 2020 02:47:01 +0100 Subject: [PATCH] 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: https://github.com/nodejs/node/pull/31547#issuecomment-581076515 PR-URL: https://github.com/nodejs/node/pull/31612 Reviewed-By: Rich Trott Reviewed-By: Colin Ihrig Reviewed-By: Anna Henningsen --- test/parallel/test-startup-large-pages.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/parallel/test-startup-large-pages.js b/test/parallel/test-startup-large-pages.js index d5589ee547f760..ed0542b6c6dc68 100644 --- a/test/parallel/test-startup-large-pages.js +++ b/test/parallel/test-startup-large-pages.js @@ -8,7 +8,8 @@ const { spawnSync } = require('child_process'); { const child = spawnSync(process.execPath, - [ '--use-largepages=on', '-p', '42' ]); + [ '--use-largepages=on', '-p', '42' ], + { stdio: ['inherit', 'pipe', 'inherit'] }); const stdout = child.stdout.toString().match(/\S+/g); assert.strictEqual(child.status, 0); assert.strictEqual(child.signal, null);