diff options
| author | cjihrig <cjihrig@gmail.com> | 2017-02-03 14:54:19 -0500 |
|---|---|---|
| committer | cjihrig <cjihrig@gmail.com> | 2017-02-06 14:07:55 -0500 |
| commit | 7dd82dd1c36e7e38a05ae9709956ae257cf75282 (patch) | |
| tree | 30486fbdbc1e8231674978b0e4f7ade765bdc59c /test/sequential/test-child-process-emfile.js | |
| parent | 9549329158ecde9653892161cc1890ac8c4e83c8 (diff) | |
| download | android-node-v8-7dd82dd1c36e7e38a05ae9709956ae257cf75282.tar.gz android-node-v8-7dd82dd1c36e7e38a05ae9709956ae257cf75282.tar.bz2 android-node-v8-7dd82dd1c36e7e38a05ae9709956ae257cf75282.zip | |
test: add common.mustNotCall()
This commit adds a mustNotCall() helper for testing. This provides
an alternative to using common.fail() as a callback, or creating
a callback function for the sole purpose of calling common.fail().
PR-URL: https://github.com/nodejs/node/pull/11152
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Diffstat (limited to 'test/sequential/test-child-process-emfile.js')
| -rw-r--r-- | test/sequential/test-child-process-emfile.js | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/test/sequential/test-child-process-emfile.js b/test/sequential/test-child-process-emfile.js index 55712b405a..ff141bdf4b 100644 --- a/test/sequential/test-child-process-emfile.js +++ b/test/sequential/test-child-process-emfile.js @@ -43,9 +43,7 @@ proc.on('error', common.mustCall(function(err) { assert.strictEqual(err.code, 'EMFILE'); })); -proc.on('exit', function() { - common.fail('"exit" should not be emitted (the process never spawned!)'); -}); +proc.on('exit', common.mustNotCall('"exit" event should not be emitted')); // close one fd for LSan if (openFds.length >= 1) { |