Skip to content

test(CustomizeCTA): add responsive breakpoint test coverage#8086

Merged
JhaSourav07 merged 2 commits into
JhaSourav07:mainfrom
Shikhartiwari967:test-customizecta-responsive-breakpoints
Jul 18, 2026
Merged

test(CustomizeCTA): add responsive breakpoint test coverage#8086
JhaSourav07 merged 2 commits into
JhaSourav07:mainfrom
Shikhartiwari967:test-customizecta-responsive-breakpoints

Conversation

@Shikhartiwari967

Copy link
Copy Markdown
Contributor

Description

Fixes #6824

Added a new responsive breakpoint test suite for the CustomizeCTA component.

The test suite verifies:

  • Responsive flex layout classes
  • Responsive heading typography
  • CTA link accessibility
  • Responsive spacing utilities
  • Successful rendering

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-only change)

Checklist before requesting a review:

  • I have read the CONTRIBUTING.md file.
  • I have tested these changes locally.
  • I have run npm run lint locally.
  • My commits follow the Conventional Commits format.
  • I have updated README.md if required.
  • I have starred the repository.
  • I have made sure I have only one commit in this PR.
  • The change only adds test coverage.
  • I joined the CommitPulse Discord community.

@vercel

vercel Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

@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 Jul 15, 2026
@Shikhartiwari967

Shikhartiwari967 commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

Hi! @Aamod007 Sir, I verified the CI failure locally by running npm run typecheck on the latest main branch.

The reported TypeScript errors are in:

app/generator/components/EditorPanel.type-compiler.test.tsx
app/generator/components/PreviewPanel.type-compiler.test.tsx
lib/rss.ts

This PR only adds app/components/CustomizeCTA.responsive-breakpoints.test.tsx, and the new test passes locally. Please let me know if you'd like me to make any changes to this PR.

@Shikhartiwari967

Copy link
Copy Markdown
Contributor Author

Hi @Aamod007 sir , please review it.

@Aamod007 Aamod007 added level:intermediate Moderate complexity tasks quality:clean PR follows clean coding practices, proper formatting, documentation, and maintainability standards. type:testing Adding, updating, or fixing tests mentor:Aamod007 labels Jul 17, 2026

@Aamod007 Aamod007 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job adding the responsive breakpoint test coverage for the CustomizeCTA component. The assertions for flex layout and typography classes are comprehensive. However, this PR is blocked by a failing CI check. Please resolve the issues and rerun the CI pipeline.

@github-actions github-actions Bot removed the status:blocked This PR is blocked due to a failing CI check. label Jul 17, 2026
@Shikhartiwari967

Copy link
Copy Markdown
Contributor Author

Hi @Aamod007 sir, please review and merge it.

@github-actions

Copy link
Copy Markdown
Contributor

📦 Next.js Bundle Size Report (Gzipped Sizes)

Asset PR Size Base Size Difference Status
static/chunks/1915-[hash].js 15.54 KB - +15.54 KB (++100%) 🆕 New
static/chunks/4200-[hash].js - 10.25 KB -10.25 KB (-100.00%) 🗑️ Deleted
static/chunks/4305-[hash].js - 16.27 KB -16.27 KB (-100.00%) 🗑️ Deleted
static/chunks/5285-[hash].js 10.08 KB - +10.08 KB (++100%) 🆕 New
static/chunks/6297-[hash].js - 52.11 KB -52.11 KB (-100.00%) 🗑️ Deleted
static/chunks/9389-[hash].js 7.01 KB 7.08 KB -0.07 KB (-0.98%) 🟢 Improvement
static/chunks/9425-[hash].js 52.20 KB - +52.20 KB (++100%) 🆕 New
static/chunks/app/api/spotify/route-[hash].js - 0.24 KB -0.24 KB (-100.00%) 🗑️ Deleted
static/chunks/app/burnout-analyzer/page-[hash].js 9.33 KB 9.81 KB -0.47 KB (-4.82%) 🟢 Improvement
static/css/[hash].css 40.63 KB 40.96 KB -0.33 KB (-0.80%) 🟢 Improvement

📊 Summary of Totals

Category PR Size Base Size Difference
Total JS 3958.92 KB 3966.23 KB -7.32 KB (-0.18%)
Total CSS 327.67 KB 330.26 KB -2.59 KB (-0.78%)

@JhaSourav07 JhaSourav07 added the gssoc:approved PR has been reviewed and accepted for valid contribution points label Jul 18, 2026
@JhaSourav07
JhaSourav07 merged commit 3c9bdd6 into JhaSourav07:main Jul 18, 2026
5 of 6 checks passed
@github-actions github-actions Bot added this to the GSSoC 2026 milestone Jul 18, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🎉 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 level:intermediate Moderate complexity tasks mentor:Aamod007 quality:clean PR follows clean coding practices, proper formatting, documentation, and maintainability standards. type:testing Adding, updating, or fixing tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test(CustomizeCTA-responsive-breakpoints): verify Responsive Multi-device Columns & Mobile Viewport Layouts (Variation 7)

3 participants