Skip to content

Conversation

@Skn0tt
Copy link
Member

@Skn0tt Skn0tt commented Aug 19, 2025

The connect call in this test was causing all other browsers in the worker to be closed:

// In pre-launched mode, keep only the pre-launched browser.
for (const b of this._playwright.allBrowsers()) {
if (b !== browser)
await b.close({ reason: 'Connection terminated' });
}

To prevent this contamination, i'm moving this test into subprocess, just like all the other tests in the file.

@Skn0tt Skn0tt requested a review from mxschmitt August 19, 2025 13:04
@Skn0tt Skn0tt self-assigned this Aug 19, 2025
@github-actions

This comment has been minimized.

@Skn0tt Skn0tt merged commit face8d4 into microsoft:main Aug 19, 2025
32 checks passed
@github-actions
Copy link
Contributor

Test results for "tests 1"

4 flaky ⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1079 › cli codegen › should not throw csp directive violation errors `@firefox-ubuntu-22.04-node18`
⚠️ [firefox-page] › page/page-event-request.spec.ts:182 › should return response body when Cross-Origin-Opener-Policy is set `@firefox-ubuntu-22.04-node18`
⚠️ [firefox-page] › page/page-wait-for-function.spec.ts:104 › should work with strict CSP policy `@firefox-ubuntu-22.04-node18`
⚠️ [playwright-test] › ui-mode-test-output.spec.ts:80 › should show console messages for test `@ubuntu-latest-node22-1`

46618 passed, 805 skipped


Merge workflow run.

mxschmitt added a commit that referenced this pull request Aug 19, 2025
Skn0tt added a commit to Skn0tt/playwright that referenced this pull request Oct 1, 2025
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.

2 participants