numTestsKeptInMemory wrongly blamed for crash with `cypress run` in CI

cypress-io / cypress

Fast, easy and reliable testing for anything that runs in a browser.

https://cypress.io

MIT License

47.04k stars 3.19k forks source link

numTestsKeptInMemory wrongly blamed for crash with `cypress run` in CI #28578

Open nealeu opened 10 months ago

nealeu commented 10 months ago

Current behavior

The output generated on a crash as in this image is misleading users. The line referencing numTestsKeptInMemory should not appear when it is ignored for cypress run, and could also be intelligent about whether that value is actually greater than zero.

Desired behavior

No response

Test code to reproduce

This does not need reproducing.

Cypress Version

13.6.1

Node version

v18.14.2

Operating System

Ubuntu 22.04

Debug Logs

No response

Other

No response

jennifer-shehane commented 10 months ago

@nealeu You can open a pull request to update it. It was a lot simpler at the time to have the text alone to explain the conditions it works than to connect all the logic to know which mode it’s running under.