-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
PR tests reporting #5356
base: main
Are you sure you want to change the base?
PR tests reporting #5356
Conversation
🦋 Changeset detectedLatest commit: f51a4ea The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Differences FoundExpandLicense Package MIT playwright-ctrf-json-reporter SummaryExpand
|
What if test if flaky? Now we have info only about test case status in merge reports, before I saw a "flaky" field |
@karola312 I disabled it for now because when we post failed/flaky or all results we have tags in the test name with @ so each time we notify someone for our PRs which isn't a great vibe. We need to change that and then we can enable more options |
Scope of the change
I want to have test reports available in PRs and measure flaky tests
This PR adds a new PW reporter https://github.com/ctrf-io/playwright-ctrf-json-reporter
and publish test summary https://github.com/ctrf-io/github-test-reporter to PRs
Next, we will collect data form PRs and enable option for historical data:
fail-rate-report
flaky-rate-report
previous-results-report
and add reporting for other workflows