test: add theme contrast coverage for RefreshButton component

[kanishka-2007-tech]

Description

Adds comprehensive theme contrast tests for the RefreshButton component to ensure visual cohesion and accessibility across both light and dark modes.

Changes Made

• Added new test file:

  • components/dashboard/RefreshButton.theme-contrast.test.tsx

• Added coverage for:

  • Dark mode border contrast styling
  • Dark mode text/background contrast styling
  • Light mode contrast styling
  • Theme-specific hover states
  • Disabled-state accessibility styling

Fixes #2595

Pillar

• 🎨 Pillar 1 — New Theme Design
• 📐 Pillar 2 — Geometric SVG Improvement

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 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 community for contributor discussions, mentorship, and faster PR support.

[vercel]

@kanishka-2007-tech is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

[Aamod-Dev]

Great coverage! Ensuring the RefreshButton correctly respects light and dark mode variables means it will always look integrated into the chosen theme. Approving!

[Aamod-Dev]

Thanks for the contribution. I went through the changes and the overall approach looks good. The theme contrast test cases ensure our UI remains cohesive in both light and dark modes. Looks good to merge.

[Aamod-Dev]

Thanks for the PR. I've reviewed the changes and they look good to me.

I'm happy to approve this. Great job!

[Aamod-Dev]

Thanks for the contribution. I went through the changes and the overall approach looks good, but there are a few issues that should be addressed before this can be merged. Most of the concerns are related to correctness and maintainability.

• There are merge conflicts with the base branch. Please resolve them to ensure existing functionality isn't broken.

Once these issues are addressed, I'll be happy to take another look. Thanks again for the contribution.

[kanishka-2007-tech]

@Aamod007 sir i am working on it .

thankyou

[github-actions]

🚨 Hey @kanishka-2007-tech, 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. 💪

[Aamod-Dev]

Thanks for the contribution. I went through the changes and the overall approach looks good. The theme contrast test cases are written cleanly and cover the component's styling well. However, I noticed that the CI pipeline is currently failing for this PR.

• The \Format · Lint · Typecheck · Test\ CI job has failed. Please run
  pm run format,
  pm run lint, and
  pm run test\ locally to identify and resolve the issue.

Once these issues are addressed and the CI is green, I'll be happy to take another look and get this merged. Thanks again for the contribution.

[Aamod-Dev]

Thanks for the contribution. The overall approach looks good, but the CI pipeline is currently failing. Please run the checks (like npm run format, npm run lint, and npm run test) locally to identify and fix the issues. Let me know once it's green!

[kanishka-2007-tech]

@Aamod007 @JhaSourav07 app/api/student/resume/confirm/route.theme-contrast.test.ts is an unecessary file as it is a theme contrast test but it is testing api logic can you please remove it or resolve the error as it is causing ci pipeline failure

[vercel]

Deployment failed with the following error:

The provided GitHub repository does not contain the requested branch or commit reference. Please ensure the repository is not empty.

[Aamod-Dev]

Thanks for fixing the CI pipeline! Approving.

[github-actions]

🎉 Congratulations @kanishka-2007-tech! Your PR has been successfully merged. 🚀

Thank you for contributing to CommitPulse. Your work helps us build a better tool for the community.

⚠️ Important for GSSoC Contributors:
You are strictly advised to join our Discord Server as it is mandatory for all GSSoC participants. All important announcements, point claims, and community discussions happen there.

Keep building! 💻✨