RocketChat / RocketChat/EmbeddedChat
Add jest-axe baseline accessibility tests for core components
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 165
- Forks
- 381
- Avg merge
- 1d 2h
- Merged PRs (30d)
- 1
Description
While going through the codebase I noticed packages/react already has Jest and @testing-library/react set up but there are no accessibility checks anywhere. Since EmbeddedChat gets embedded into third-party apps, any a11y issues in core components become the host app's problem too.
jest-axe slots in cleanly on top of the existing setup — no major config changes needed.
What I'm proposing:
Add jest-axe as a dev dependency in packages/react
Extend Jest with toHaveNoViolations
Write baseline tests for ChatInput, MessageList, and Sidebar
Hook into existing CI so it runs on every PR
I already have this working locally. Happy to open a draft PR if this looks good.
Contributor guide
No contributing guide indexed for this repository
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 in packages/react by inspecting the existing Jest and @testing-library/react setup, then locate ChatInput, MessageList, and Sidebar and their current tests. Add baseline jest-axe coverage for those components, extend Jest with toHaveNoViolations, and verify the checks run in the existing CI on every pull request.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, react
- Domain
- accessibility, ci-cd, frontend, testing
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 65/100