From 7f74fbf8fd0043a9470dabd055d1953d956c6477 Mon Sep 17 00:00:00 2001 From: Tim van der Meij Date: Sun, 17 Aug 2025 17:10:58 +0200 Subject: [PATCH] Fix block spacing inconsistencies in the integration test code Most places have a newline before/after `before{Each,All}`, `after{Each,All}` and `it` to visually separate the blocks for clarity, but in a handful of places this wasn't done. This commit removes the inconsistencies so that the test code is formatted consistently. --- test/integration/text_layer_spec.mjs | 4 ++++ test/integration/viewer_spec.mjs | 2 ++ 2 files changed, 6 insertions(+) diff --git a/test/integration/text_layer_spec.mjs b/test/integration/text_layer_spec.mjs index a9c7fa41e..47799fef4 100644 --- a/test/integration/text_layer_spec.mjs +++ b/test/integration/text_layer_spec.mjs @@ -117,6 +117,7 @@ describe("Text layer", () => { `.page[data-page-number = "1"] .endOfContent` ); }); + afterEach(async () => { await closePages(pages); }); @@ -226,6 +227,7 @@ describe("Text layer", () => { `.page[data-page-number = "1"] .endOfContent` ); }); + afterEach(async () => { await closePages(pages); }); @@ -315,6 +317,7 @@ describe("Text layer", () => { `.page[data-page-number = "1"] .endOfContent` ); }); + afterEach(async () => { await closePages(pages); }); @@ -456,6 +459,7 @@ describe("Text layer", () => { { timeout: 0 } ); }); + afterEach(async () => { await closeSinglePage(page); await browser.close(); diff --git a/test/integration/viewer_spec.mjs b/test/integration/viewer_spec.mjs index 907a8745b..f2d3447eb 100644 --- a/test/integration/viewer_spec.mjs +++ b/test/integration/viewer_spec.mjs @@ -1358,12 +1358,14 @@ describe("PDF viewer", () => { describe("Pinch-zoom", () => { let pages; + beforeEach(async () => { pages = await loadAndWait( "tracemonkey.pdf", `.page[data-page-number = "1"] .endOfContent` ); }); + it("keeps the content under the pinch centre fixed on the screen", async () => { await Promise.all( pages.map(async ([browserName, page]) => {