From 17dc9e9917e59ead7fd3d30ef329d2fd919a6372 Mon Sep 17 00:00:00 2001 From: Tim van der Meij Date: Mon, 28 Jul 2025 21:08:12 +0200 Subject: [PATCH] Skip the "must check that an existing highlight is ignored on hovering" integration test on Windows This is a temporary measure to reduce noise until #20136 is fixed. Note that this shouldn't be an issue in terms of coverage because we still run the test on Linux. --- test/integration/highlight_editor_spec.mjs | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/test/integration/highlight_editor_spec.mjs b/test/integration/highlight_editor_spec.mjs index 2c4dd0f6c..c23f9999a 100644 --- a/test/integration/highlight_editor_spec.mjs +++ b/test/integration/highlight_editor_spec.mjs @@ -1853,6 +1853,12 @@ describe("Highlight Editor", () => { it("must check that an existing highlight is ignored on hovering", async () => { await Promise.all( pages.map(async ([browserName, page]) => { + if (navigator.platform.includes("Win")) { + // Skip the test on Windows because it permafails. + // TODO: Remove this check once #20136 is fixed. + return; + } + await switchToHighlight(page); const rect = await getSpanRectFromText(