diff options
author | Rich Trott <rtrott@gmail.com> | 2017-03-28 20:07:39 -0700 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2017-03-30 22:49:19 -0700 |
commit | a10e657657de19d2482307676199c8ccd218da28 (patch) | |
tree | 79cf909dfeeef20d61453818da8867110c868e39 /test | |
parent | 92de91d57029a61bcd5284d4ef345b26ca83237d (diff) | |
download | android-node-v8-a10e657657de19d2482307676199c8ccd218da28.tar.gz android-node-v8-a10e657657de19d2482307676199c8ccd218da28.tar.bz2 android-node-v8-a10e657657de19d2482307676199c8ccd218da28.zip |
test: fix flaky child-process-exec-kill-throws
This is a fix for test-child-process-exec-kill-throws which is currently
flaky on Windows.
A bug in the test was causing the child process to fail for reasons
other than those intended by the test. Instead of failing for exceeding
the `maxBuffer` setting, the test was failing because it was trying to
load `internal/child_process` without being passed the
`expose-internals` flag. Move that module to where only the parent
process (which gets the flag) loads it.
Additionally, improve an assertion message to help debug problems like
this.
PR-URL: https://github.com/nodejs/node/pull/12111
Fixes: https://github.com/nodejs/node/issues/12053
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Diffstat (limited to 'test')
-rw-r--r-- | test/parallel/parallel.status | 3 | ||||
-rw-r--r-- | test/parallel/test-child-process-exec-kill-throws.js | 9 |
2 files changed, 5 insertions, 7 deletions
diff --git a/test/parallel/parallel.status b/test/parallel/parallel.status index e01e142895..4703fdd05e 100644 --- a/test/parallel/parallel.status +++ b/test/parallel/parallel.status @@ -7,9 +7,6 @@ prefix parallel [true] # This section applies to all platforms [$system==win32] -# See https://github.com/nodejs/node/issues/12053, this test may be exposing a -# genuine bug with kill() on Windows. -test-child-process-exec-kill-throws : PASS,FLAKY [$system==linux] diff --git a/test/parallel/test-child-process-exec-kill-throws.js b/test/parallel/test-child-process-exec-kill-throws.js index d9473222a4..d5bc530584 100644 --- a/test/parallel/test-child-process-exec-kill-throws.js +++ b/test/parallel/test-child-process-exec-kill-throws.js @@ -3,12 +3,13 @@ const common = require('../common'); const assert = require('assert'); const cp = require('child_process'); -const internalCp = require('internal/child_process'); if (process.argv[2] === 'child') { - // Keep the process alive and printing to stdout. - setInterval(() => { console.log('foo'); }, 1); + // Since maxBuffer is 0, this should trigger an error. + console.log('foo'); } else { + const internalCp = require('internal/child_process'); + // Monkey patch ChildProcess#kill() to kill the process and then throw. const kill = internalCp.ChildProcess.prototype.kill; @@ -21,7 +22,7 @@ if (process.argv[2] === 'child') { const options = { maxBuffer: 0 }; const child = cp.exec(cmd, options, common.mustCall((err, stdout, stderr) => { // Verify that if ChildProcess#kill() throws, the error is reported. - assert(/^Error: mock error$/.test(err)); + assert.strictEqual(err.message, 'mock error', err); assert.strictEqual(stdout, ''); assert.strictEqual(stderr, ''); assert.strictEqual(child.killed, true); |