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

docs: Trace-viewer-intro #16254

Merged
merged 9 commits into from
Aug 10, 2022
Merged

Conversation

debs-obrien
Copy link
Contributor

@debs-obrien debs-obrien commented Aug 4, 2022

image

@debs-obrien debs-obrien marked this pull request as ready for review August 4, 2022 16:42
@debs-obrien debs-obrien marked this pull request as draft August 4, 2022 17:54
@debs-obrien debs-obrien marked this pull request as ready for review August 5, 2022 17:34
context = await browser.new_context()

# Start tracing before creating / navigating a page.
await context.tracing.start(screenshots=True, snapshots=True, sources=True)
Copy link
Member

Choose a reason for hiding this comment

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

We should mention the pytest plugin way there instead of the library approach.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

you will have to add what you mean here as no idea what the pytest plugin way is

Copy link
Member

Choose a reason for hiding this comment

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

Available options to record a trace:
- `'on-first-retry'` - Record a trace only when retrying a test for the first time.
Copy link
Member

Choose a reason for hiding this comment

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

Mention to make sure to have retries >= 1?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This doc needs a full rework which will be part of another pr

@mxschmitt mxschmitt merged commit 05dc5cf into microsoft:main Aug 10, 2022
mxschmitt pushed a commit to mxschmitt/playwright that referenced this pull request Aug 10, 2022
mxschmitt added a commit that referenced this pull request Aug 10, 2022
* docs: Remove html reporter page (#16407)

* docs: intro doc on ci (#16346)

* docs: Trace-viewer-intro (#16254)

Co-authored-by: Debbie O'Brien <debs-obrien@users.noreply.github.com>
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.

None yet

2 participants