Skip to content

test(atomic-a11y): add comprehensive unit test suite for reporter#7140

Draft
y-lakhdar wants to merge 3 commits intofeat/a11y-reporterfrom
feat/a11y-reporter-unit-tests
Draft

test(atomic-a11y): add comprehensive unit test suite for reporter#7140
y-lakhdar wants to merge 3 commits intofeat/a11y-reporterfrom
feat/a11y-reporter-unit-tests

Conversation

@y-lakhdar
Copy link
Copy Markdown
Contributor

@y-lakhdar y-lakhdar commented Feb 17, 2026

Summary

Adds 247 unit tests across 12 test files covering the full atomic-a11y reporter pipeline. Split out from #7123 to keep the reporter source PR reviewable.

How to Run

cd packages/atomic-a11y && pnpm test

Related

Add vitest.config.js and 12 test files covering all testable source modules:
- guards, sorting, shard-resolution, criterion-metadata, axe-rule-mappings
- storybook-extraction, error-parsing, axe-integration, reporter-utils
- summary, report-builder, vitest-a11y-reporter (integration)
- Shared mock factories for test data reuse

247 tests total, all passing.
@alexprudhomme
Copy link
Copy Markdown
Contributor

alexprudhomme commented Apr 8, 2026

Hey @y-lakhdar 👋 ✅ This PR does not need a changeset file — you don't have one currently, so it's perfect. This PR only affects @coveo/atomic-a11y (a private package) and only adds tests. For more info on when changesets are needed, see the Changesets README.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a11y Accessibility issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants