meinaart / cypress-plugin-snapshots

Plugin for snapshot tests in Cypress.io
MIT License
489 stars 116 forks source link

fix: support Cypress v5.0 (test-retries) #141

Open kuceb opened 3 years ago

kuceb commented 3 years ago

Hi, Cypress version 5.0 is landing soon, and included in that release is test-retries, which will break this plugin. Thankfully only a few changes were needed to support 5.0, so I've included them in this PR.

Although the changes in this PR will not break Cypress versions <5.0, This PR's tests will fail since it's expecting test-retries support. So I would recommend waiting for the Cypress 5.0 release and then updating to run against 5.0 before merging.

Note: this will also fix compatibility with cypress-plugin-retries (obsolete with 5.0)

Additional Info

to test against the 5.0 prerelease, I've used the following (Linux/docker):

export CYPRESS_INSTALL_BINARY=https://cdn.cypress.io/beta/binary/5.0.0/linux-x64/circle-test-retries-b0e7db25b4d6c91010b09f6ff4404bf5a7bcb74b-417938/cypress.zip
npm install --no-save https://cdn.cypress.io/beta/npm/5.0.0/circle-test-retries-b0e7db25b4d6c91010b09f6ff4404bf5a7bcb74b-417890/cypress.tgz

npm run cy:run # or cy:open

Edit

5.0 is released, I've added CI test runs for 5.0 in addition to previous versions of Cypress.

meinaart commented 3 years ago

Thanks for the work. I always tried to be backwards compatible. Have to think about if I want to break backwards compatibility. I do encourage users to keep up to date with latest Cypress version.

kuceb commented 3 years ago

@meinaart This is backwards compatible. However the tests for the plugin will fail on a previous Cypress version. I'll add some code to test on multiple Cypress versions

kuceb commented 3 years ago

@meinaart take a look at the changes to CI, I've added testing for previous versions of cypress and 5.0.

jcasasg commented 3 years ago

+1

davidzambrana commented 3 years ago

+1

paolo-chiabrera commented 3 years ago

+1 Any update on this? 🤞

BioCarmen commented 3 years ago

Any update?

henetiriki commented 3 years ago

Eagerly awaiting an update on this too :)

reidmovis commented 3 years ago

Hope we can get this merged soon!

hariaw commented 3 years ago

Any update on this? Would love to see it merged in, and looks ready to go? cc @meinaart

edimitchel commented 3 years ago

Any news here ?

cloakedninjas commented 2 years ago

Would be good to rebase this with latest master @Bkucera

francisco-polaco commented 1 year ago

Any progress on this?