-
Notifications
You must be signed in to change notification settings - Fork 54
Node crash if dimension mismatch in I.seeVisualDiff
#41
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
Comments
That error is set by choice. Because if the dimensions aren't same then the tool behaves a bit unpredictably. |
I get that the diff-image produced by resemblejs with two images of different size isn't meaningful. But NODE crashes completely... which isn't a good thing. The error thrown should just cause the scenario to fail and then codeceptjs should get on with the next scenario. You still have the error in your tests about size mismatch in the end. |
Instead of throwing error here, also |
Which node version are you using? |
To quote myself here:
|
I ran into the same today. |
When using release 1.6.0 of this helper, node crashes with the following error if the dimension of the screenshots is not the same:
This is most likely due to the version of resemblejs being used. In the package.json the version specified is: ^3.0.0 which installs 3.2.0. A simple fix would be to always use 3.0.0, which worked for me.
The text was updated successfully, but these errors were encountered: