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-https-timeout-server-2.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-https-timeout-server-2.js')
| -rw-r--r-- | test/parallel/test-https-timeout-server-2.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/parallel/test-https-timeout-server-2.js b/test/parallel/test-https-timeout-server-2.js index 9970688fe7..df605958cf 100644 --- a/test/parallel/test-https-timeout-server-2.js +++ b/test/parallel/test-https-timeout-server-2.js @@ -18,7 +18,7 @@ var options = { cert: fs.readFileSync(common.fixturesDir + '/keys/agent1-cert.pem') }; -var server = https.createServer(options, assert.fail); +var server = https.createServer(options, common.fail); server.on('secureConnection', function(cleartext) { var s = cleartext.setTimeout(50, function() { |