-
Notifications
You must be signed in to change notification settings - Fork 266
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
Accessibility tests #12900
base: master
Are you sure you want to change the base?
Accessibility tests #12900
Conversation
@aalves08 Can you take a look - would be good to get this in and then build out the tests |
My only critiques have to do with the formatting of the output. I'm finding it incredibly difficult to associate violations to test suites. For example, it appears that the output is formatted in such a way that information is grouped by:
When the next test suite starts, it bleeds into the rest of the information. Is there a better way that can group the output? |
Summary
Fixes #8986
Occurred changes and/or fixed issues
This PR is a re-work and update to #9457.
Technical notes summary
This PR adds a new e2e test suite
a11y-tests
which run a set of e2e tests with support for checking accessibility using thecypress-axe
plugin.This is done as a separate test suite, so that we get an easy-to-run suite that produces a single audit report, rather than the report being spread across multiple suites.
Checklist