From c786d6341dba61980060281d4ae1a31d451f05c0 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Wed, 24 Jun 2015 23:33:11 -0700
Subject: [PATCH] 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>
---
 test/internet/test-net-connect-timeout.js | 55 ++++++-----------------
 1 file changed, 13 insertions(+), 42 deletions(-)

diff --git a/test/internet/test-net-connect-timeout.js b/test/internet/test-net-connect-timeout.js
index 60c2837485e898..436d7731827ef0 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);
 });