From 923753a66b289b6c2413b64e0ed8ec7b89ae6301 Mon Sep 17 00:00:00 2001 From: Tim van der Meij Date: Sun, 13 Apr 2025 19:26:42 +0200 Subject: [PATCH] Skip the "must check that canvas perfectly fits the page whatever the zoom level" integration test in Chrome This is a temporary measure to reduce noise until #19811 is fixed. Note that this shouldn't be an issue in terms of coverage because we still run the test in Firefox. --- test/integration/viewer_spec.mjs | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/test/integration/viewer_spec.mjs b/test/integration/viewer_spec.mjs index c9416fbae..52b89c90f 100644 --- a/test/integration/viewer_spec.mjs +++ b/test/integration/viewer_spec.mjs @@ -400,6 +400,14 @@ describe("PDF viewer", () => { it("must check that canvas perfectly fits the page whatever the zoom level is", async () => { await Promise.all( pages.map(async ([browserName, page]) => { + if (browserName === "chrome") { + // Skip the test for Chrome as `scrollIntoView` below hangs since + // Puppeteer 24.5.0 and higher. + // See https://github.com/mozilla/pdf.js/issues/19811. + // TODO: Remove this check once the issue is fixed. + return; + } + // The pdf has a single page with a red background. // We set the viewer background to red, because when screenshoting // some part of the viewer background can be visible.