CYPRESS_VERIFY_TIMEOUT ignored when passed via .npmrc

[legap]

Current behavior

When setting CYPRESS_VERIFY_TIMEOUT in .npmrc it is currently ignored and VERIFY_TEST_RUNNER_TIMEOUT_MS falls back to the default value.

Desired behavior

CYPRESS_VERIFY_TIMEOUT can also be set in .npmrc and is picked up in verify.js

Test code to reproduce

In .npmrc use:

CYPRESS_VERIFY_TIMEOUT=100000

Cypress Version

9.2.0

Other

@tbiethman, @emilyrohrbough: Sorry for the string to integer issue and thanks for the quick fix!
I just tested the new version today and realized the problem myself 🤦‍♂️

This issue here would improve the usability and handle this environment variable like I've seen in other parts in the code.

I will open another PR that will only change this line in verify.js:

const VERIFY_TEST_RUNNER_TIMEOUT_MS = parseInt(util.getEnv('CYPRESS_VERIFY_TIMEOUT')) || 30000;

Thanks for your effort and I would be happy if you could review the new pull-request 🙏

[cypress-bot]

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

[cypress-bot]

Released in 9.3.0.

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