`cy.window().its( 'mw' )` is executing the `inspect` method on the subject

[micgro42]

Current behavior

The instruction

cy.window().its( 'mw' ).should( 'have.property', 'propertyToCheckFor' );

is executing the mw.inspect method if it exists.

That is fully unexpected and might lead to undefined behavior, including throwing an Exception and thus failing the test.

Desired behavior

The instruction

cy.window().its( 'mw' ).should( 'have.property', 'propertyToCheckFor' );

is not executing any methods of the window.mw object at all.

Test code to reproduce

https://github.com/micgro42/cypress-MFE-inspect

Cypress Version

13.16.0

Node version

v20.15.0

Operating System

Ubuntu 24.04.1

Debug Logs

GitHub complained that the comment is too long when I tried to add the entire debug output here. Please be more specific with what you need here. Alternatively, the debug output should be trivial to reproduce with the test code provided above.

Other

No response

[ryanthemanuel]

Hi @micgro42. This is actually a function of how chai works in terms of its matchers (in this case the have.property matcher). This can be seen by changing the test code to:

    cy.window().then((win) => {
      expect(win.mw).to.have.property('propertyToCheckFor');
    })

or even:

    cy.window().then((win) => {
      expect(win.mw).to.exist;
    })

If calling inspect is causing a problem, I would recommend switching up how you're asserting. Something like this could work:

    cy.window().then((win) => {
      expect(win.mw.propertyToCheckFor).to.exist;
    })

[micgro42]

@ryanthemanuel: Thank you for your quick response and suggestions! 🙏
(Though doesn't .then break retry-ability?)

However, I do miss somewhat it acknowledging the core point that this is a bug resulting in unexpected and undocumented behavior in Cypress.

I understand, that from Cypress' point of view, this is also an upstream bug in a library that you're using. But I still think this should remain open until Chai fixes that behavior and you can update it in your dependencies.

[jennifer-shehane]

@micgro42 I think this is expected behavior, so it should likely be documented better.

[cypress-app-bot]

This issue has not had any activity in 180 days. Cypress evolves quickly and the reported behavior should be tested on the latest version of Cypress to verify the behavior is still occurring. It will be closed in 14 days if no updates are provided.