Skip to content

test: add RefreshButton accessibility test suite#3581

Open
star07770 wants to merge 2 commits into
JhaSourav07:mainfrom
star07770:refreshbutton-a11y-clean
Open

test: add RefreshButton accessibility test suite#3581
star07770 wants to merge 2 commits into
JhaSourav07:mainfrom
star07770:refreshbutton-a11y-clean

Conversation

@star07770
Copy link
Copy Markdown

Description

Fixes #2596

Adds accessibility test coverage for the RefreshButton component.

Pillar

  • 🛠️ Other (Bug fix, refactoring, docs)

What changed

  • Added aria-label test for screen readers
  • Added title attribute test for tooltip accessibility
  • Added keyboard focus test
  • Added disabled state test when pending

Visual Preview

N/A

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)
  • My commits follow the Conventional Commits format (e.g., feat(themes): ..., fix(calculate): ...)
  • I have made sure that I have only one commit to merge in this PR

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2026

👋 Hey @star07770, welcome to CommitPulse! 🎉

Thanks for opening your first pull request — this is a big deal and we appreciate the effort!

While you wait for a review, please double-check:

  • ✅ You've read the CONTRIBUTING.md checklist
  • npm run lint, npm run format, and npm run test all pass locally
  • ✅ Your PR has a visual preview if it touches any SVG output
  • 💬 You've joined our Discord for faster PR feedback

A maintainer will review your PR shortly. Hang tight! 🚀

@github-actions github-actions Bot added the status:blocked This PR is blocked due to a failing CI check. label Jun 4, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2026

🚨 Hey @star07770, the CI Pipeline is failing on this PR and it has been marked as status:blocked.

Please fix the issues before this can be reviewed. Here's how:

1. Run checks locally before pushing:

npm run format:check   # Check Prettier formatting
npm run lint           # Run ESLint
npm run typecheck      # TypeScript type check
npm run test           # Run unit tests (Vitest)
npm run build          # Verify production build passes

2. Auto-fix common issues:

npm run format         # Auto-fix formatting with Prettier
npm run lint -- --fix  # Auto-fix lint errors where possible

3. Check the full failure log here:
👉 View CI Run

Once you push a fix and the CI passes, the status:blocked label will be removed automatically. 💪

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

Labels

status:blocked This PR is blocked due to a failing CI check.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

1 participant