fix(driver): cy.pause() should not be ignored with cypress run --headed --no-exit

[davidmunechika]

• Closes pause() should not be ignored with cypress run --headed --no-exit #4044

User facing changelog

Users will be able to use cy.pause() and observe that a test paused when using cypress run --headed --no-exit.

Additional details

Previous behavior: cy.pause() was ignored when using cypress run even if --headed and --no-exit flags were passed.

New behavior: cy.pause() works as expected (i.e. the same as with cypress open) when --headed and --no-exit flags are passed to cypress run.

PR Tasks

• Have tests been added/updated?
• Has the original issue or this PR been tagged with a release in ZenHub?
• [n/a] Has a PR for user-facing changes been opened in cypress-documentation?
• Have API changes been updated in the type definitions?
• Have new configuration options been added to the cypress.schema.json? --> N/A

[cypress-bot]

Thanks for taking the time to open a PR!

• Create a Draft Pull Request if your PR is not ready for review. Mark the PR as Ready for Review when you're ready for a Cypress team member to review the PR.
• Become familiar with the Code Review Checklist for guidelines on coding standards and what needs to be done before a PR can be merged.

[cypress]

---

Test summary

4205 • 0 • 50 • 2 • 0

---

Run details

Project	cypress
Status	Passed
Commit	ccc845d
Started	Oct 8, 2021 8:23 PM
Ended	Oct 8, 2021 8:33 PM
Duration	09:36 💡
OS	Linux Debian - 10.9
Browser	Chrome 94

View run in Cypress Dashboard ➡️

---

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

[flotwig]

Yeowch, this is a hard PR to test all the way...

Since you are not creating a full e2e test for this fix, only an integration test that uses mocks, i think it would probably be prudent to update a unit/integration test somewhere that also checks that exit is being passed to the runner controller, along with isHeadless, etc. Surely such a test exists already and can be updated to add exit.

Past that small testing concern, this looks great!

[totomaze]

Is this PR is 9.5.2 ?
I can't get it to pause when doing a cypress run --headed --no-exit

I downgraded to 8.6.0 and it worked but with the 9.5.2 it is not. Is it a regression ?

[BlueWinds]

@totomaze - This was released in 9.6.0, and is not in 9.5.2. #4044 (comment)

[totomaze]

The last version where it s working is 9.1.1, after this version using --headed --no-exit no longer pause during a test run.

I believe it s caused by this update in 9.2.0:
Cypress will throw an error when a user attempts to update a configuration
value at run-time using either the Test Configuration or using
Cypress.config() that is a
readonly option.
Addresses #6407 and
#19001.

Is there any workaround to get the test to pause when running it by cypress run ?