Skip to content

test(customize): add ControlsPanel component tests#1924

Merged
JhaSourav07 merged 5 commits into
JhaSourav07:mainfrom
Shikhartiwari967:test-controls-panel
May 31, 2026
Merged

test(customize): add ControlsPanel component tests#1924
JhaSourav07 merged 5 commits into
JhaSourav07:mainfrom
Shikhartiwari967:test-controls-panel

Conversation

@Shikhartiwari967
Copy link
Copy Markdown
Contributor

Description

Fixes #676

Pillar

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

Changes Made

  • Added ControlsPanel.test.tsx
  • Added test for GitHub Username label rendering
  • Added test for username input id
  • Added test for username change callback
  • Added test for scale button rendering
  • Added test for scale change callback
  • Added test for Clear overrides visibility

Visual Preview

N/A (test-only changes)

Checklist before requesting a review

  • I have read the CONTRIBUTING.md file.
  • I have tested these changes locally.
  • I have run npm run test successfully.
  • My commits follow the Conventional Commits format.
  • I have only one commit in this PR.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 31, 2026

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

A member of the Team first needs to authorize it.

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

🚨 Hey @Shikhartiwari967, 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. 💪

@Shikhartiwari967
Copy link
Copy Markdown
Contributor Author

Hi @JhaSourav07 , all build/test checks are passing. The remaining failed checks seem to be caused by GitHub API rate limiting (403 API rate limit exceeded). Could you please rerun the workflows when possible?

@JhaSourav07 JhaSourav07 added GSSoC 2026 level:beginner Small changes Usually isolated fixes or simple UI/text updates. quality:exceptional Outstanding contribution with exceptional implementation quality, testing. gssoc:approved PR has been reviewed and accepted for valid contribution points mentor:Aamod007 labels May 31, 2026
@JhaSourav07 JhaSourav07 merged commit 0e9b7a6 into JhaSourav07:main May 31, 2026
6 of 9 checks passed
@github-actions github-actions Bot added this to the GSSoC 2026 milestone May 31, 2026
@github-actions
Copy link
Copy Markdown

🎉 Congratulations @Shikhartiwari967! 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! 💻✨

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

Labels

gssoc:approved PR has been reviewed and accepted for valid contribution points GSSoC 2026 level:beginner Small changes Usually isolated fixes or simple UI/text updates. mentor:Aamod007 quality:exceptional Outstanding contribution with exceptional implementation quality, testing. 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(ControlsPanel): create app/customize/components/ControlsPanel.test.tsx

2 participants