test: fix up pummel/test-net-pingpong
authorBen Noordhuis <info@bnoordhuis.nl>
Thu, 22 May 2014 11:00:55 +0000 (13:00 +0200)
committerTimothy J Fontaine <tjfontaine@gmail.com>
Thu, 22 May 2014 19:51:10 +0000 (12:51 -0700)
Fix up a bad assumption in pummel/test-net-pingpong, namely that binding
to 'localhost' or '' means that incoming connections will have an IPv4
address.

Signed-off-by: Timothy J Fontaine <tjfontaine@gmail.com>
test/pummel/test-net-pingpong.js

index c665444..aab03c2 100644 (file)
@@ -36,12 +36,14 @@ function pingPongTest(port, host, on_complete) {
   var server = net.createServer({ allowHalfOpen: true }, function(socket) {
     assert.equal(true, socket.remoteAddress !== null);
     assert.equal(true, socket.remoteAddress !== undefined);
-    if (host === '127.0.0.1' || host === 'localhost' || !host) {
-      assert.equal(socket.remoteAddress, '127.0.0.1');
+    var address = socket.remoteAddress;
+    if (host === '127.0.0.1') {
+      assert.equal(address, '127.0.0.1');
+    } else if (host == null || host === 'localhost') {
+      assert(address === '127.0.0.1' || address === '::ffff:127.0.0.1');
     } else {
-      console.log('host = ' + host +
-                  ', remoteAddress = ' + socket.remoteAddress);
-      assert.equal(socket.remoteAddress, '::1');
+      console.log('host = ' + host + ', remoteAddress = ' + address);
+      assert.equal(address, '::1');
     }
 
     socket.setEncoding('utf8');