Skip to content

UILD-841: Add jest-axe for base accessibility tests - #432

Merged
zepheiryan merged 6 commits into
masterfrom
UILD-841
Jul 30, 2026
Merged

UILD-841: Add jest-axe for base accessibility tests#432
zepheiryan merged 6 commits into
masterfrom
UILD-841

Conversation

@zepheiryan

@zepheiryan zepheiryan commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Purpose

Add automated checks for basic accessibility issues.

Approach

Wherever a render is used in a test suite, generate a new subset of tests to check the output for accessibility violations, building off variations in rendering found throughout the test suite. Review and adjust tests for external factors, particularly those that take aria-labelledby values as arguments, which were not previously included in the test suite.

Also add tests for where clicking will alter the document (like context menus).

Refs

https://folio-org.atlassian.net/browse/UILD-841

@zepheiryan zepheiryan added the ai label Jul 23, 2026
@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown

Jest Unit Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 7309ca8. ± Comparison against base commit fba9ff6.

♻️ This comment has been updated with latest results.

…ecks. Remove tests on wrappers. Remove unnecessary format / input mocks interfering with tests. Add aria-labelledby to Dropzone input. Comment out tests that need fixes and note which story should fix them.
@sonarqubecloud

Copy link
Copy Markdown

@zepheiryan
zepheiryan marked this pull request as ready for review July 27, 2026 20:26
@zepheiryan
zepheiryan requested a review from a team as a code owner July 27, 2026 20:26
@zepheiryan
zepheiryan merged commit bb2423e into master Jul 30, 2026
16 checks passed
@zepheiryan
zepheiryan deleted the UILD-841 branch July 30, 2026 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants