Dl/cypress@12 expect fix

[wKich]

Motivation

Fix #234

Approach

It seems Cypress@12 doesn't expose _commands field https://github.com/cypress-io/cypress/blob/ae4cf2b553255f38ca83e14a09dd891b487e1a39/packages/driver/src/cypress/commands.ts#L45 which cause the error mentioned in #234

[changeset-bot]

🦋 Changeset detected

Latest commit: 4615e5b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@interactors/with-cypress	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[netlify]

✅ Deploy Preview for interactors canceled.

Name	Link
🔨 Latest commit	4615e5b
🔍 Latest deploy log	https://app.netlify.com/sites/interactors/deploys/64ef18b605b4a50007bcf2c0

[zburke]

pedantic: s/overwiritability/overwritability/

[wKich]

Thanks :)

[cowboyd]

Good to go! Although it looks like it's not passing on CI.

[ostapwd]

Hi @wKich!
It looks like cypres.json should be replaced by cypress.config.ts in /packages/with-cypress dir

[cowboyd]

@wKich Odd that this is still failing Any idea what's up?

[wKich]

@cowboyd yep. It's time to fix cypress-io/cypress#22733

[ostapwd]

Hi @cowboyd @wKich
In my team, we are using Interactors for the FOLIO project, and we need the latest version of the Cypress. As a workaround is there a chance to add a new command instead of overriding existing, somethings like 'expectThat' (cy.expectThat())?

This approach might look ugly but we will avoid any conflicts in the future.
Also if it's possible to keep this logic
cy.expectThat = ( interaction: AssertionInteraction<Element> | AssertionInteraction<Element>[] | unknown, message?: string ) => { let interactions = Array.isArray(interaction) ? interaction : [interaction] if (isInteractions(interactions)) { return interactionExpect(interactions) } else { return chaiExpect(interaction, message) } }
all we have to do in our project is just to replace cy.expect() with cy.expectThat and that shouldn't be painful

[wKich]

@ostapwd added a fallback expectInteractor. According cypress code they are trying to restrict users from overwriting built-in commands. So I think we need to do a breaking change and rename cy.expect to something else. In this PR I just added the fallback.

@cowboyd fyi

[cowboyd]

I really like @ostapwd's suggestion! cy.expectThat() /cc @wKich