GeezerVison is showing github.com — test_runner: remove stdout and stderr from error · nodejs/node@20f6fa3
File treeExpand file treeCollapse file tree Open diff view settings Expand file treeCollapse file tree Open diff view settings Collapse file Copy file name to clipboardExpand all lines: lib/internal/test_runner/runner.js +1-8Lines changed: 1 addition & 8 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -233,18 +233,13 @@ function runTestFile(path, root, inspectPort, filesWatcher) { | 233 | 233 | runningProcesses.set(path, child); | 234 | 234 |
| 235 | 235 | let err; | 236 | | - let stderr = ''; | 237 | 236 |
| 238 | 237 | filesWatcher?.watchChildProcessModules(child, path); | 239 | 238 |
| 240 | 239 | child.on('error', (error) => { | 241 | 240 | err = error; | 242 | 241 | }); | 243 | 242 |
| 244 | | - child.stderr.on('data', (data) => { | 245 | | - stderr += data; | 246 | | - }); | 247 | | - | 248 | 243 | if (isUsingInspector()) { | 249 | 244 | const rl = createInterface({ input: child.stderr }); | 250 | 245 | rl.on('line', (line) => { | @@ -265,7 +260,7 @@ function runTestFile(path, root, inspectPort, filesWatcher) { | 265 | 260 | subtest.addToReport(ast); | 266 | 261 | }); | 267 | 262 |
| 268 | | - const { 0: { 0: code, 1: signal }, 1: stdout } = await SafePromiseAll([ | | 263 | + const { 0: { 0: code, 1: signal } } = await SafePromiseAll([ | 269 | 264 | once(child, 'exit', { signal: t.signal }), | 270 | 265 | child.stdout.toArray({ signal: t.signal }), | 271 | 266 | ]); | @@ -278,8 +273,6 @@ function runTestFile(path, root, inspectPort, filesWatcher) { | 278 | 273 | __proto__: null, | 279 | 274 | exitCode: code, | 280 | 275 | signal: signal, | 281 | | - stdout: ArrayPrototypeJoin(stdout, ''), | 282 | | - stderr, | 283 | 276 | // The stack will not be useful since the failures came from tests | 284 | 277 | // in a child process. | 285 | 278 | stack: undefined, | |
|