microlinkhq / browserless / 42411130289

77%

push

github

test(browserless): assert screenshot properties instead of pixel snapshots (#1029)

* test(browserless): assert screenshot properties instead of pixel snapshots

CI showed the committed snapshots match neither platform: a correct
linux render is 754 pixels off example.png while a blank page is 426.
Pixel diffing against them could not tell a correct render from a
blank one, and only passed on CI because the threshold was 50000.

The screenshot tests now check what browserless controls: the file
format, the dimensions (viewport and device emulation), and that the
content was painted, measured against a render of the same page with
invisible text. This runs on every platform and drops the binary
snapshots and fs-extra.

Closes #1021

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* test(browserless): name the screenshot format each case covers

The png and device cases inherited the helper default, so the suite still passed when that default became jpeg.

Co-authored-by: Cursor <cursoragent@cursor.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>

2210 of 2490 branches covered (88.76%)

Branch coverage included in aggregate %.

7825 of 10419 relevant lines covered (75.1%)

57688.48 hits per line