Skip to content

Add Playwright failure screenshot instructions#38307

Merged
juan-fernandez merged 8 commits into
masterfrom
juan-fernandez/playwright-failure-screenshots
Jul 23, 2026
Merged

Add Playwright failure screenshot instructions#38307
juan-fernandez merged 8 commits into
masterfrom
juan-fernandez/playwright-failure-screenshots

Conversation

@juan-fernandez

@juan-fernandez juan-fernandez commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

Adds Playwright guidance for uploading test failure screenshots to Test Optimization. Documents the supported screenshot values and clarifies that when Playwright uses 'on', Test Optimization uploads only screenshots from failed tests.

https://docs-staging.datadoghq.com/juan-fernandez/playwright-failure-screenshots/tests/setup/javascript/?tab=playwright#upload-test-failure-screenshots

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

AI-assisted drafting, source verification, and validation.

Additional notes

Validation: Vale reported no findings on changed lines, git diff --check passed, and Hugo rendered the updated page locally.

@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

juan-fernandez commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

Updated this PR to document the minimum supported dd-trace versions for failure screenshot uploads:

@juan-fernandez

Copy link
Copy Markdown
Contributor Author

/review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤖 Automated review by Claude. AI-generated; verify before acting.

1 style/consistency nit on the new Playwright reference link, plus a minor note on the reused Cypress screenshot image.

Reviewed d9bd627651b0791daf9eb422c6b7428e8856105eworkflow run

Comment thread content/en/tests/setup/javascript.md Outdated
Comment thread content/en/tests/setup/javascript.md Outdated
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@juan-fernandez
juan-fernandez marked this pull request as ready for review July 23, 2026 07:44
@juan-fernandez
juan-fernandez requested review from a team as code owners July 23, 2026 07:44
@github-actions github-actions Bot added the Images Images are added/removed with this PR label Jul 23, 2026

@maycmlee maycmlee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a small suggestion!

Comment thread content/en/tests/setup/javascript.md Outdated
Co-authored-by: May Lee <may.lee@datadoghq.com>
@juan-fernandez
juan-fernandez requested a review from maycmlee July 23, 2026 14:16
@juan-fernandez

Copy link
Copy Markdown
Contributor Author

Just a small suggestion!

@maycmlee thanks! I applied it

@juan-fernandez
juan-fernandez merged commit 36dfafe into master Jul 23, 2026
23 of 24 checks passed
@juan-fernandez
juan-fernandez deleted the juan-fernandez/playwright-failure-screenshots branch July 23, 2026 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Images Images are added/removed with this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants