Merge pull request #18574 from calixteman/add_window_uimanager_test

[Editor] Add the uiManager to the window object when testing
mozilla · Aug 7, 2024 · 3cad8d1 · 3cad8d1
2 parents 341a0b6 + e037c57
commit 3cad8d1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 11 deletions.
diff --git a/src/display/editor/tools.js b/src/display/editor/tools.js
@@ -817,6 +817,7 @@ class AnnotationEditorUIManager {
     this.isShiftKeyDown = false;
 
     if (typeof PDFJSDev !== "undefined" && PDFJSDev.test("TESTING")) {
+      window.uiManager = this;
       Object.defineProperty(this, "reset", {
         value: () => {
           this.selectAll();

diff --git a/test/integration/highlight_editor_spec.mjs b/test/integration/highlight_editor_spec.mjs
@@ -997,17 +997,7 @@ describe("Highlight Editor", () => {
         "tracemonkey.pdf",
         ".annotationEditorLayer",
         null,
-        async page => {
-          await page.evaluate(async () => {
-            await window.PDFViewerApplication.initializedPromise;
-            window.PDFViewerApplication.eventBus.on(
-              "annotationeditoruimanager",
-              ({ uiManager }) => {
-                window.uiManager = uiManager;
-              }
-            );
-          });
-        },
+        null,
         {
           highlightEditorColors: "red=#AB0000",
           supportsCaretBrowsingMode: true,