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

timeouts with upgrade to >=13.6.3 #29126

Closed
jenkma02 opened this issue Mar 12, 2024 · 11 comments · Fixed by #29349
Closed

timeouts with upgrade to >=13.6.3 #29126

jenkma02 opened this issue Mar 12, 2024 · 11 comments · Fixed by #29349
Assignees

Comments

@jenkma02
Copy link

jenkma02 commented Mar 12, 2024

Current behavior

I uncovered this issue when I was attempting to upgrade from 13.1.0 to the latest 13.6.6, but i narrowed it down to something introduced in 13.6.3 - that's the version that my tests stopped working in. the issue i'm running into is that routes that im intercepting and then waiting on are taking much longer to load than previous versions and are causing timeouts. we have a really big project with intercepts/aliases everywhere - i know that i could add timeouts to all of them but it's not really feasible for me to do that, especially when it has been working as expected until this upgrade.

Desired behavior

No response

Test code to reproduce

cy.intercept('POST', '**/jobs/search').as('postJobsSearch')
cy.wait('@postJobsSearch')

this is very basic - hopefully the debug logs are more helpful

Cypress Version

= 13.6.3

Node version

18.16.0

Operating System

macOS 14.3.1 (23D60)

Debug Logs

here is the log for the test ran in 13.6.3, when it started failing. it also fails in higher versions: https://gist.github.com/jenkma02/55a574d39d8e273003fa404514889cb4

logs for the same test, but run in 13.6.2, passing https://gist.github.com/jenkma02/6391ee92d554c31f1b69e665bd6b62ac.js

Other

Screenshot 2024-03-12 at 3 16 14 PM

@jennifer-shehane
Copy link
Member

You’re likely running into this issue. We have a fix for this that will be in our next release, hopefully in the next few days. #28868

@jennifer-shehane jennifer-shehane added the stage: awaiting response Potential fix was proposed; awaiting response label Mar 12, 2024
@jenkma02
Copy link
Author

thanks for looking at this @jennifer-shehane! I will be on the lookout for that new version and will try it again then.

@jennifer-shehane
Copy link
Member

@jenkma02 The 13.7.0 version is out, can you try and verify if this is fixed?

@jenkma02
Copy link
Author

I have been testing the new version out. It is running a lot better than the previous one, but it is still hanging up in some tests waiting on intercepted routes.

@smartdrop85
Copy link

hello. I am experiencing exactly the same issue. in v13.7.0 it's much better with local runs, but in Jenkins+cypress dashboard 30s timeout for requests is not enough to wait for the call to happen any more -> while in v.13.6.2 it works very well with 15s timeout.

@jenkma02
Copy link
Author

@jennifer-shehane i'm having to update pretty much all of my tests for this update after all. do you guys have any plans to make this work the same way it used to in earlier versions?

@jenkma02
Copy link
Author

jenkma02 commented Mar 20, 2024

something else i've noticed occasionally is that the css doesn't load properly on cypress cloud runs either. this doesn't happen in every test, but i run into it at least once during each run (~950 tests)

here is an example of that - https://d.pr/v/7Yu3NO
same test run locally - https://d.pr/v/KJ5gWu

@sajjad541
Copy link

sajjad541 commented Mar 21, 2024

Hi,
After updating to 13.7.0 version all of my existing tests with intercepts, alias are failing. Route matcher is completely failing for all the tests. Any idea when will this issue be fixed?

image

Thanks

@mschile
Copy link
Contributor

mschile commented Mar 25, 2024

@sajjad541, what version were you upgrading from? Is 13.7.0 the first version that is started failing? Do you have a project I could look at or are you be able to recreate the issue in a reproduction repository by forking cypress-test-tiny or by other means?

@sajjad541
Copy link

@mschile , I've upgraded cypress version from 13.0.0 to 13.7.0 with node version 16.15.1. Unfortunately I can't share the code due to security perspective but I will try to reproduce it in cypress-test-tiny repo. For me dynamic intercept calls were working fine earlier but later with version update route matching is completely failing for all existing case.

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Apr 23, 2024

Released in 13.8.1.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v13.8.1, please open a new issue.

@cypress-bot cypress-bot bot removed the stage: awaiting response Potential fix was proposed; awaiting response label Apr 23, 2024
@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Apr 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants