TEST

Author: timvandermeij

test/integration/reorganize_pages_spec.mjs

@@ -209,27 +209,36 @@ describe("Reorganize Pages View", () => {
       );
     });
 
-    it("should reorder thumbnails after dropping", async () => {
+    fit("should reorder thumbnails after dropping", async () => {
       await Promise.all(
         pages.map(async ([browserName, page]) => {
+          page.on('console', msg => console.log('PAGE LOG:', msg.text()));
+          console.log('1');
           await waitForThumbnailVisible(page, 1);
+          console.log('2');
           const rect1 = await getRect(page, getThumbnailSelector(1));
           const rect2 = await getRect(page, getThumbnailSelector(2));
+          console.log('3');
 
           const handlePagesEdited = await waitForPagesEdited(page);
+          console.log('4');
           await dragAndDrop(
             page,
             getThumbnailSelector(1),
             [[0, rect2.y - rect1.y + rect2.height / 2]],
             10
           );
+          console.log('5');
           const pagesMapping = await awaitPromise(handlePagesEdited);
+          console.log('6');
           const expected = [
             2, 1, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17,
           ];
+          console.log('7');
           expect(pagesMapping)
             .withContext(`In ${browserName}`)
             .toEqual(expected);
+          console.log('8');
           await waitForHavingContents(page, expected);
         })
       );

test/integration/test_utils.mjs

@@ -586,6 +586,7 @@ async function dragAndDrop(page, selector, translations, steps = 1) {
   }
   await page.mouse.up();
   await page.waitForSelector("#viewer:not(.noUserSelect)");
+  await page.waitForSelector("#thumbnailsView:not(.isDragging)");
 }
 
 function waitForPageChanging(page) {

web/pdf_thumbnail_viewer.js

@@ -733,6 +733,7 @@ class PDFThumbnailViewer {
   }
 
   #onStopDragging(isDropping = false) {
+    console.error('onStopDragging called');
     const draggedContainer = this.#draggedContainer;
     this.#draggedContainer = null;
     const lastDraggedOverIndex = this.#lastDraggedOverIndex;