Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No mousemove events before clicks #2521

Closed
NoriSte opened this issue Sep 25, 2018 · 4 comments · Fixed by #3030
Closed

No mousemove events before clicks #2521

NoriSte opened this issue Sep 25, 2018 · 4 comments · Fixed by #3030
Assignees

Comments

@NoriSte
Copy link
Contributor

NoriSte commented Sep 25, 2018

Current behavior:

When I cy.get('...').click(); no mousemove events are triggered, so every mousemove related scripts are broken.

Here an example showing the GitHub's milestone panel broke because of the missing mousemove events. I run the same test with Puppeteer and TestCafè too to understand what is the common behavior in such scenario

and here the log related to the three frameworks

TestCafè

mouseover <summary class="text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label="Set milestone" aria-haspopup="menu" data-hotkey="m">...</summary>
mousemove <summary class="text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label="Set milestone" aria-haspopup="menu" data-hotkey="m">...</summary>
mousemove <summary class="text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label="Set milestone" aria-haspopup="menu" data-hotkey="m">...</summary>
mousedown <summary class="text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label="Set milestone" aria-haspopup="menu" data-hotkey="m">...</summary>
mouseup <summary class="text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label="Set milestone" aria-haspopup="menu" data-hotkey="m">...</summary>
click <summary class="text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label="Set milestone" aria-haspopup="menu" data-hotkey="m">...</summary>

Puppeteer

mouseover <summary class="text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label=​"Set milestone" aria-haspopup=​"menu" data-hotkey=​"m">​…​</summary>​
mousemove <summary class=​"text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label=​"Set milestone" aria-haspopup=​"menu" data-hotkey=​"m">​…​</summary>​
mousedown <summary class=​"text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label=​"Set milestone" aria-haspopup=​"menu" data-hotkey=​"m">​…​</summary>​
mouseup <summary class=​"text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label=​"Set milestone" aria-haspopup=​"menu" data-hotkey=​"m">​…​</summary>​
click <summary class=​"text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label=​"Set milestone" aria-haspopup=​"menu" data-hotkey=​"m">​…​</summary>​

Cypress

mousedown <summary class="text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label="Set milestone" aria-haspopup="menu" data-hotkey="m">...</summary>
mouseup <summary class="text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label="Set milestone" aria-haspopup="menu" data-hotkey="m">...</summary>
click <summary class="text-bold discussion-sidebar-heading discussion-sidebar-toggle" aria-label="Set milestone" aria-haspopup="menu" data-hotkey="m">...</summary>

Desired behavior:

I know that I can simulate the mousemove event and if this is the exactly expected behavior... sorry if I bored you...

Steps to reproduce:

In this repo you can find the most complete version of my tests but I prepared even an easiest one without any useless stuff.

Versions

Cypress 3.1.0 on Mac OSX 10.13.6

@kuceb
Copy link
Contributor

kuceb commented Dec 17, 2018

Hi @NoriSte, we are adding the mouseover, mousemove, pointerdown and pointerup as a part of native events in this issue: #2956
Thanks!

@NoriSte
Copy link
Contributor Author

NoriSte commented Dec 23, 2018

Thank you @bkucera 😊

@cypress-bot cypress-bot bot added stage: needs review The PR code is done & tested, needs review stage: work in progress and removed stage: needs review The PR code is done & tested, needs review labels Aug 8, 2019
@cypress-bot cypress-bot bot added stage: needs review The PR code is done & tested, needs review and removed stage: work in progress labels Sep 6, 2019
@cypress-bot cypress-bot bot added stage: work in progress stage: needs review The PR code is done & tested, needs review and removed stage: needs review The PR code is done & tested, needs review stage: work in progress labels Sep 25, 2019
@cypress-bot cypress-bot bot added stage: needs review The PR code is done & tested, needs review and removed stage: work in progress labels Oct 7, 2019
@cypress-bot cypress-bot bot added stage: pending release and removed stage: needs review The PR code is done & tested, needs review labels Oct 11, 2019
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Oct 11, 2019

The code for this is done in cypress-io/cypress#3030, but has yet to be released.
We'll update this issue and reference the changelog when it's released.

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Oct 23, 2019

Released in 3.5.0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants