Isolate the viewer integration tests

To avoid being able to introduce dependencies between tests this commit
makes sure that we close the document between tests so that we can't
accidentally rely on state set by a previous test.
This commit is contained in:
Tim van der Meij 2025-04-13 14:23:22 +02:00
parent 965949c259
commit 75b321b000
No known key found for this signature in database
GPG Key ID: 8C3FD2925A5F2762

View File

@ -28,7 +28,7 @@ describe("PDF viewer", () => {
describe("Zoom origin", () => { describe("Zoom origin", () => {
let pages; let pages;
beforeAll(async () => { beforeEach(async () => {
pages = await loadAndWait( pages = await loadAndWait(
"tracemonkey.pdf", "tracemonkey.pdf",
".textLayer .endOfContent", ".textLayer .endOfContent",
@ -38,7 +38,7 @@ describe("PDF viewer", () => {
); );
}); });
afterAll(async () => { afterEach(async () => {
await closePages(pages); await closePages(pages);
}); });
@ -107,11 +107,11 @@ describe("PDF viewer", () => {
describe("Zoom with the mouse wheel", () => { describe("Zoom with the mouse wheel", () => {
let pages; let pages;
beforeAll(async () => { beforeEach(async () => {
pages = await loadAndWait("empty.pdf", ".textLayer .endOfContent", 1000); pages = await loadAndWait("empty.pdf", ".textLayer .endOfContent", 1000);
}); });
afterAll(async () => { afterEach(async () => {
await closePages(pages); await closePages(pages);
}); });
@ -141,11 +141,11 @@ describe("PDF viewer", () => {
describe("Zoom commands", () => { describe("Zoom commands", () => {
let pages; let pages;
beforeAll(async () => { beforeEach(async () => {
pages = await loadAndWait("tracemonkey.pdf", ".textLayer .endOfContent"); pages = await loadAndWait("tracemonkey.pdf", ".textLayer .endOfContent");
}); });
afterAll(async () => { afterEach(async () => {
await closePages(pages); await closePages(pages);
}); });
@ -191,7 +191,7 @@ describe("PDF viewer", () => {
describe("forced (maxCanvasPixels: 0)", () => { describe("forced (maxCanvasPixels: 0)", () => {
let pages; let pages;
beforeAll(async () => { beforeEach(async () => {
pages = await loadAndWait( pages = await loadAndWait(
"tracemonkey.pdf", "tracemonkey.pdf",
".textLayer .endOfContent", ".textLayer .endOfContent",
@ -201,7 +201,7 @@ describe("PDF viewer", () => {
); );
}); });
afterAll(async () => { afterEach(async () => {
await closePages(pages); await closePages(pages);
}); });
@ -257,7 +257,7 @@ describe("PDF viewer", () => {
const MAX_CANVAS_PIXELS = new Map(); const MAX_CANVAS_PIXELS = new Map();
beforeAll(async () => { beforeEach(async () => {
pages = await loadAndWait( pages = await loadAndWait(
"tracemonkey.pdf", "tracemonkey.pdf",
".textLayer .endOfContent", ".textLayer .endOfContent",
@ -271,9 +271,7 @@ describe("PDF viewer", () => {
return { maxCanvasPixels }; return { maxCanvasPixels };
} }
); );
});
beforeEach(async () => {
await Promise.all( await Promise.all(
pages.map(async ([browserName, page]) => { pages.map(async ([browserName, page]) => {
const handle = await waitForPageRendered(page); const handle = await waitForPageRendered(page);
@ -294,7 +292,7 @@ describe("PDF viewer", () => {
); );
}); });
afterAll(async () => { afterEach(async () => {
await closePages(pages); await closePages(pages);
}); });
@ -387,7 +385,7 @@ describe("PDF viewer", () => {
describe("Canvas fits the page", () => { describe("Canvas fits the page", () => {
let pages; let pages;
beforeAll(async () => { beforeEach(async () => {
pages = await loadAndWait( pages = await loadAndWait(
"issue18694.pdf", "issue18694.pdf",
".textLayer .endOfContent", ".textLayer .endOfContent",
@ -395,7 +393,7 @@ describe("PDF viewer", () => {
); );
}); });
afterAll(async () => { afterEach(async () => {
await closePages(pages); await closePages(pages);
}); });
@ -461,7 +459,7 @@ describe("PDF viewer", () => {
function setupPages(zoom, devicePixelRatio, setups = {}) { function setupPages(zoom, devicePixelRatio, setups = {}) {
let pages; let pages;
beforeAll(async () => { beforeEach(async () => {
pages = await loadAndWait( pages = await loadAndWait(
"colors.pdf", "colors.pdf",
null, null,
@ -480,7 +478,7 @@ describe("PDF viewer", () => {
); );
}); });
afterAll(async () => { afterEach(async () => {
await closePages(pages); await closePages(pages);
}); });
@ -1121,11 +1119,11 @@ describe("PDF viewer", () => {
); );
} }
beforeAll(async () => { beforeEach(async () => {
pages = await loadAndWait("issue18694.pdf", ".textLayer .endOfContent"); pages = await loadAndWait("issue18694.pdf", ".textLayer .endOfContent");
}); });
afterAll(async () => { afterEach(async () => {
await closePages(pages); await closePages(pages);
}); });