Skip to content

Conversation

@dgozman
Copy link
Contributor

@dgozman dgozman commented Sep 17, 2025

This allows someone to not listen for all console messages or page errors, but retrive the last ones when needed, currently up to 200.

Especially handy when connecting to a page later on.

@github-actions

This comment has been minimized.

Copy link
Member

@pavelfeldman pavelfeldman left a comment

Choose a reason for hiding this comment

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

consoleMessage() & pageErrors(). What do you think about network requests?

This allows someone to not listen for all console messages or
page errors, but retrive the last ones when needed, currently up to 200.

Especially handy when connecting to a page later on.
@dgozman dgozman force-pushed the feat-last-console-page-errors branch from e464423 to 9d0ddc6 Compare September 18, 2025 10:57
@dgozman dgozman changed the title feat: page.lastConsoleMessages and page.lastPageErrors feat: page.consoleMessages and page.pageErrors Sep 18, 2025
@github-actions
Copy link
Contributor

Test results for "tests 1"

3 failed
❌ [playwright-test] › runner.spec.ts:118 › should ignore subprocess creation error because of SIGINT @macos-latest-node18-1
❌ [installation tests] › playwright-test-package-managers.spec.ts:54 › npm: uninstalling ct removes playwright bin @package-installations-macos-latest
❌ [installation tests] › playwright-electron-should-work.spec.ts:21 › electron should work @package-installations-windows-latest

10 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 `@macos-latest-node18-1`
⚠️ [installation tests] › playwright-component-testing.spec.ts:21 › pnpm: @playwright/experimental-ct-react should work `@package-installations-windows-latest`
⚠️ [installation tests] › playwright-electron-should-work.spec.ts:31 › electron should work with special characters in path `@package-installations-windows-latest`
⚠️ [installation tests] › playwright-electron-should-work.spec.ts:44 › should work when wrapped inside @playwright/test and trace is enabled `@package-installations-windows-latest`
⚠️ [installation tests] › playwright-packages-install-behavior.spec.ts:24 › playwright-firefox should work `@package-installations-windows-latest`
⚠️ [installation tests] › playwright-packages-install-behavior.spec.ts:40 › @playwright/browser-chromium should work `@package-installations-windows-latest`
⚠️ [webkit-library] › library/trace-viewer.spec.ts:1934 › should load trace from HTTP with progress indicator `@webkit-ubuntu-22.04-node18`

46881 passed, 821 skipped


Merge workflow run.

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