diff options
author | Timothy J Fontaine <tjfontaine@gmail.com> | 2013-07-01 21:29:15 +0000 |
---|---|---|
committer | Timothy J Fontaine <tjfontaine@gmail.com> | 2013-07-01 17:41:38 -0700 |
commit | 4c38742dd8a41f119b7113af6440014e25ae71c2 (patch) | |
tree | 465be127cd6cd0febd2adc4036bfca69fae8a65e /test | |
parent | 16b59cbc74c8fe2f8b30f3af4c2f885b7bfb6030 (diff) | |
download | node-4c38742dd8a41f119b7113af6440014e25ae71c2.tar.gz |
test: fix tls-hello-parser-failure on smartos
Assert that when the client closes it has seen an error, this prevents
the test from timing out.
Also queue a second write in the case that we were able to send the
buffer before the other side closed the connection.
Diffstat (limited to 'test')
-rw-r--r-- | test/simple/test-tls-hello-parser-failure.js | 19 |
1 files changed, 15 insertions, 4 deletions
diff --git a/test/simple/test-tls-hello-parser-failure.js b/test/simple/test-tls-hello-parser-failure.js index f7e1f741f..cce94007f 100644 --- a/test/simple/test-tls-hello-parser-failure.js +++ b/test/simple/test-tls-hello-parser-failure.js @@ -30,21 +30,32 @@ var options = { cert: fs.readFileSync(common.fixturesDir + '/test_cert.pem') }; +var bonkers = new Buffer(1024 * 1024); +bonkers.fill(42); + var server = tls.createServer(options, function(c) { }).listen(common.PORT, function() { var client = net.connect(common.PORT, function() { - var bonkers = new Buffer(1024 * 1024); - bonkers.fill(42); - client.end(bonkers); + client.write(bonkers); }); var once = false; - client.on('error', function() { + + var writeAgain = setTimeout(function() { + client.write(bonkers); + }); + + client.on('error', function(err) { if (!once) { + clearTimeout(writeAgain); once = true; client.destroy(); server.close(); } }); + + client.on('close', function (hadError) { + assert.strictEqual(hadError, true, 'Client never errored'); + }); }); |