test(ThemeSwitch-mouse-interactivity): verify Interactive Tooltips, Cursor Hovers & Touch Event Propagation (Variation 5) #2837#3636
Conversation
|
@alishaalmeida10 is attempting to deploy a commit to the jhasourav07's projects Team on Vercel. A member of the Team first needs to authorize it. |
Aamod007
left a comment
There was a problem hiding this comment.
Hi there! 👋 Thanks for your PR. Similar to another recent test PR, I noticed that heme-switch.mouse-interactivity.test.tsx creates and tests a mock component (MockThemeSwitch) directly inside the test file instead of importing the actual ThemeSwitch component from the application.
To ensure we're properly verifying the application's actual behavior, could you update the test to import and test the real component instead of the mock? Let me know if you need any pointers on how to do this!
|
Hi @Aamod007, thanks for the feedback.. That makes total sense, of testing actual components than using mocks. I'm swapping them out right now for both files and I'll update the PRs as soon as they're ready. Will let you know if I get stuck anywhere. Thanks |
Aamod007
left a comment
There was a problem hiding this comment.
LGTM on the code side! However, the CI checks are failing. Please fix the CI issues so this can be merged.
Description
Fixes #2837
This PR introduces a robust suite of 5 target unit tests for the
ThemeSwitchinteraction layout to ensure precise mouse tracking, pointer state validation, and custom touch event propagation.What was done:
mouseenterandmousemoveactions to validate real-time coordinate tooltip rendering.cursor-pointer) trigger on active hover fields.touchStartandtouchEndevents.mouseleaveresets.Pillar
Checklist
Visual Preview
All 5 required interaction states, computed coordinate calculations, and element touch sweeps compile cleanly and pass local environments seamlessly.