test(animated-cursor): add mock integration coverage for async services and event lifecycle - #7351
Conversation
📦 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.
Hey! Thanks for writing mock integration tests for the animated cursor. Ensuring async lifecycle transitions and matchMedia listeners don't leak handlers is crucial for client performance.
I'm assigning level:intermediate, type:testing, and quality:clean.
Solid work, approving!
|
🎉 Congratulations @imuniqueshiv! 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
Fixes #7006
Pillar
Visual Preview
N/A (Test-only change)
Changes Made
components/AnimatedCursor.mock-integrations.test.tsx.AnimatedCursor.requestAnimationFrame.Test Results
npm run test -- AnimatedCursor.mock-integrations.test.tsxResult
npm run lintcompleted successfully (repository-wide warnings only, no errors)npm run buildcompleted successfullyChecklist 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.