Skip to content

test(themes): add vitest accessibility coverage for #4641#4864

Open
kumar-ayan wants to merge 1 commit into
JhaSourav07:mainfrom
kumar-ayan:themes-accessibility
Open

test(themes): add vitest accessibility coverage for #4641#4864
kumar-ayan wants to merge 1 commit into
JhaSourav07:mainfrom
kumar-ayan:themes-accessibility

Conversation

@kumar-ayan
Copy link
Copy Markdown
Contributor

Description

Fixes #4641

This PR adds isolated unit and integration tests specifically targeting Accessibility Standards & Screen Reader Aria Compliance for the themes.ts components, achieving complete coverage on the specified test conditions.

Pillar

  • 🎨 Pillar 1 — New Theme Design
  • 📐 Pillar 2 — Geometric SVG Improvement
  • 🕐 Pillar 3 — Timezone Logic Optimization
  • 🛠️ Other (Bug fix, refactoring, docs)

Visual Preview

N/A — Test coverage addition only. No visual changes.

Checklist before requesting a review:

  • I have read the CONTRIBUTING.md file.
  • I have tested these changes locally (localhost:3000/api/streak?user=YOUR_USERNAME).
  • I have run npm run format and npm run lint locally and resolved all errors (CI will fail otherwise).
  • I have run npm run test and all tests pass locally.
  • I have run npm run test:coverage and branch coverage is at or above 70%.
  • My commits follow the Conventional Commits format (e.g., feat(themes): ..., fix(calculate): ...).
  • I have updated README.md if I added a new theme or URL parameter.
  • I have started the repo.
  • I have made sure that i have only one commit to merge in this PR.
  • The SVG output matches the CommitPulse "premium quality" aesthetic standard (no raw elements, smooth animations, correct fonts).
  • (Recommended) I joined the CommitPulse Discord server for faster collaboration, mentorship, and PR support.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Jun 7, 2026

@kumar-ayan is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test(themes-accessibility): verify Accessibility Standards & Screen Reader Aria Compliance (Variation 4)

1 participant