Skip to content
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

Image regression tests #110

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Image regression tests #110

wants to merge 14 commits into from

Conversation

w0rm
Copy link
Collaborator

@w0rm w0rm commented Apr 1, 2024

This adds image regression tests using Playwright. Unfortunately not all browsers were reliable. Chrome was rendering some black areas in white. Webkit didn't work in some cases - was rendering white. At first Firefox didn't work at all — WebGL is not available in the headless mode.

But then I tried to run it with a fake display using coactions/setup-xvfb and it only failed on two images. Apparently these were renderings of points with multisampling. I increased the tolerance for these two images, and all the tests passed!

The failed cases upload reports, that can be downloaded and examined. E.g. I changed the colours and made the tests fail:

Screenshot 2024-04-01 at 22 05 57 (1)

I couldn't make playwright work in nix, so I installed it with npm instead. Locally, the nix devshell provides nodejs, that could be used to install playwright and its browsers.

In order to speedup the tests, I added sharding and a cache for playwright.

@w0rm w0rm force-pushed the image-regression-tests branch from 40c8530 to 662540b Compare April 1, 2024 16:54
@w0rm w0rm force-pushed the image-regression-tests branch from b76ec1f to 5004cca Compare April 1, 2024 17:26
@w0rm w0rm force-pushed the image-regression-tests branch from b20cad7 to 1db868a Compare April 1, 2024 18:00
@w0rm w0rm force-pushed the image-regression-tests branch from fcbd21a to 4f90b7f Compare April 1, 2024 18:36
@w0rm w0rm force-pushed the image-regression-tests branch from 5108da0 to 85a7238 Compare April 1, 2024 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant