Skip to content

Enable accessibilityAudit on `client/web/src/integration/extension-registry.test.ts`

Administrator requested to merge contractors/SG-33093.4 into main

Created by: gitstart-sourcegraph

Description

Enable accessibility audit on client/web/src/integration/extension-registry.test.ts

Implementation details

Here's how we should implement:

  • Add accessibilityAudit in the same place where we call percySnapshot in the integration tests.
  • Run the tests locally and observe any failures. See the above PR for example output.

To fix the failures, we should:

  1. If it's simple or quick to fix, then make the changes on the branch. There should be relevant documentation attached to the test error that will help you work out how to fix this.
  2. If it's complex, or will take a long time to fix. Add the a11y-ignore class to exclude the element from the audit, raise an issue specifically to solve this problem, leave a comment next to the class with a link to the issue.

Refs

SG Issue Gitstart Ticket

Test plan

Run the integration test ENTERPRISE=1 HEADLESS=true yarn test-integration:base client/web/src/integration/extension-registry.test.ts

Success criteria

  • All integration tests with percy snapshots have accessibilityAudit enabled

App preview:

Check out the client app preview documentation to learn more.

Merge request reports