node/test/parallel/test-tls-timeout-server.js
Myles Borins af55641a69 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>
2015-10-29 08:38:42 -07:00

36 lines
783 B
JavaScript

'use strict';
var common = require('../common');
var assert = require('assert');
if (!common.hasCrypto) {
console.log('1..0 # Skipped: missing crypto');
return;
}
var tls = require('tls');
var net = require('net');
var fs = require('fs');
var clientErrors = 0;
process.on('exit', function() {
assert.equal(clientErrors, 1);
});
var options = {
key: fs.readFileSync(common.fixturesDir + '/keys/agent1-key.pem'),
cert: fs.readFileSync(common.fixturesDir + '/keys/agent1-cert.pem'),
handshakeTimeout: 50
};
var server = tls.createServer(options, common.fail);
server.on('clientError', function(err, conn) {
conn.destroy();
server.close();
clientErrors++;
});
server.listen(common.PORT, function() {
net.connect({ host: '127.0.0.1', port: common.PORT });
});