-
Notifications
You must be signed in to change notification settings - Fork 571
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
Remove Applitools dependency for visual testing #1259
Conversation
Switch to Firefox for e2e due to cypress-io/cypress#2034
I am getting this:
|
Oh my bad, I am not familiar with cypress, turns out I had to run the site locally using
|
I'm sorry, my bad: Cypress by default was still running on the embedded electron browser. |
We can increase the threshold for screenshot matching, this can happen when the testing environment is not exactly the same, are you perchance using Windows? |
I emailed @valorkin if he doesn't respond maybe I should simply fork the repo to my own user and add you as maintainer. To your question, I am using windows. |
@santam85 I tried to run the test inside a linux container, they still fail... I used the |
Opened #1262 to try and address this. |
Not sure how to do that |
My bad, i introduce a syntax error in the travis.yml, will fix it ASAP |
As requested by the maintainer. Switching to local snapshot diff.
Had to switch to Firefox for e2e testing due to bugs in Cypress related to SVG snapshots on chrome (cypress-io/cypress#2034) which seems far away from resolution.
Also updated various dependencies.