-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
feat: toggle avt test #14449
feat: toggle avt test #14449
Conversation
✅ Deploy Preview for carbon-components-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
abelledby
…rdona/carbon into 14331-toggle-avt-tests
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.
Should we include the avt test for the skeleton variant also? I see that a few files have on it
Co-authored-by: Guilherme Datilio Ribeiro <guilhermedatilio@gmail.com>
abelledby
Co-authored-by: Guilherme Datilio Ribeiro <guilhermedatilio@gmail.com>
8960d95
to
6e43ffa
Compare
…rdona/carbon into 14331-toggle-avt-tests
Part of #13747
Closes #14331
Changelog
New
Broke https://github.com/AVT tests out into their own Tabs-test.avt.e2e.js file
@vrt tests remain in the Tabs-test.e2e.js file
Add keyboard navigation tests covering default Tabs state
Update playwright config glob to match and include the new test file naming structure (avt.e2e.js)
Testing / Reviewing
CI should pass
Ensure the tests pass a sniff test and don't look problematic
yarn playwright test components/Toggle/Toggle-test.avt.e2e.js