[test] Opt out of screenshot for flaky tests
Open
Nobody has claimed this yet.
scope: code-infra
test
type: enhancement
- Dominant language
- JavaScript
- Stars
- 99.1k
- Forks
- 32.5k
- Avg merge
- 2d 17h
- Merged PRs (30d)
- 106
Description
Suggested in https://github.com/mui/material-ui/pull/39711#issue-1973332553
- 1. Implement the
NoSnapconvention like MUI X https://github.com/mui/mui-x/blob/612fb24d6686565dd1adac613c2d42aadf3047d6/test/regressions/index.js#L19
const blacklist = [
/^docs-(.*)(?<=NoSnap)\.png$/, // Excludes demos that we don't want
// 'docs-system-typography',
];
- 2. Update the
blacklistto use the prefix for demos.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the referenced test/regressions/index.js in MUI X to understand the NoSnap convention and blacklist pattern, then locate the corresponding screenshot regression setup in this repository. Update the blacklist to recognize the NoSnap prefix and verify that marked demos no longer produce screenshots.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- testing-qa
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100