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

Configurable or increased VERIFY_TEST_RUNNER_TIMEOUT_MS #18290

Closed
TimothyGillespie opened this issue Sep 29, 2021 · 4 comments · Fixed by #19282
Closed

Configurable or increased VERIFY_TEST_RUNNER_TIMEOUT_MS #18290

TimothyGillespie opened this issue Sep 29, 2021 · 4 comments · Fixed by #19282

Comments

@TimothyGillespie
Copy link

What would you like?

The VERIFY_TEST_RUNNER_TIMEOUT_MS constant is increased to, for example, 100000, or the option smokeTestTimeout can be used via a static file (i.e. cypress.json). Being able to run it in via command line parameter would work also but is a less convenient solution for internal distribution.

Why is this needed?

The static timeout causes issues at my workplace and seems to be a shared issue where ever slow systems are verifying the cypress or certain background software is slowing down the execution.

The currently worked out fix involves editing the source code in node_modules which is not very neat nor does it seems reliable for automated solutions.

see fix suggestions here: https://swingli.com/how-to-fix-cypress-verification-timed-out-exception/
and here: https://stackoverflow.com/questions/63667880/cypress-verification-timed-out-after-30000-milliseconds

Other

The constant VERIFY_TEST_RUNNER_TIMEOUT_MS in node_modules\cypress\lib\tasks\verify.js will be used in the options as smokeTestTimeout. But only the dev option seems to be usuable.

@Narretz
Copy link
Contributor

Narretz commented Sep 29, 2021

There's a PR for that #18243

You can voice your support there

@legap
Copy link
Contributor

legap commented Dec 6, 2021

Because PR #18243 seems to be stale I opened a new one including a test. Hope this one will be available in the next version.

@johndpotts
Copy link

great to see there's a PR open for this! Getting cypress to work at my workplace involves manually updating this value in node_modules right now.

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Dec 21, 2021

Released in 9.2.0.

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

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Dec 21, 2021
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.

4 participants