Clean up before closing the tab in integration tests

mozilla · Jun 18, 2024 · c1f7363 · c1f7363
1 parent 06800cd
commit c1f7363
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/test/integration/test_utils.mjs b/test/integration/test_utils.mjs
@@ -79,7 +79,12 @@ function closePages(pages) {
  return Promise.all(
  pages.map(async ([_, page]) => {
  // Avoid to keep something from a previous test.
- await page.evaluate(() => window.localStorage.clear());
+ await page.evaluate(() => {
+ const viewer = window.PDFViewerApplication;
+ viewer.unbindWindowEvents();
+ viewer.unbindEvents();
+ window.localStorage.clear();
+ });
  await page.close({ runBeforeUnload: false });
  })
  );

diff --git a/web/app.js b/web/app.js
@@ -2098,9 +2098,13 @@ const PDFViewerApplication = {
  unbindWindowEvents() {
  this._windowAbortController?.abort();
  this._windowAbortController = null;
- if (AppOptions.get("isInAutomation")) {
+ if (
+ (typeof PDFJSDev !== "undefined" && PDFJSDev.test("TESTING")) ||
+ AppOptions.get("isInAutomation")
+ ) {
  this._globalAbortController?.abort();
  this._globalAbortController = null;
+ this.pdfHistory?.reset();
  }
  },