cypress-plugin-snapshots icon indicating copy to clipboard operation
cypress-plugin-snapshots copied to clipboard

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

Open kuceb opened this issue 3 years ago • 13 comments

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.

kuceb avatar Aug 12 '20 16:08 kuceb

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.

meinaart avatar Aug 13 '20 08:08 meinaart

@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 avatar Aug 20 '20 15:08 kuceb

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

kuceb avatar Aug 20 '20 16:08 kuceb

+1

jcasasg avatar Sep 29 '20 12:09 jcasasg

+1

davidzambrana avatar Oct 23 '20 13:10 davidzambrana

+1 Any update on this? 🤞

paolo-chiabrera avatar Oct 27 '20 23:10 paolo-chiabrera

Any update?

BioCarmen avatar Jan 15 '21 13:01 BioCarmen

Eagerly awaiting an update on this too :)

henetiriki avatar Jan 25 '21 19:01 henetiriki

Hope we can get this merged soon!

reidmovis avatar Mar 17 '21 18:03 reidmovis

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

hariaw avatar Mar 25 '21 22:03 hariaw

Any news here ?

edimitchel avatar Jun 23 '21 13:06 edimitchel

Would be good to rebase this with latest master @Bkucera

cloakedninjas avatar Oct 14 '21 08:10 cloakedninjas

Any progress on this?

francisco-polaco avatar May 22 '23 15:05 francisco-polaco