test: handle IPv6 localhost issues within tests · nodejs/node@814b8c3

test: handle IPv6 localhost issues within tests · nodejs/node@814b8c3

3 files changed

lines changed

Original file line number	Diff line number	Diff line change
`@@ -12,13 +12,6 @@ test-tick-processor : PASS,FLAKY`
`12`	`12`	`[$system==linux]`
`13`	`13`	`test-tick-processor : PASS,FLAKY`
`14`	`14`
`15`		`-# Flaky until https://github.com/nodejs/build/issues/415 is resolved.`
`16`		`-# On some of the buildbots, AAAA queries for localhost don't resolve`
`17`		`-# to an address and neither do any of the alternatives from the`
`18`		`-# localIPv6Hosts list from test/common.js.`
`19`		`-test-https-connect-address-family : PASS,FLAKY`
`20`		`-test-tls-connect-address-family : PASS,FLAKY`
`21`		`-`
`22`	`15`	`[$system==macos]`
`23`	`16`
`24`	`17`	`[$system==solaris] # Also applies to SmartOS`

Original file line number	Diff line number	Diff line change
`@@ -36,8 +36,13 @@ function runTest() {`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`dns.lookup('localhost', {family: 6, all: true}, (err, addresses) => {`
`39`		`-if (err)`
	`39`	`+if (err) {`
	`40`	`+if (err.code === 'ENOTFOUND') {`
	`41`	`+common.skip('localhost does not resolve to ::1');`
	`42`	`+return;`
	`43`	`+}`
`40`	`44`	`throw err;`
	`45`	`+}`
`41`	`46`
`42`	`47`	`if (addresses.some((val) => val.address === '::1'))`
`43`	`48`	`runTest();`

Original file line number	Diff line number	Diff line change
`@@ -35,8 +35,13 @@ function runTest() {`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`dns.lookup('localhost', {family: 6, all: true}, (err, addresses) => {`
`38`		`-if (err)`
	`38`	`+if (err) {`
	`39`	`+if (err.code === 'ENOTFOUND') {`
	`40`	`+common.skip('localhost does not resolve to ::1');`
	`41`	`+return;`
	`42`	`+}`
`39`	`43`	`throw err;`
	`44`	`+}`
`40`	`45`
`41`	`46`	`if (addresses.some((val) => val.address === '::1'))`
`42`	`47`	`runTest();`