Skip to content

feat(test runner): use happy eyeballs in webServer port check#40458

Merged
Skn0tt merged 1 commit intomicrosoft:mainfrom
Skn0tt:feat-isPortUsed-happy-eyeballs
Apr 28, 2026
Merged

feat(test runner): use happy eyeballs in webServer port check#40458
Skn0tt merged 1 commit intomicrosoft:mainfrom
Skn0tt:feat-isPortUsed-happy-eyeballs

Conversation

@Skn0tt
Copy link
Copy Markdown
Member

@Skn0tt Skn0tt commented Apr 28, 2026

Closes #40430

isPortUsed now delegates to createSocket from @utils/happyEyeballs
instead of probing 127.0.0.1 then ::1 sequentially.

Also re-export happyEyeballs from @utils/index so the cross-package
@utils/* -> coreBundle.utils.* build rewrite resolves createSocket
for callers outside playwright-core.
@github-actions
Copy link
Copy Markdown
Contributor

Test results for "tests 1"

13 flaky ⚠️ [installation tests] › playwright-cli.spec.ts:21 › cli should work `@package-installations-ubuntu-latest`
⚠️ [chromium-library] › library/chromium/connect-over-cdp.spec.ts:449 › should be able to connect via localhost `@ubuntu-22.04-chromium-tip-of-tree`
⚠️ [chromium-library] › library/video.spec.ts:476 › screencast › should capture static page in persistent context @smoke `@ubuntu-22.04-chromium-tip-of-tree`
⚠️ [chromium-library] › library/video.spec.ts:682 › screencast › should capture full viewport on hidpi `@ubuntu-22.04-chromium-tip-of-tree`
⚠️ [chromium-library] › library/chromium/connect-over-cdp.spec.ts:449 › should be able to connect via localhost `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/chromium/connect-over-cdp.spec.ts:449 › should be able to connect via localhost `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/video.spec.ts:719 › screencast › should work with video+trace `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/chromium/connect-over-cdp.spec.ts:449 › should be able to connect via localhost `@chromium-ubuntu-22.04-node20`
⚠️ [chromium-library] › library/video.spec.ts:719 › screencast › should work with video+trace `@chromium-ubuntu-22.04-node20`
⚠️ [chromium-library] › library/chromium/connect-over-cdp.spec.ts:449 › should be able to connect via localhost `@chromium-ubuntu-22.04-node22`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1080 › cli codegen › should not throw csp directive violation errors `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-page] › page/page-wait-for-function.spec.ts:104 › should work with strict CSP policy `@firefox-ubuntu-22.04-node20`
⚠️ [playwright-test] › ui-mode-trace.spec.ts:700 › should indicate current test status `@windows-latest-node20`

41466 passed, 847 skipped


Merge workflow run.

@github-actions
Copy link
Copy Markdown
Contributor

Test results for "MCP"

6808 passed, 927 skipped


Merge workflow run.

@Skn0tt Skn0tt requested a review from dgozman April 28, 2026 08:27
@Skn0tt Skn0tt merged commit 39109eb into microsoft:main Apr 28, 2026
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: ECONNREFUSED, PW waits for a 2 mins timeout to start a server in WSL

2 participants