-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Testsuite console log #15903
Testsuite console log #15903
Conversation
56863d8
to
5ef8451
Compare
This pull request has conflicts ☹ |
I rebased this on v2.5.x-release, as we discussed on the October 31 testing call. |
Thanks for changing the branch! |
This pull request has conflicts ☹ |
5ef8451
to
39a8bf7
Compare
…ion, drops http references
39a8bf7
to
0c32f30
Compare
Committing the API tests (PR #15653) created a merge conflict with this PR because both PRs added npm packages and modified |
Kudos, SonarCloud Quality Gate passed!
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice addition!
Allows testsuite users to print browser console logs to playwright's standard output.
From the updated README:
You can print the browser console log to standard output by setting the environment variable
CONSOLE
:CONSOLE
can be blank (as in the example), or can be a comma-separated list of the following options: