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/parallel/test-process-exit-from-before-exit.js | |
| parent | 28d9485c25a0ddeb8817525e25d4c7bb1ee5013b (diff) | |
| download | node-new-6510eb5ddc802dcd25713c2b3cbb82711c94d075.tar.gz | |
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/parallel/test-process-exit-from-before-exit.js')
| -rw-r--r-- | test/parallel/test-process-exit-from-before-exit.js | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/test/parallel/test-process-exit-from-before-exit.js b/test/parallel/test-process-exit-from-before-exit.js index 30d358cb76..b37df0a5bd 100644 --- a/test/parallel/test-process-exit-from-before-exit.js +++ b/test/parallel/test-process-exit-from-before-exit.js @@ -1,9 +1,8 @@ 'use strict'; -var assert = require('assert'); var common = require('../common'); process.on('beforeExit', common.mustCall(function() { - setTimeout(assert.fail, 5); + setTimeout(common.fail, 5); process.exit(0); // Should execute immediately even if we schedule new work. - assert.fail(); + common.fail(); })); |