test/integration/freetext_editor_spec.mjs

@@ -87,11 +87,11 @@ describe("FreeText Editor", () => {
   describe("FreeText", () => {
     let pages;
 
-    beforeEach(async () => {
+    beforeAll(async () => {
       pages = await loadAndWait("aboutstacks.pdf", ".annotationEditorLayer");
     });
 
-    afterEach(async () => {
+    afterAll(async () => {
       await closePages(pages);
     });
 
@@ -134,17 +134,7 @@ describe("FreeText Editor", () => {
     it("must copy/paste", async () => {
       // Run sequentially to avoid clipboard issues.
       for (const [browserName, page] of pages) {
-        await switchToFreeText(page);
-
-        const rect = await getRect(page, ".annotationEditorLayer");
         const firstEditorSelector = getEditorSelector(0);
-        const data = "Hello PDF.js World !!";
-        await page.mouse.click(rect.x + 100, rect.y + 100);
-        await page.waitForSelector(firstEditorSelector, { visible: true });
-        await page.type(`${firstEditorSelector} .internal`, data);
-        await commit(page);
-        await waitForStorageEntries(page, 1);
-
         await selectEditor(page, firstEditorSelector);
         await copy(page);
         await paste(page);
@@ -178,35 +168,18 @@ describe("FreeText Editor", () => {
     it("must clear all", async () => {
       await Promise.all(
         pages.map(async ([browserName, page]) => {
-          await switchToFreeText(page);
-
-          const rect = await getRect(page, ".annotationEditorLayer");
-          for (const n of [0, 1, 2]) {
-            const editorSelector = getEditorSelector(n);
-            const data = "Hello PDF.js World !!";
-            await page.mouse.click(rect.x + 100 * n, rect.y + 100 * n);
-            await page.waitForSelector(editorSelector, { visible: true });
-            await page.type(`${editorSelector} .internal`, data);
-            await commit(page);
-
-            const hasEditor = await page.evaluate(
-              sel => !!document.querySelector(sel),
-              editorSelector
-            );
-            expect(hasEditor).withContext(`In ${browserName}`).toEqual(true);
-          }
-
-          await waitForStorageEntries(page, 3);
           await clearAll(page);
-          await waitForStorageEntries(page, 0);
 
           for (const n of [0, 1, 2]) {
             const hasEditor = await page.evaluate(
               sel => !!document.querySelector(sel),
               getEditorSelector(n)
             );
+
             expect(hasEditor).withContext(`In ${browserName}`).toEqual(false);
           }
+
+          await waitForStorageEntries(page, 0);
         })
       );
     });
@@ -214,10 +187,8 @@ describe("FreeText Editor", () => {
     it("must check that a paste has been undone", async () => {
       // Run sequentially to avoid clipboard issues.
       for (const [, page] of pages) {
-        await switchToFreeText(page);
-
         const rect = await getRect(page, ".annotationEditorLayer");
-        let editorSelector = getEditorSelector(0);
+        let editorSelector = getEditorSelector(3);
         const data = "Hello PDF.js World !!";
         await page.mouse.click(rect.x + 100, rect.y + 100);
         await page.waitForSelector(editorSelector, { visible: true });
@@ -227,7 +198,7 @@ describe("FreeText Editor", () => {
         await selectEditor(page, editorSelector);
         await copy(page);
         await paste(page);
-        editorSelector = getEditorSelector(1);
+        editorSelector = getEditorSelector(4);
         await page.waitForSelector(editorSelector, { visible: true });
 
         await kbUndo(page);
@@ -239,7 +210,7 @@ describe("FreeText Editor", () => {
 
         for (let i = 0; i < 2; i++) {
           await paste(page);
-          await page.waitForSelector(getEditorSelector(2 + i));
+          await page.waitForSelector(getEditorSelector(5 + i));
         }
 
         for (let i = 0; i < 2; i++) {
@@ -247,7 +218,7 @@ describe("FreeText Editor", () => {
           await page.waitForFunction(
             sel => !document.querySelector(sel),
             {},
-            getEditorSelector(3 - i)
+            getEditorSelector(6 - i)
           );
         }
       }
@@ -256,8 +227,6 @@ describe("FreeText Editor", () => {
     it("must check that aria-owns is correct", async () => {
       await Promise.all(
         pages.map(async ([browserName, page]) => {
-          await switchToFreeText(page);
-
           await page.$eval(".textLayer", el => {
             for (const span of el.querySelectorAll(
               `span[role="presentation"]`
@@ -277,7 +246,7 @@ describe("FreeText Editor", () => {
 
           expect(oldAriaOwns).withContext(`In ${browserName}`).toEqual(null);
 
-          const editorSelector = getEditorSelector(0);
+          const editorSelector = getEditorSelector(7);
           const data = "Hello PDF.js World !!";
           await page.mouse.click(
             stacksRect.x + stacksRect.width + 1,
@@ -292,7 +261,7 @@ describe("FreeText Editor", () => {
             return span?.getAttribute("aria-owns") || null;
           });
 
-          expect(ariaOwns.endsWith("_0-editor"))
+          expect(ariaOwns.endsWith("_7-editor"))
             .withContext(`In ${browserName}`)
             .toEqual(true);
           await scrollIntoView(page, ".annotationEditorLayer");
@@ -303,10 +272,11 @@ describe("FreeText Editor", () => {
     it("must check that right click doesn't select", async () => {
       await Promise.all(
         pages.map(async ([browserName, page]) => {
-          await switchToFreeText(page);
-
           const rect = await getRect(page, ".annotationEditorLayer");
-          const editorSelector = getEditorSelector(0);
+
+          await clearAll(page);
+
+          const editorSelector = getEditorSelector(8);
           const data = "Hello PDF.js World !!";
           await page.mouse.click(rect.x + 100, rect.y + 100);
           await page.waitForSelector(editorSelector, { visible: true });
@@ -315,7 +285,7 @@ describe("FreeText Editor", () => {
 
           expect(await getEditors(page, "selected"))
             .withContext(`In ${browserName}`)
-            .toEqual([0]);
+            .toEqual([8]);
 
           await page.keyboard.press("Escape");
           await page.waitForFunction(
@@ -325,7 +295,7 @@ describe("FreeText Editor", () => {
           await selectEditor(page, editorSelector);
           expect(await getEditors(page, "selected"))
             .withContext(`In ${browserName}`)
-            .toEqual([0]);
+            .toEqual([8]);
 
           // Escape.
           await page.keyboard.press("Escape");
@@ -347,10 +317,11 @@ describe("FreeText Editor", () => {
     it("must check that text change can be undone/redone", async () => {
       // Run sequentially to avoid clipboard issues.
       for (const [browserName, page] of pages) {
-        await switchToFreeText(page);
-
         const rect = await getRect(page, ".annotationEditorLayer");
-        const editorSelector = getEditorSelector(0);
+
+        await clearAll(page);
+
+        const editorSelector = getEditorSelector(9);
         await page.mouse.click(rect.x + 200, rect.y + 100);
         await page.waitForSelector(editorSelector, { visible: true });