diff options
| author | Myles Borins <mborins@us.ibm.com> | 2015-10-20 11:40:54 -0700 |
|---|---|---|
| committer | Rich Trott <rtrott@gmail.com> | 2015-10-24 14:42:41 -0700 |
| commit | 28e9a022df3ef89a0ea21e4ad86655eb408a8d96 (patch) | |
| tree | cfa007439af61736767bab8df5556c871d091468 /test/parallel/test-cluster-rr-ref.js | |
| parent | b6207906c452be4c5f049a098fc645fdfb897306 (diff) | |
| download | android-node-v8-28e9a022df3ef89a0ea21e4ad86655eb408a8d96.tar.gz android-node-v8-28e9a022df3ef89a0ea21e4ad86655eb408a8d96.tar.bz2 android-node-v8-28e9a022df3ef89a0ea21e4ad86655eb408a8d96.zip | |
test: wrap assert.fail when passed to callback
Currently there are many instances where assert.fail is directly passed
to a callback for error handling. Unfortunately this will swallow the
error as it is the third argument of assert.fail that sets the message
not the first.
This commit adds a new function to test/common.js that simply wraps
assert.fail and calls it with the provided message.
Tip of the hat to @trott for pointing me in the direction of this.
PR-URL: https://github.com/nodejs/node/pull/3453
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Diffstat (limited to 'test/parallel/test-cluster-rr-ref.js')
| -rw-r--r-- | test/parallel/test-cluster-rr-ref.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/parallel/test-cluster-rr-ref.js b/test/parallel/test-cluster-rr-ref.js index 606ff708e9..474e4d69f2 100644 --- a/test/parallel/test-cluster-rr-ref.js +++ b/test/parallel/test-cluster-rr-ref.js @@ -10,7 +10,7 @@ if (cluster.isMaster) { if (msg === 'done') this.kill(); }); } else { - const server = net.createServer(assert.fail); + const server = net.createServer(common.fail); server.listen(common.PORT, function() { server.unref(); server.ref(); |