test: add empty fallback coverage for CustomizeCTA#5737
Conversation
|
@deepsikha-dash is attempting to deploy a commit to the jhasourav07's projects Team on Vercel. A member of the Team first needs to authorize it. |
📦 Next.js Bundle Size Report (Gzipped Sizes)✨ No significant bundle size changes detected. 📊 Summary of Totals
|
Aamod007
left a comment
There was a problem hiding this comment.
This is blocked by the Vercel deployment authorization check rather than a code problem. Please authorize the linked deploy, rerun checks, and then we can take another look at the empty-fallback coverage in app/components/CustomizeCTA.empty-fallback.test.tsx.
Aamod007
left a comment
There was a problem hiding this comment.
Good fallback coverage for the CTA. app/components/CustomizeCTA.empty-fallback.test.tsx checks empty, undefined, and null-like translations while keeping the customize link and structure intact, which is exactly the kind of regression coverage we want.
|
🎉 Congratulations @deepsikha-dash! Your PR has been successfully merged. 🚀 Thank you for contributing to CommitPulse. Your work helps us build a better tool for the community.
Keep building! 💻✨ |
Description
Added empty fallback test coverage for
CustomizeCTAto verify component stability when translation values are empty, undefined, or missing. The new tests ensure the CTA structure and navigation link remain available without runtime failures.Fixes #4211
Pillar
Visual Preview
N/A
Checklist before requesting a review:
CONTRIBUTING.mdfile.localhost:3000/api/streak?user=YOUR_USERNAME).npm run formatandnpm run lintlocally and resolved all errors (CI will fail otherwise).feat(themes): ...,fix(calculate): ...).README.mdif I added a new theme or URL parameter.