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

Cypress/update to cypress10 #267

Merged
merged 26 commits into from
Nov 7, 2022
Merged

Cypress/update to cypress10 #267

merged 26 commits into from
Nov 7, 2022

Conversation

mmartin24
Copy link
Contributor

@mmartin24 mmartin24 commented Nov 7, 2022

Bumping Cypress 10.

This bump has small differences between Chrome and Firefox due to problems found in actionable elements like dropdowns.
It seems FF versions after 102 not always work well in headless mode. For this reason tests running on FF have a different version compared to those running in Chrome. Also, I noticed differences when versions of Cypress are too far from the ones pre-selected for browsers so the following combinations have been observed to the most stable ones:

So this is the tested stable configuration:

STD-UI-Latest-Chrome: Cypress 10.11.0 | Chrome Chrome 105
STD-UI-Latest-Firefox: Cypress 10.9.0 | Firefox 99
Rancher-UI-1-Chrome: Cypress 10.11.0 | Chrome 100
Rancher-UI-1-Firefox: Cypress 10.11.0 | Firefox 99

Results CI (all sucessful):

Reducing numTestKeptinMemory for better local performance in UI
creating new branch with cypress 10 included
Adding visibility assertion on source type before clicking
Adding debug pause container
adding force click on dropdown
@mmartin24 mmartin24 marked this pull request as ready for review November 7, 2022 09:02
@mmartin24 mmartin24 linked an issue Nov 7, 2022 that may be closed by this pull request
Copy link
Contributor

@thehejik thehejik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mmartin24 mmartin24 merged commit 25cde35 into main Nov 7, 2022
@mmartin24 mmartin24 deleted the cypress/update-to-cypress10 branch November 7, 2022 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Cypress] Bump Cypress to v10
2 participants