Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: fix Windows browser process close test #34469

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

agg23
Copy link
Contributor

@agg23 agg23 commented Jan 24, 2025

No description provided.

This comment has been minimized.

@agg23 agg23 added the CQ1 label Jan 24, 2025

This comment has been minimized.

This comment has been minimized.

Copy link
Member

@yury-s yury-s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see any failures on Windows, is it something that you observed locally?

@@ -26,7 +26,7 @@ test('should close the browser when the node process closes', async ({ startRemo
const remoteServer = await startRemoteServer('launchServer', { url: server.EMPTY_PAGE });
try {
if (isWindows)
execSync(`taskkill /pid ${remoteServer.child().pid} /T /F`, { stdio: 'ignore' });
execSync(`taskkill /pid ${remoteServer.child().pid} /T /F`);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How does this help?

@mxschmitt mxschmitt marked this pull request as draft January 27, 2025 10:41
@agg23 agg23 changed the title draft: chore: fix Windows browser process close test chore: fix Windows browser process close test Jan 27, 2025

This comment has been minimized.

@agg23 agg23 added CQ1 and removed CQ1 labels Jan 27, 2025

This comment has been minimized.

This comment has been minimized.

@agg23 agg23 added CQ1 and removed CQ1 labels Jan 27, 2025
Copy link
Contributor

Test results for "tests others"

2 failed
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @frozen-time-library-chromium-linux
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @realtime-time-library-chromium-linux

21564 passed, 503 skipped
✔️✔️✔️

Merge workflow run.

Copy link
Contributor

Test results for "tests 1"

6 failed
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-ubuntu-22.04-node18
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-ubuntu-22.04-node20
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-ubuntu-22.04-node22
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-ubuntu-22.04-node18
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @ubuntu-20.04-chromium-tip-of-tree
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @webkit-ubuntu-22.04-node18

17 flaky ⚠️ [firefox-page] › tests/page/page-evaluate.spec.ts:403:3 › should throw for too deep reference chain @firefox-ubuntu-22.04-node18
⚠️ [installation tests] › tests/playwright-electron-should-work.spec.ts:31:5 › electron should work with special characters in path @package-installations-macos-latest
⚠️ [webkit-library] › tests/library/browsercontext-pages.spec.ts:105:3 › should return bounding box with page scale @webkit-ubuntu-22.04-node18
⚠️ [webkit-library] › tests/library/popup.spec.ts💯3 › should inherit touch support from browser context @webkit-ubuntu-22.04-node18
⚠️ [webkit-library] › tests/library/popup.spec.ts:115:3 › should inherit viewport size from browser context @webkit-ubuntu-22.04-node18
⚠️ [webkit-library] › tests/library/popup.spec.ts:129:3 › should use viewport size from window features @webkit-ubuntu-22.04-node18
⚠️ [webkit-library] › tests/library/proxy.spec.ts:93:11 › should proxy local network requests › with other bypasses › localhost @webkit-ubuntu-22.04-node18
⚠️ [webkit-library] › tests/library/screenshot.spec.ts:278:14 › element screenshot › should restore viewport after page screenshot and exception @webkit-ubuntu-22.04-node18
⚠️ [webkit-library] › tests/library/screenshot.spec.ts:291:14 › element screenshot › should restore viewport after page screenshot and timeout @webkit-ubuntu-22.04-node18
⚠️ [webkit-library] › tests/library/trace-viewer.spec.ts:169:1 › should show action context on locators and other common actions @webkit-ubuntu-22.04-node18
⚠️ [webkit-library] › tests/library/video.spec.ts:411:5 › screencast › should capture css transformation @webkit-ubuntu-22.04-node18
⚠️ [webkit-library] › tests/library/video.spec.ts:441:5 › screencast › should work for popups @webkit-ubuntu-22.04-node18
⚠️ [webkit-page] › tests/page/page-leaks.spec.ts:82:5 › click should not leak @webkit-ubuntu-22.04-node18
⚠️ [webkit-page] › tests/page/page-leaks.spec.ts:107:5 › fill should not leak @webkit-ubuntu-22.04-node18
⚠️ [webkit-page] › tests/page/page-leaks.spec.ts:161:5 › waitFor should not leak @webkit-ubuntu-22.04-node18
⚠️ [webkit-page] › tests/page/page-screenshot.spec.ts:406:5 › page screenshot › should prefer type over extension @webkit-ubuntu-22.04-node18
⚠️ [webkit-page] › tests/page/page-screenshot.spec.ts:868:5 › page screenshot animations › should wait for fonts to load @webkit-ubuntu-22.04-node18

37692 passed, 649 skipped
✔️✔️✔️

Merge workflow run.

Copy link
Contributor

Test results for "tests 2"

3 fatal errors, not part of any test
88 failed
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @channel-chromium-macos-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @channel-chromium-ubuntu-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @channel-chromium-windows-latest
❌ [chromium-library] › tests/library/browsertype-connect.spec.ts:735:5 › run-server › setInputFiles should preserve lastModified timestamp @chrome-beta-macos-latest
❌ [chromium-library] › tests/library/inspector/cli-codegen-csharp.spec.ts:171:5 › should work with --save-har @chrome-beta-macos-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chrome-beta-macos-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chrome-beta-ubuntu-20.04
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chrome-beta-windows-latest
❌ [chromium-library] › tests/library/browsertype-connect.spec.ts:424:5 › run-server › should reject waitForEvent before browser.onDisconnect fires @chrome-macos-latest
❌ [chromium-library] › tests/library/inspector/cli-codegen-python.spec.ts:78:5 › should save the codegen output to a file if specified @chrome-macos-latest
❌ [chromium-library] › tests/library/inspector/cli-codegen-python.spec.ts:133:5 › should work with --save-har @chrome-macos-latest
❌ [chromium-library] › tests/library/inspector/cli-codegen-test.spec.ts:88:5 › should not generate recordHAR with --save-har @chrome-macos-latest
❌ [chromium-library] › tests/library/inspector/cli-codegen-test.spec.ts:99:5 › should generate routeFromHAR with --save-har @chrome-macos-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chrome-macos-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chrome-ubuntu-20.04
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chrome-windows-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-headed-macos-14-xlarge
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-headed-ubuntu-24.04
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-headed-windows-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-macos-13-large
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-macos-13-xlarge
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-macos-14-large
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-macos-14-xlarge
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-tip-of-tree-headless-shell-ubuntu-20.04
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-tip-of-tree-macos-13
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-tip-of-tree-macos-13--headed
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-tip-of-tree-ubuntu-20.04
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-tip-of-tree-ubuntu-20.04--headed
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-tip-of-tree-windows-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-tip-of-tree-windows-latest--headed
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-ubuntu-20.04
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-ubuntu-24.04
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @chromium-windows-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @driver
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-beta-macos-latest
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-beta-ubuntu-20.04
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-beta-windows-latest
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-headed-macos-14-xlarge
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-headed-ubuntu-24.04
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-headed-windows-latest
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-macos-13-large
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-macos-13-xlarge
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-macos-14-large
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-macos-14-xlarge
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-ubuntu-20.04
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-ubuntu-24.04
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @firefox-windows-latest
❌ [chromium-library] › tests/library/downloads-path.spec.ts:91:5 › downloads path › should accept downloads in persistent context @msedge-beta-macos-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @msedge-beta-macos-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @msedge-beta-ubuntu-20.04
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @msedge-beta-windows-latest
❌ [chromium-library] › tests/library/downloads-path.spec.ts:63:5 › downloads path › should report downloads in downloadsPath folder @msedge-dev-macos-latest
❌ [chromium-library] › tests/library/downloads-path.spec.ts:77:5 › downloads path › should report downloads in downloadsPath folder with a relative path @msedge-dev-macos-latest
❌ [chromium-library] › tests/library/downloads-path.spec.ts:91:5 › downloads path › should accept downloads in persistent context @msedge-dev-macos-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @msedge-dev-macos-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @msedge-dev-ubuntu-20.04
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @msedge-dev-windows-latest
❌ [chromium-library] › tests/library/browsertype-connect.spec.ts:898:7 › run-server › socks proxy › should check proxy pattern on the client @msedge-macos-latest
❌ [chromium-library] › tests/library/inspector/cli-codegen-2.spec.ts:454:7 › cli codegen › should save assets via SIGINT @msedge-macos-latest
❌ [chromium-library] › tests/library/proxy.spec.ts:93:11 › should proxy local network requests › by default › loopback address @msedge-macos-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @msedge-macos-latest
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @msedge-ubuntu-20.04
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @service
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @tracing-chromium
❌ [chromium-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @tracing-chromium-tip-of-tree
❌ [firefox-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @tracing-firefox
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @tracing-webkit
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @webkit-headed-macos-14-xlarge
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @webkit-headed-ubuntu-20.04
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @webkit-headed-ubuntu-22.04
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @webkit-headed-ubuntu-24.04
❌ [webkit-library] › tests/library/inspector/cli-codegen-3.spec.ts:636:7 › cli codegen › should consume pointer events @webkit-headed-windows-latest
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @webkit-headed-windows-latest
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @webkit-macos-13-xlarge
❌ [webkit-library] › tests/library/tracing.spec.ts:432:14 › should produce screencast frames fit @webkit-macos-13-xlarge
❌ [webkit-library] › tests/library/tracing.spec.ts:432:14 › should produce screencast frames crop @webkit-macos-13-xlarge
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @webkit-macos-14-large
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @webkit-macos-14-xlarge
❌ [webkit-library] › tests/library/tracing.spec.ts:432:14 › should produce screencast frames fit @webkit-macos-14-xlarge
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @webkit-macos-15-xlarge
❌ [webkit-library] › tests/library/tracing.spec.ts:432:14 › should produce screencast frames crop @webkit-macos-15-xlarge
❌ [webkit-library] › tests/library/tracing.spec.ts:432:14 › should produce screencast frames scale @webkit-macos-15-xlarge
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @webkit-ubuntu-20.04
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @webkit-ubuntu-24.04
❌ [webkit-page] › tests/page/page-leaks.spec.ts:82:5 › click should not leak @webkit-ubuntu-24.04
❌ [webkit-page] › tests/page/page-leaks.spec.ts:107:5 › fill should not leak @webkit-ubuntu-24.04
❌ [webkit-page] › tests/page/page-leaks.spec.ts:161:5 › waitFor should not leak @webkit-ubuntu-24.04
❌ [webkit-library] › tests/library/signals.spec.ts:25:5 › should close the browser when the node process closes @webkit-windows-latest

140 flaky ⚠️ [chromium-library] › tests/library/browser.spec.ts:54:5 › should dispatch page.on(close) upon browser.close and reject evaluate @channel-chromium-macos-latest
⚠️ [chromium-library] › tests/library/downloads-path.spec.ts:48:5 › downloads path › should delete downloads when context closes @channel-chromium-macos-latest
⚠️ [chromium-library] › tests/library/har.spec.ts:85:3 › should have pages in persistent context @channel-chromium-macos-latest
⚠️ [chromium-library] › tests/library/video.spec.ts:379:5 › screencast › should capture navigation @channel-chromium-macos-latest
⚠️ [chromium-library] › tests/library/video.spec.ts:379:5 › screencast › should capture navigation @channel-chromium-ubuntu-latest
⚠️ [chromium-library] › tests/library/browsercontext-base-url.spec.ts:37:3 › should construct a new URL when a baseURL in browserType.launchPersistentContext is passed to page.goto @chrome-beta-macos-latest
⚠️ [chromium-library] › tests/library/browsertype-connect.spec.ts:667:5 › run-server › should fulfill with global fetch result @chrome-beta-macos-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-2.spec.ts:454:7 › cli codegen › should save assets via SIGINT @chrome-beta-macos-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-csharp.spec.ts:182:5 › should work with --save-har and --save-har-glob @chrome-beta-macos-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-test.spec.ts:88:5 › should not generate recordHAR with --save-har @chrome-beta-macos-latest
⚠️ [chromium-library] › tests/library/inspector/pause.spec.ts:23:3 › should resume when closing inspector @chrome-beta-macos-latest
⚠️ [chromium-library] › tests/library/proxy.spec.ts:31:3 › should use proxy @smoke @chrome-beta-macos-latest
⚠️ [chromium-page] › tests/page/page-event-request.spec.ts:124:3 › should report navigation requests and responses handled by service worker @chrome-beta-ubuntu-20.04
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-1.spec.ts:23:7 › cli codegen › should click @chrome-beta-windows-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-2.spec.ts:24:7 › cli codegen › should contain open page @chrome-beta-windows-latest
⚠️ [chromium-library] › tests/library/video.spec.ts:379:5 › screencast › should capture navigation @chrome-beta-windows-latest
⚠️ [chromium-library] › tests/library/browser.spec.ts:54:5 › should dispatch page.on(close) upon browser.close and reject evaluate @chrome-macos-latest
⚠️ [chromium-library] › tests/library/browsercontext-proxy.spec.ts:27:3 › should work when passing the proxy only on the context level @chrome-macos-latest
⚠️ [chromium-library] › tests/library/browsertype-connect.spec.ts:141:5 › launchServer › should be able to reconnect to a browser @chrome-macos-latest
⚠️ [chromium-library] › tests/library/browsertype-connect.spec.ts:436:5 › run-server › should respect selectors @chrome-macos-latest
⚠️ [chromium-library] › tests/library/browsertype-connect.spec.ts:492:5 › run-server › should saveAs videos from remote browser @chrome-macos-latest
⚠️ [chromium-library] › tests/library/browsertype-launch-server.spec.ts:23:5 › launch server › should work @chrome-macos-latest
⚠️ [chromium-library] › tests/library/chromium/oopif.spec.ts:235:3 › should report google.com frame with headed @chrome-macos-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-2.spec.ts:454:7 › cli codegen › should save assets via SIGINT @chrome-macos-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-csharp.spec.ts:171:5 › should work with --save-har @chrome-macos-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-pytest.spec.ts:57:5 › should save the codegen output to a file if specified @chrome-macos-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-python.spec.ts:144:5 › should work with --save-har and --save-har-glob @chrome-macos-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-test.spec.ts:112:5 › should generate routeFromHAR with --save-har and --save-har-glob @chrome-macos-latest
⚠️ [chromium-library] › tests/library/selector-generator.spec.ts:39:5 › selector generator › should prefer button over inner span @chrome-ubuntu-20.04
⚠️ [chromium-library] › tests/library/video.spec.ts:379:5 › screencast › should capture navigation @chrome-ubuntu-20.04
⚠️ [chromium-library] › tests/library/video.spec.ts:580:5 › screencast › should capture static page in persistent context @smoke @chrome-windows-latest
⚠️ [chromium-library] › tests/library/video.spec.ts:163:5 › screencast › should work with old options @chromium-headed-macos-14-xlarge
⚠️ [chromium-page] › tests/page/page-mouse.spec.ts:244:3 › should tween mouse movement @chromium-headed-ubuntu-24.04
⚠️ [chromium-library] › tests/library/video.spec.ts:379:5 › screencast › should capture navigation @chromium-tip-of-tree-macos-13--headed
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-1.spec.ts:23:7 › cli codegen › should click @chromium-tip-of-tree-windows-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-2.spec.ts:24:7 › cli codegen › should contain open page @chromium-tip-of-tree-windows-latest
⚠️ [firefox-page] › tests/page/page-evaluate.spec.ts:403:3 › should throw for too deep reference chain @firefox-beta-ubuntu-20.04
⚠️ [firefox-library] › tests/library/inspector/cli-codegen-3.spec.ts:222:7 › cli codegen › should generate frame locators (4) @firefox-beta-windows-latest
⚠️ [firefox-page] › tests/page/page-evaluate.spec.ts:403:3 › should throw for too deep reference chain @firefox-beta-windows-latest
⚠️ [firefox-library] › tests/library/capabilities.spec.ts:252:3 › requestFullscreen @firefox-headed-macos-14-xlarge
⚠️ [firefox-library] › tests/library/inspector/cli-codegen-1.spec.ts:116:7 › cli codegen › should click after same-document navigation @firefox-headed-macos-14-xlarge
⚠️ [firefox-library] › tests/library/emulation-focus.spec.ts:190:12 › should not fire blur events when interacting with more than one page/context @firefox-headed-ubuntu-24.04
⚠️ [firefox-page] › tests/page/page-evaluate.spec.ts:403:3 › should throw for too deep reference chain @firefox-headed-ubuntu-24.04
⚠️ [firefox-library] › tests/library/browsercontext-add-cookies.spec.ts:141:3 › should isolate persistent cookies @firefox-headed-windows-latest
⚠️ [firefox-library] › tests/library/browsercontext-locale.spec.ts:118:3 › should be isolated between contexts @firefox-headed-windows-latest
⚠️ [firefox-library] › tests/library/browsercontext-reuse.spec.ts:206:1 › should reset mouse position @firefox-headed-windows-latest
⚠️ [firefox-library] › tests/library/inspector/cli-codegen-1.spec.ts:643:7 › cli codegen › should select with size attribute @firefox-headed-windows-latest
⚠️ [firefox-library] › tests/library/trace-viewer.spec.ts:1175:1 › should update highlight when typing snapshot @firefox-headed-windows-latest
⚠️ [firefox-page] › tests/page/page-click-timeout-4.spec.ts:20:3 › should timeout waiting for stable position @firefox-headed-windows-latest
⚠️ [firefox-page] › tests/page/page-evaluate.spec.ts:403:3 › should throw for too deep reference chain @firefox-headed-windows-latest
⚠️ [firefox-page] › tests/page/page-evaluate.spec.ts:403:3 › should throw for too deep reference chain @firefox-macos-13-large
⚠️ [firefox-library] › tests/library/tracing.spec.ts:432:14 › should produce screencast frames crop @firefox-macos-13-xlarge
⚠️ [firefox-library] › tests/library/video.spec.ts:163:5 › screencast › should work with old options @firefox-macos-14-large
⚠️ [firefox-page] › tests/page/page-evaluate.spec.ts:403:3 › should throw for too deep reference chain @firefox-macos-14-large
⚠️ [firefox-library] › tests/library/inspector/cli-codegen-1.spec.ts:116:7 › cli codegen › should click after same-document navigation @firefox-macos-14-xlarge
⚠️ [firefox-library] › tests/library/inspector/cli-codegen-aria.spec.ts:76:7 › should update aria snapshot highlight @firefox-macos-14-xlarge
⚠️ [firefox-library] › tests/library/tracing.spec.ts:432:14 › should produce screencast frames crop @firefox-macos-14-xlarge
⚠️ [firefox-page] › tests/page/page-evaluate.spec.ts:403:3 › should throw for too deep reference chain @firefox-ubuntu-20.04
⚠️ [firefox-page] › tests/page/page-event-request.spec.ts:183:3 › should return response body when Cross-Origin-Opener-Policy is set @firefox-ubuntu-20.04
⚠️ [firefox-page] › tests/page/page-evaluate.spec.ts:403:3 › should throw for too deep reference chain @firefox-ubuntu-24.04
⚠️ [firefox-page] › tests/page/page-evaluate.spec.ts:403:3 › should throw for too deep reference chain @firefox-windows-latest
⚠️ [chromium-library] › tests/library/beforeunload.spec.ts:20:3 › should close browser with beforeunload page @msedge-beta-macos-latest
⚠️ [chromium-library] › tests/library/browsercontext-base-url.spec.ts:37:3 › should construct a new URL when a baseURL in browserType.launchPersistentContext is passed to page.goto @msedge-beta-macos-latest
⚠️ [chromium-library] › tests/library/browsercontext-proxy.spec.ts:27:3 › should work when passing the proxy only on the context level @msedge-beta-macos-latest
⚠️ [chromium-library] › tests/library/browsercontext-reuse.spec.ts:30:1 › should re-add binding after reset @msedge-beta-macos-latest
⚠️ [chromium-library] › tests/library/browsertype-launch-server.spec.ts:23:5 › launch server › should work @msedge-beta-macos-latest
⚠️ [chromium-library] › tests/library/downloads-path.spec.ts:77:5 › downloads path › should report downloads in downloadsPath folder with a relative path @msedge-beta-macos-latest
⚠️ [chromium-library] › tests/library/downloads-path.spec.ts:105:5 › downloads path › should delete downloads when persistent context closes @msedge-beta-macos-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-2.spec.ts:454:7 › cli codegen › should save assets via SIGINT @msedge-beta-macos-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-csharp.spec.ts:171:5 › should work with --save-har @msedge-beta-macos-latest
⚠️ [chromium-library] › tests/library/browsercontext-reuse.spec.ts:30:1 › should re-add binding after reset @msedge-beta-ubuntu-20.04
⚠️ [chromium-library] › tests/library/video.spec.ts:379:5 › screencast › should capture navigation @msedge-beta-ubuntu-20.04
⚠️ [chromium-library] › tests/library/browsercontext-reuse.spec.ts:30:1 › should re-add binding after reset @msedge-beta-windows-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-1.spec.ts:23:7 › cli codegen › should click @msedge-beta-windows-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-2.spec.ts:24:7 › cli codegen › should contain open page @msedge-beta-windows-latest
⚠️ [chromium-library] › tests/library/trace-viewer.spec.ts:951:1 › should highlight expect failure @msedge-beta-windows-latest
⚠️ [chromium-library] › tests/library/beforeunload.spec.ts:20:3 › should close browser with beforeunload page @msedge-dev-macos-latest
⚠️ [chromium-library] › tests/library/browsercontext-base-url.spec.ts:37:3 › should construct a new URL when a baseURL in browserType.launchPersistentContext is passed to page.goto @msedge-dev-macos-latest
⚠️ [chromium-library] › tests/library/browsercontext-fetch.spec.ts:1229:3 › should work with connectOverCDP @msedge-dev-macos-latest
⚠️ [chromium-library] › tests/library/browsertype-connect.spec.ts:161:5 › launchServer › should be able to visit ipv6 @msedge-dev-macos-latest
⚠️ [chromium-library] › tests/library/browsertype-launch-server.spec.ts:29:5 › launch server › should work with host @msedge-dev-macos-latest
⚠️ [chromium-library] › tests/library/downloads-path.spec.ts:48:5 › downloads path › should delete downloads when context closes @msedge-dev-macos-latest
⚠️ [chromium-library] › tests/library/browsercontext-reuse.spec.ts:30:1 › should re-add binding after reset @msedge-dev-ubuntu-20.04
⚠️ [chromium-library] › tests/library/browsercontext-reuse.spec.ts:30:1 › should re-add binding after reset @msedge-dev-windows-latest
⚠️ [chromium-library] › tests/library/browsertype-connect.spec.ts:492:5 › run-server › should saveAs videos from remote browser @msedge-dev-windows-latest
⚠️ [chromium-library] › tests/library/browsercontext-base-url.spec.ts:37:3 › should construct a new URL when a baseURL in browserType.launchPersistentContext is passed to page.goto @msedge-macos-latest
⚠️ [chromium-library] › tests/library/browsercontext-reuse.spec.ts:30:1 › should re-add binding after reset @msedge-macos-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-csharp.spec.ts:171:5 › should work with --save-har @msedge-macos-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-java.spec.ts:92:5 › should work with --save-har and --save-har-glob as java-library @msedge-macos-latest
⚠️ [chromium-library] › tests/library/inspector/cli-codegen-test.spec.ts:99:5 › should generate routeFromHAR with --save-har @msedge-macos-latest
⚠️ [chromium-library] › tests/library/inspector/pause.spec.ts:23:3 › should resume when closing inspector @msedge-macos-latest
⚠️ [chromium-library] › tests/library/proxy.spec.ts:31:3 › should use proxy @smoke @msedge-macos-latest
⚠️ [chromium-library] › tests/library/proxy.spec.ts:93:11 › should proxy local network requests › by default › link-local @msedge-macos-latest
⚠️ [chromium-library] › tests/library/trace-viewer.spec.ts:1298:1 › should pick locator in iframe @msedge-macos-latest
⚠️ [chromium-library] › tests/library/tracing.spec.ts:370:5 › should survive browser.close with auto-created traces dir @msedge-macos-latest
⚠️ [chromium-library] › tests/library/video.spec.ts:379:5 › screencast › should capture navigation @msedge-macos-latest
⚠️ [chromium-library] › tests/library/browsercontext-reuse.spec.ts:30:1 › should re-add binding after reset @msedge-ubuntu-20.04
⚠️ [chromium-library] › tests/library/video.spec.ts:379:5 › screencast › should capture navigation @tracing-chromium-tip-of-tree
⚠️ [firefox-page] › tests/page/page-evaluate.spec.ts:403:3 › should throw for too deep reference chain @tracing-firefox
⚠️ [webkit-library] › tests/library/browsercontext-viewport-mobile.spec.ts:157:5 › mobile viewport › mouse should work with mobile viewports and cross process navigations @tracing-webkit
⚠️ [webkit-library] › tests/library/screenshot.spec.ts:55:14 › page screenshot › should work with a mobile viewport and clip @tracing-webkit
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-1.spec.ts:610:7 › cli codegen › should select @webkit-headed-macos-14-xlarge
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-1.spec.ts:146:7 › cli codegen › should make a positioned click on a canvas @webkit-headed-ubuntu-20.04
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-1.spec.ts:199:7 › cli codegen › should work with TrustedTypes @webkit-headed-ubuntu-20.04
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-1.spec.ts:540:7 › cli codegen › should check a radio button @webkit-headed-ubuntu-20.04
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-3.spec.ts:607:7 › cli codegen › should generate getByLabel without regex @webkit-headed-ubuntu-22.04
⚠️ [webkit-library] › tests/library/trace-viewer.spec.ts:966:1 › should show action source @webkit-headed-ubuntu-22.04
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-1.spec.ts:55:7 › cli codegen › should double click @webkit-headed-ubuntu-24.04
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-1.spec.ts:116:7 › cli codegen › should click after same-document navigation @webkit-headed-ubuntu-24.04
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-3.spec.ts:488:7 › cli codegen › should generate getByTestId @webkit-headed-ubuntu-24.04
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-3.spec.ts:737:7 › cli codegen › should assert value @webkit-headed-ubuntu-24.04
⚠️ [webkit-page] › tests/page/page-click.spec.ts:261:3 › should click on checkbox input and toggle @webkit-headed-ubuntu-24.04
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-2.spec.ts:473:7 › cli codegen › should fill tricky characters @webkit-headed-windows-latest
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-3.spec.ts:520:7 › cli codegen › should generate getByPlaceholder @webkit-headed-windows-latest
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-3.spec.ts:578:7 › cli codegen › should generate getByLabel @webkit-headed-windows-latest
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-3.spec.ts:737:7 › cli codegen › should assert value @webkit-headed-windows-latest
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-3.spec.ts:784:7 › cli codegen › should assert value on disabled input @webkit-headed-windows-latest
⚠️ [webkit-library] › tests/library/inspector/cli-codegen-3.spec.ts:830:7 › cli codegen › should assert visibility @webkit-headed-windows-latest
⚠️ [webkit-library] › tests/library/tracing.spec.ts:432:14 › should produce screencast frames scale @webkit-macos-13-xlarge
⚠️ [webkit-page] › tests/page/page-request-continue.spec.ts:185:5 › post data › should amend post data @webkit-macos-13-xlarge
⚠️ [webkit-library] › tests/library/video.spec.ts:379:5 › screencast › should capture navigation @webkit-macos-14-large
⚠️ [webkit-library] › tests/library/tracing.spec.ts:432:14 › should produce screencast frames crop @webkit-macos-14-xlarge
⚠️ [webkit-library] › tests/library/tracing.spec.ts:432:14 › should produce screencast frames scale @webkit-macos-14-xlarge
⚠️ [webkit-page] › tests/page/page-request-continue.spec.ts:215:5 › post data › should amend method and post data @webkit-macos-14-xlarge
⚠️ [webkit-library] › tests/library/browsertype-connect.spec.ts:609:5 › run-server › should save har @webkit-macos-15-xlarge
⚠️ [webkit-library] › tests/library/tracing.spec.ts:432:14 › should produce screencast frames fit @webkit-macos-15-xlarge
⚠️ [webkit-library] › tests/library/browsercontext-device.spec.ts:45:5 › device › should scroll to click @webkit-ubuntu-20.04
⚠️ [webkit-library] › tests/library/screenshot.spec.ts:66:14 › page screenshot › should work with a mobile viewport and fullPage @webkit-ubuntu-20.04
⚠️ [webkit-library] › tests/library/screenshot.spec.ts:206:14 › element screenshot › element screenshot should work with a mobile viewport @webkit-ubuntu-20.04
⚠️ [webkit-library] › tests/library/browsercontext-pages.spec.ts:105:3 › should return bounding box with page scale @webkit-ubuntu-24.04
⚠️ [webkit-library] › tests/library/browsercontext-viewport-mobile.spec.ts:87:5 › mobile viewport › should support window.orientation emulation @webkit-ubuntu-24.04
⚠️ [webkit-library] › tests/library/browsercontext-viewport-mobile.spec.ts:97:5 › mobile viewport › should fire orientationchange event @webkit-ubuntu-24.04
⚠️ [webkit-library] › tests/library/browsercontext-viewport-mobile.spec.ts:124:5 › mobile viewport › respect meta viewport tag @webkit-ubuntu-24.04
⚠️ [webkit-library] › tests/library/browsercontext-viewport-mobile.spec.ts:157:5 › mobile viewport › mouse should work with mobile viewports and cross process navigations @webkit-ubuntu-24.04
⚠️ [webkit-library] › tests/library/browsercontext-viewport-mobile.spec.ts:206:5 › mobile viewport › view scale should reset after navigation @webkit-ubuntu-24.04
⚠️ [webkit-library] › tests/library/popup.spec.ts:115:3 › should inherit viewport size from browser context @webkit-ubuntu-24.04
⚠️ [webkit-library] › tests/library/screenshot.spec.ts:44:14 › page screenshot › should work with a mobile viewport @webkit-ubuntu-24.04
⚠️ [webkit-library] › tests/library/video.spec.ts:441:5 › screencast › should work for popups @webkit-ubuntu-24.04
⚠️ [webkit-page] › tests/page/page-screenshot.spec.ts:345:5 › page screenshot › should work while navigating @webkit-ubuntu-24.04
⚠️ [webkit-page] › tests/page/page-screenshot.spec.ts:868:5 › page screenshot animations › should wait for fonts to load @webkit-ubuntu-24.04

254253 passed, 9763 skipped
✔️✔️✔️

Merge workflow run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants