diff options
| author | cjihrig <cjihrig@gmail.com> | 2016-07-14 11:26:35 -0400 |
|---|---|---|
| committer | cjihrig <cjihrig@gmail.com> | 2016-07-15 15:50:01 -0400 |
| commit | 6510eb5ddc802dcd25713c2b3cbb82711c94d075 (patch) | |
| tree | 9f22bb1adff8ef2474267297873f03f1c3704d49 /test/sequential/test-child-process-emfile.js | |
| parent | 28d9485c25a0ddeb8817525e25d4c7bb1ee5013b (diff) | |
| download | android-node-v8-6510eb5ddc802dcd25713c2b3cbb82711c94d075.tar.gz android-node-v8-6510eb5ddc802dcd25713c2b3cbb82711c94d075.tar.bz2 android-node-v8-6510eb5ddc802dcd25713c2b3cbb82711c94d075.zip | |
test: s/assert.fail/common.fail as appropriate
Many tests use assert.fail(null, null, msg) where it would be
simpler to use common.fail(msg). This is largely because
common.fail() is fairly new. This commit makes the replacement
when applicable.
PR-URL: https://github.com/nodejs/node/pull/7735
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Diffstat (limited to 'test/sequential/test-child-process-emfile.js')
| -rw-r--r-- | test/sequential/test-child-process-emfile.js | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/test/sequential/test-child-process-emfile.js b/test/sequential/test-child-process-emfile.js index f186db5401..8f59842f99 100644 --- a/test/sequential/test-child-process-emfile.js +++ b/test/sequential/test-child-process-emfile.js @@ -44,8 +44,7 @@ proc.on('error', common.mustCall(function(err) { })); proc.on('exit', function() { - const msg = '"exit" should not be emitted (the process never spawned!)'; - assert.fail(null, null, msg); + common.fail('"exit" should not be emitted (the process never spawned!)'); }); // close one fd for LSan |