Beanz/pdf.js
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use the closePages helper function more in the integration test code

Browse Source 
This test called `closeSinglePage` manually at the end of the test,
which is inconsistent with all other tests that call `closePages` in an
`afterEach` block. This commit fixes the difference for consistency.
This commit is contained in:
Tim van der Meij 2025-08-17 15:16:29 +02:00
parent 2a93ade197
commit c09ad5a2ce
No known key found for this signature in database
GPG Key ID: 8C3FD2925A5F2762
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
test/integration/scripting_spec.mjs
View File

@ -17,7 +17,6 @@ import {
awaitPromise, awaitPromise,
clearInput, clearInput,
closePages, closePages,
closeSinglePage,
getAnnotationStorage, getAnnotationStorage,
getComputedStyleSelector, getComputedStyleSelector,
getFirstSerialized, getFirstSerialized,
@ -427,6 +426,10 @@ describe("Interaction", () => {
}); });
}); });
afterEach(async () => {
await closePages(pages);
});
it("must execute WillPrint and DidPrint actions", async () => { it("must execute WillPrint and DidPrint actions", async () => {
await Promise.all( await Promise.all(
pages.map(async ([browserName, page]) => { pages.map(async ([browserName, page]) => {
@ -450,7 +453,6 @@ describe("Interaction", () => {
await page.waitForFunction( await page.waitForFunction(
`${getQuerySelector("50R")}.value === "DidPrint"` `${getQuerySelector("50R")}.value === "DidPrint"`
); );
await closeSinglePage(page);
}) })
); );
}); });