[test][checkbox] Add axe tests and WCAG conformance report - #48918
[test][checkbox] Add axe tests and WCAG conformance report#48918michelengelen wants to merge 2 commits into
Conversation
Deploy previewBundle size
Check out the code infra dashboard for more information about this PR. |
ab010c4 to
c3fc8c7
Compare
cab0815 to
8ba1e37
Compare
cb67f12 to
2ee61e1
Compare
PR reviewClean PR. The Tests (1)1. 🟡 Test title claims more than it assertsLocation: it('reflects the checked state and notifies on change', async () => {The test asserts only the checked flip after Failure scenario: an Fix: assert a spy's call count, or rename the test to "reflects the checked state". VerdictApprove after nits - solid coverage; one test name to align with its assertions. 🤖 Review generated with Claude Code |
2ee61e1 to
d8df9d5
Compare
Applies mui#48916 (comment): the shared reports table starts empty, and each component PR appends its own row. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013HgD6GEW3UxNRvC2abbTy9
Adds axe coverage and the WCAG 2.2 Level A/AA conformance report for
Checkbox.Known gaps: focus-indicator contrast and custom icons below 3:1 (1.4.11),
disableRippleremoving the focus indicator (2.4.7), andindeterminatesettingaria-checked="mixed"on a native checkbox whosecheckedisfalse(4.1.2) — the one genuine defect in the series.Report:
Checkbox/accessibility.mdImportant
How to review this PR
Layer 5 of 14 in a stacked series (#48915 → #48926). Its branch is built on
a11y/stack/avatar(#48917). GitHub cannot chain PR bases across a fork, so every PR in the series targetsmasterinstead.Review only the last commit —
2ee61e1. Everything above it belongs to the layers below, and disappears as those merge.Original work by @mj12albert. Supersedes #48734, rebased onto current master.
Part of the WCAG conformance effort (#14187).