fix(tests): complete GeneratorState mock in EditorPanel mouse interactivity test#6141
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
|
Aamod-Dev
left a comment
There was a problem hiding this comment.
Thank you for completing the GeneratorState mock in app/generator/components/EditorPanel.mouse-interactivity.test.tsx. Removing the 'as unknown as' cast and fully typing the object with the missing properties (showSnakeGraph, showPacmanGraph, graphPlacement) is exactly what we need to maintain type safety in tests. Approved!
|
🎉 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
Updated the GeneratorState mock used in
EditorPanel.mouse-interactivity.test.tsxto fully satisfy the currentGeneratorStateinterface.Removed the unsafe
as unknown as GeneratorStatecast and added the missing properties:showSnakeGraphshowPacmanGraphgraphPlacementThis keeps the test mock aligned with the actual type definition and improves type safety.
Fixes #5353
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.