test: fix test-dns.js flakiness
authorRich Trott <rtrott@gmail.com>
Fri, 1 Apr 2016 13:38:00 +0000 (06:38 -0700)
committerMyles Borins <mborins@us.ibm.com>
Mon, 11 Apr 2016 16:34:23 +0000 (12:34 -0400)
Use empty string instead of `www.google.com` for tests where we are just
doing parameter evaluation. This will avoid DNS lookups which appear to
be causing flakiness on Raspberry Pi devices in CI.

PR-URL: https://github.com/nodejs/node/pull/5996
Fixes: https://github.com/nodejs/node/issues/5554
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
test/parallel/test-dns.js

index 0426490..9a2577e 100644 (file)
@@ -1,8 +1,8 @@
 'use strict';
 require('../common');
-var assert = require('assert');
+const assert = require('assert');
 
-var dns = require('dns');
+const dns = require('dns');
 
 var existing = dns.getServers();
 assert(existing.length);
@@ -121,27 +121,27 @@ assert.doesNotThrow(function() {
 });
 
 assert.doesNotThrow(function() {
-  dns.lookup('www.google.com', {
+  dns.lookup('', {
     family: 4,
     hints: 0
   }, noop);
 });
 
 assert.doesNotThrow(function() {
-  dns.lookup('www.google.com', {
+  dns.lookup('', {
     family: 6,
     hints: dns.ADDRCONFIG
   }, noop);
 });
 
 assert.doesNotThrow(function() {
-  dns.lookup('www.google.com', {
+  dns.lookup('', {
     hints: dns.V4MAPPED
   }, noop);
 });
 
 assert.doesNotThrow(function() {
-  dns.lookup('www.google.com', {
+  dns.lookup('', {
     hints: dns.ADDRCONFIG | dns.V4MAPPED
   }, noop);
 });