test(sanitizer-accessibility): add tests for Accessibility Standards & Aria Compliance (#4640)#6224
Conversation
…n Reader Aria Compliance
f8b6b6a to
6ea1214
Compare
6ea1214 to
c666864
Compare
📦 Next.js Bundle Size Report (Gzipped Sizes)✨ No significant bundle size changes detected. 📊 Summary of Totals
|
Aamod-Dev
left a comment
There was a problem hiding this comment.
Adding explicit accessibility tests in lib/svg/sanitizer.accessibility.test.ts is a great way to ensure our utilities maintain compliance. Testing fallback colors, animation boundaries, and luminance calculations ensures the visual accessibility of the generated SVGs won't regress. Approved!
|
🎉 Congratulations @diksha78dev! Your PR has been successfully merged. 🚀 Thank you for contributing to CommitPulse. Your work helps us build a better tool for the community.
Keep building! 💻✨ |
Description
Fixes #4640
Added
lib/svg/sanitizer.accessibility.test.tsto satisfy the accessibility & aria compliance testing requirements for thesanitizer.tsutility. Included 5 passing unit tests specifically verifying safe fallback colors, animation speed boundaries, contrast luminance logic, XML script escaping, and external Google Font URL sanitization.Pillar
Visual Preview
N/A (This PR only adds unit tests for a background utility, no visual changes).
Checklist before requesting a review:
CONTRIBUTING.mdfile.localhost:3000/api/streak?user=YOUR_USERNAME).npm run formatandnpm run lintlocally and resolved all errors (CI will fail otherwise).feat(themes): ...,fix(calculate): ...).README.mdif I added a new theme or URL parameter.