-
Notifications
You must be signed in to change notification settings - Fork 0
Improve Playwright Check Suites docs #48
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
Conversation
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.
Pull Request Overview
This PR improves Playwright Check Suites documentation by adding a comprehensive test organization guide and updating existing documentation for clarity and accuracy.
- Adds a new "test-organization.mdx" document explaining how to structure and organize Playwright tests for synthetic monitoring
- Updates existing documentation to improve clarity, fix typos, and update outdated references
- Removes the quickstart.mdx file (likely moved elsewhere based on reference updates)
Reviewed Changes
Copilot reviewed 7 out of 12 changed files in this pull request and generated 6 comments.
Show a summary per file
File | Description |
---|---|
docs.json | Adds the new test organization page to the documentation navigation |
test-organization.mdx | New comprehensive guide covering test organization strategies and examples |
quickstart.mdx | Removes the quickstart guide (moved to different location) |
overview.mdx | Updates terminology, fixes references, and improves code examples |
custom-dependencies.mdx | Improves formatting and clarity of dependency management instructions |
configuration.mdx | Updates code examples and improves documentation structure |
add-to-group.mdx | Cleans up code examples and updates reference links |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
detect/synthetic-monitoring/playwright-checks/test-organization.mdx
Outdated
Show resolved
Hide resolved
detect/synthetic-monitoring/playwright-checks/test-organization.mdx
Outdated
Show resolved
Hide resolved
detect/synthetic-monitoring/playwright-checks/test-organization.mdx
Outdated
Show resolved
Hide resolved
detect/synthetic-monitoring/playwright-checks/test-organization.mdx
Outdated
Show resolved
Hide resolved
detect/synthetic-monitoring/playwright-checks/test-organization.mdx
Outdated
Show resolved
Hide resolved
detect/synthetic-monitoring/playwright-checks/configuration.mdx
Outdated
Show resolved
Hide resolved
…n.mdx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…n.mdx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…n.mdx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…n.mdx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…n.mdx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
Pull Request Overview
Copilot reviewed 7 out of 12 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
detect/synthetic-monitoring/playwright-checks/test-organization.mdx
Outdated
Show resolved
Hide resolved
detect/synthetic-monitoring/playwright-checks/test-organization.mdx
Outdated
Show resolved
Hide resolved
…n.mdx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…n.mdx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
Pull Request Overview
Copilot reviewed 7 out of 12 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
// Pick the Playwright project for the staging environment | ||
pwProjects: ["user-sign-up"], |
Copilot
AI
Oct 2, 2025
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.
The comment mentions 'staging environment' but the code is configuring a 'user-sign-up' project. This comment appears to be copied from the environment grouping example above and should be updated to reflect the actual functionality.
Copilot uses AI. Check for mistakes.
detect/synthetic-monitoring/playwright-checks/test-organization.mdx
Outdated
Show resolved
Hide resolved
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.
Wow, thanks for the thorough update and all the examples, these are all great @stefanjudis !
I've done a swipe for clarity and accuracy, please do accept/tweak as you see fit and let me know where I can help!
Use the `checkly.config.ts/js` file to define your Playwright Check Suite. Each Playwright Check Suite can be connected to references in your `playwright.config.ts/js` file. | ||
|
||
<Info> | ||
During the Beta, **a Playwright Check Suite can run up to 20 minutes**. This limit can be adjusted based on feedback. |
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.
During the Beta, **a Playwright Check Suite can run up to 20 minutes**. This limit can be adjusted based on feedback. | |
**A Playwright Check Suite can run up to 15 minutes**. Please contact us if you're interested in longer runs. |
## Playwright references | ||
|
||
The Checkly infrastructure will run and deploy all your existing Playwright tests (similar to what `npx playwright test` runs) as monitors if you don't reference Playwright projects or tags. | ||
Without limiting and selecting specific tests or Playwright projects, the Checkly infrastructure will run and deploy **all your existing Playwright tests** (similar to what `npx playwright test` runs) as monitors. |
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.
💯 for clarity
detect/synthetic-monitoring/playwright-checks/configuration.mdx
Outdated
Show resolved
Hide resolved
detect/synthetic-monitoring/playwright-checks/configuration.mdx
Outdated
Show resolved
Hide resolved
detect/synthetic-monitoring/playwright-checks/configuration.mdx
Outdated
Show resolved
Hide resolved
|
||
And configure different monitoring configuration for each Playwright project in your Playwright Check Suite. | ||
|
||
```typescript checkly.config.ts highlight={8-12, 17-21} |
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.
I love that we can now highlight lines in snippets!
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.
Yep, I'm using the heck out of it. :D
detect/synthetic-monitoring/playwright-checks/test-organization.mdx
Outdated
Show resolved
Hide resolved
detect/synthetic-monitoring/playwright-checks/test-organization.mdx
Outdated
Show resolved
Hide resolved
detect/synthetic-monitoring/playwright-checks/test-organization.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: María de Antón <maria@checklyhq.com>
Co-authored-by: María de Antón <maria@checklyhq.com>
Co-authored-by: María de Antón <maria@checklyhq.com>
Co-authored-by: María de Antón <maria@checklyhq.com>
… pwn-ga-launch-overhaul
…n.mdx Co-authored-by: María de Antón <maria@checklyhq.com>
…n.mdx Co-authored-by: María de Antón <maria@checklyhq.com>
…n.mdx Co-authored-by: María de Antón <maria@checklyhq.com>
…n.mdx Co-authored-by: María de Antón <maria@checklyhq.com>
…n.mdx Co-authored-by: María de Antón <maria@checklyhq.com>
…n.mdx Co-authored-by: María de Antón <maria@checklyhq.com>
…n.mdx Co-authored-by: María de Antón <maria@checklyhq.com>
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.
🚀 ship it!
// Mute the check so that it doesn't send alerts | ||
muted: false, | ||
// Add a check to a group | ||
groupName: 'production-group', |
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.
No docs PR in the making yet, just CLI WIP
The main piece of the PR is a new document to highlight and explain test organization.
But I looked at the existing docs, too and cleaned up some additional things. :)