microlinkhq / browserless / 42360917088

77%

push

github

test(goto): wait for fingerprint demos with waitForFunction (#1027)

* test(goto): wait for fingerprint demos with waitForFunction

Both evasion e2e tests read a fingerprint from a third-party page that
renders it asynchronously. creepjs still dereferenced
`.fingerprint-header div` without a null check, the same race #1020
reported for fingerprintjs. Both now go through page.waitForFunction,
which treats a missing element as not-ready and is bounded by the
browserless timeout instead of an unbounded hand-rolled loop.

Closes #1020

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

* test(goto): keep fingerprint waits inside the context timeout

Puppeteer's 30s waitForFunction default would fail a slow demo the previous loop kept polling. Disable that timer and cover the header states the new predicates treat as not ready.

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

---------

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

2208 of 2488 branches covered (88.75%)

Branch coverage included in aggregate %.

7820 of 10414 relevant lines covered (75.09%)

57716.05 hits per line