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-tls-timeout-server.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-tls-timeout-server.js')
-rw-r--r-- | test/parallel/test-tls-timeout-server.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/parallel/test-tls-timeout-server.js b/test/parallel/test-tls-timeout-server.js index ee932c9b1f..e3ed246386 100644 --- a/test/parallel/test-tls-timeout-server.js +++ b/test/parallel/test-tls-timeout-server.js @@ -23,7 +23,7 @@ var options = { handshakeTimeout: 50 }; -var server = tls.createServer(options, assert.fail); +var server = tls.createServer(options, common.fail); server.on('clientError', function(err, conn) { conn.destroy(); |