Skip to content

test: fix test-dns.js flakiness#5996

Closed
Trott wants to merge 1 commit intonodejs:masterfrom
Trott:void
Closed

test: fix test-dns.js flakiness#5996
Trott wants to merge 1 commit intonodejs:masterfrom
Trott:void

Conversation

@Trott
Copy link
Member

@Trott Trott commented Apr 1, 2016

Pull Request check-list

  • Does make -j8 test (UNIX) or vcbuild test nosign (Windows) pass with
    this change (including linting)?
  • Is the commit message formatted according to [CONTRIBUTING.md][0]?

Affected core subsystem(s)

test, dns

Description of change

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.

Fixes: #5554

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dns Issues and PRs related to the dns subsystem. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants