fix: ensure we have marked things as stable prior to after/afterEach …

…hooks running (#30536)

* fix: ensure we have marked things as stable prior to failing tests -- run ci

* fix: ensure we have marked things as stable prior to failing tests -- run ci

* add a test -- run ci

* clean up

* add changelog

* Update CHANGELOG.md

* Apply suggestions from code review

* Update cli/CHANGELOG.md

Co-authored-by: Matt Schile <mschile@cypress.io>

* allow failure

---------

Co-authored-by: Jennifer Shehane <jennifer@cypress.io>
Co-authored-by: Matt Schile <mschile@cypress.io>

cli/CHANGELOG.md

@@ -3,6 +3,10 @@
 
 _Released 11/5/2024 (PENDING)_
 
+**Bugfixes:**
+
+- Fixed an issue where the Cypress runner could hang in `after` or `afterEach` hooks that run Cypress commands after a page load timeout error occurs. Addresses [#30238](https://github.com/cypress-io/cypress/issues/30238).
+
 **Misc:**
 
 - Fixed a typo in CLI `global` option help text. Addresses [#30531](https://github.com/cypress-io/cypress/issues/30531).

packages/driver/cypress/e2e/issues/30238.cy.js

@@ -0,0 +1,34 @@
+after(() => {
+  // ensure that we're stable in the after hooks
+  expect(cy.state('isStable')).to.be.true
+
+  // ensure we can enqueue a command without timing out
+  cy.then(() => {
+    expect(true).to.be.true
+  })
+})
+
+afterEach(() => {
+  // ensure that we're stable in the after hooks
+  expect(cy.state('isStable')).to.be.true
+
+  // ensure that we can enqueue a command without timing out
+  cy.then(() => {
+    expect(true).to.be.true
+  })
+})
+
+it('runs an after block without timing out when the page load times out', { pageLoadTimeout: 500 }, () => {
+  cy.on('fail', (error) => {
+    expect(error.message).to.include('Timed out after')
+
+    return false
+  })
+
+  cy.on('window:before:load', (win) => {
+    // Stop the page from loading so that the page load times out
+    win.stop()
+  })
+
+  cy.visit('/fixtures/generic.html')
+})

packages/driver/src/cypress/command_queue.ts

@@ -223,6 +223,9 @@ export class CommandQueue extends Queue<$Command> {
     // end in case we have after / afterEach hooks
     // which need to run
     this.index = this.length
+
+    // Mark the state as stable, so that any cypress commands can be re-queued during the after / afterEach hooks
+    this.state('isStable', true)
   }
 
   private runCommand (command: $Command) {