diff options
author | Rich Trott <rtrott@gmail.com> | 2015-06-24 23:33:11 -0700 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2015-07-27 17:07:54 -0700 |
commit | c786d6341dba61980060281d4ae1a31d451f05c0 (patch) | |
tree | e724173dd3f5280a56d7b6836d40a729f94fa16f /test/internet | |
parent | 543dabb609226444fae0f9ed10f7b2360b6efda3 (diff) | |
download | nodejs-c786d6341dba61980060281d4ae1a31d451f05c0.tar.gz nodejs-c786d6341dba61980060281d4ae1a31d451f05c0.tar.bz2 nodejs-c786d6341dba61980060281d4ae1a31d451f05c0.zip |
test: do not use public IPs for timeout testing
PR-URL: https://github.com/nodejs/io.js/pull/2057
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Diffstat (limited to 'test/internet')
-rw-r--r-- | test/internet/test-net-connect-timeout.js | 55 |
1 files changed, 13 insertions, 42 deletions
diff --git a/test/internet/test-net-connect-timeout.js b/test/internet/test-net-connect-timeout.js index 60c283748..436d77318 100644 --- a/test/internet/test-net-connect-timeout.js +++ b/test/internet/test-net-connect-timeout.js @@ -2,8 +2,6 @@ // This example attempts to time out before the connection is established // https://groups.google.com/forum/#!topic/nodejs/UE0ZbfLt6t8 // https://groups.google.com/forum/#!topic/nodejs-dev/jR7-5UDqXkw -// -// TODO: how to do this without relying on the responses of specific sites? var common = require('../common'); var net = require('net'); @@ -11,61 +9,34 @@ var assert = require('assert'); var start = new Date(); -var gotTimeout0 = false; -var gotTimeout1 = false; +var gotTimeout = false; -var gotConnect0 = false; -var gotConnect1 = false; +var gotConnect = false; var T = 100; -// With DNS +// 240.*.*.*.* is "reserved for future use" +var socket = net.createConnection(9999, '240.0.0.0'); -var socket0 = net.createConnection(9999, 'google.com'); +socket.setTimeout(T); -socket0.setTimeout(T); - -socket0.on('timeout', function() { - console.error('timeout'); - gotTimeout0 = true; - var now = new Date(); - assert.ok(now - start < T + 500); - socket0.destroy(); -}); - -socket0.on('connect', function() { - console.error('connect'); - gotConnect0 = true; - socket0.destroy(); -}); - - -// Without DNS - -var socket1 = net.createConnection(9999, '24.24.24.24'); - -socket1.setTimeout(T); - -socket1.on('timeout', function() { +socket.on('timeout', function() { console.error('timeout'); - gotTimeout1 = true; + gotTimeout = true; var now = new Date(); assert.ok(now - start < T + 500); - socket1.destroy(); + socket.destroy(); }); -socket1.on('connect', function() { +socket.on('connect', function() { console.error('connect'); - gotConnect1 = true; - socket1.destroy(); + gotConnect = true; + socket.destroy(); }); process.on('exit', function() { - assert.ok(gotTimeout0); - assert.ok(!gotConnect0); - - assert.ok(gotTimeout1); - assert.ok(!gotConnect1); + assert.ok(gotTimeout); + assert.ok(!gotConnect); }); |