-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
use latest cy.screenshot()
#13
Comments
Hey, that's great 👏 Will definitely have a look over the weekend and see how much I can simplify things with this 👍 |
just as a heads up: in 3.0.2, the file structure of saved screenshots changed, so cypress-match-screenshot is currently broken in the newest versions of cypress. |
yeah, sorry, I'm a bit busy atm, so I didn't get to have a look at this yet, but will definitely do as soon as I can 😊 |
no worries, just wanted to spare you some frustration when you get around to it ;) |
hey, I finally got to spend some time on this 😄 I removed the cropping logic and adjusted the diffing functionality, so that the plugin works again with the latest cypress version (v3.0.2) ... check out That being said, the new screenshot functionality in cypress seems to be a bit iffy still. When changing the window size of the test runner, cypress doesn't scale the app window to the defined dimensions, which means diffing will fail ... this was part of the cropping logic that I had in place previously (which was also very fragile), so I'll have to think about how to work around this. |
@julianburr nice! Thanks for your work!
Maybe it makes sense to open an issue in the cypress repo? |
cypress gives error unresolved function or method , how can i solve the issue has I have followed all the instructions in usage documentation, thanks in advance |
Hey, nice work here! 👍
FYI,
cy.screenshot()
has been rewritten from the ground up in the3.0.0
version of cypress which eliminates the need for sophisticated cropping code here. Check it out:https://docs.cypress.io/guides/references/changelog.html#3-0-0
The text was updated successfully, but these errors were encountered: