cypress-plugin-visual-regression-diff: feat: images of difference sizes cause instant test failure
Sometimes Cypress takes screenshots of slightly differences sizes during the same test. If the screnshots are different sizes the test fails. But I notice in the code that it calls alignImagesToSameSize function if they are different sizes to try and do the comparison on resized images. If the test always fails what is the reason for alignImagesToSameSize? Perhaps it should attempt to continue with the comparison after resizing the images?
Example of the diff we see in screenshot size between runs:

About this issue
- Original URL
- State: closed
- Created 2 years ago
- Comments: 15 (12 by maintainers)
Commits related to this issue
- feat: img diff when resolution differs BREAKING CHANGE: different resolution doesn't fail test immediately - img diff is being done closes #94 Signed-off-by: Jakub Freisler <jakub@frsource.org> — committed to FRSOURCE/cypress-plugin-visual-regression-diff by FRSgit 2 years ago
- feat: img diff when resolution differs BREAKING CHANGE: different resolution doesn't fail test immediately - img diff is being done closes #94 Signed-off-by: Jakub Freisler <jakub@frsource.org> — committed to FRSOURCE/cypress-plugin-visual-regression-diff by FRSgit 2 years ago
- feat: img diff when resolution differs (#108) BREAKING CHANGE: different resolution doesn't fail test immediately - img diff is being done closes #94 — committed to FRSOURCE/cypress-plugin-visual-regression-diff by FRSgit 2 years ago
- chore(release): 2.0.0 [skip ci] # [2.0.0](https://github.com/FRSOURCE/cypress-plugin-visual-regression-diff/compare/v1.9.21...v2.0.0) (2022-09-15) ### Features * img diff when resolution differs ([... — committed to FRSOURCE/cypress-plugin-visual-regression-diff by semantic-release-bot 2 years ago
- chore(release): 1.0.0 [skip ci] # 1.0.0 (2022-10-31) ### Bug Fixes * btoa utf8 encoding/decoding error ([#114](https://github.com/braze-inc/cypress-plugin-visual-regression-diff/issues/114)) ([0137... — committed to braze-inc/cypress-plugin-visual-regression-diff by semantic-release-bot 2 years ago
Uf, I started wondering what should I do with this PR 😄 Thanks for confirming that it works @dhulme! I’ll do a full 2.0.0 release later today