From dfa4a9c14bb36c69856571637671febc3d8aa914 Mon Sep 17 00:00:00 2001
From: Jannik Stehle <jannik.stehle@kiteworks.com>
Date: Tue, 16 Jul 2024 14:33:05 +0200
Subject: [PATCH] test: fix e2e move- and copy-tests

---
 .../objects/app-files/resource/actions.ts     | 44 ++++---------------
 1 file changed, 8 insertions(+), 36 deletions(-)

diff --git a/tests/e2e/support/objects/app-files/resource/actions.ts b/tests/e2e/support/objects/app-files/resource/actions.ts
index c852f95093e..2c7afe39078 100644
--- a/tests/e2e/support/objects/app-files/resource/actions.ts
+++ b/tests/e2e/support/objects/app-files/resource/actions.ts
@@ -830,12 +830,7 @@ export const pasteResource = async (args: moveOrCopyResourceArgs): Promise<void>
   }
   if (option) {
     await Promise.all([
-      page.waitForResponse(
-        (resp) =>
-          resp.url().endsWith(resource) &&
-          resp.ok &&
-          resp.request().method() === action.toUpperCase()
-      ),
+      page.waitForResponse((resp) => resp.ok && resp.request().method() === action.toUpperCase()),
       option === 'replace'
         ? page.locator(actionSecondaryConfirmationButton).click()
         : page.locator(keepBothButton).click()
@@ -859,16 +854,11 @@ export const moveOrCopyMultipleResources = async (
 
   const waitForMoveResponses = []
   if (['drag-drop-breadcrumb', 'drag-drop'].includes(method)) {
-    for (const resource of resources) {
+    resources.forEach(() => {
       waitForMoveResponses.push(
-        page.waitForResponse(
-          (resp) =>
-            resp.url().endsWith(resource) &&
-            resp.status() === 201 &&
-            resp.request().method() === 'MOVE'
-        )
+        page.waitForResponse((resp) => resp.status() === 201 && resp.request().method() === 'MOVE')
       )
-    }
+    })
   }
 
   switch (method) {
@@ -985,10 +975,7 @@ export const moveOrCopyResource = async (args: moveOrCopyResourceArgs): Promise<
       }
       await Promise.all([
         page.waitForResponse(
-          (resp) =>
-            resp.url().endsWith(resource) &&
-            resp.status() === 201 &&
-            resp.request().method() === action.toUpperCase()
+          (resp) => resp.status() === 201 && resp.request().method() === action.toUpperCase()
         ),
         page.keyboard.press('ControlOrMeta+v')
       ])
@@ -999,12 +986,7 @@ export const moveOrCopyResource = async (args: moveOrCopyResourceArgs): Promise<
       const target = page.locator(util.format(resourceNameSelector, newLocation))
 
       await Promise.all([
-        page.waitForResponse(
-          (resp) =>
-            resp.url().endsWith(resource) &&
-            resp.status() === 201 &&
-            resp.request().method() === 'MOVE'
-        ),
+        page.waitForResponse((resp) => resp.status() === 201 && resp.request().method() === 'MOVE'),
         source.dragTo(target)
       ])
 
@@ -1022,12 +1004,7 @@ export const moveOrCopyResource = async (args: moveOrCopyResourceArgs): Promise<
       )
 
       await Promise.all([
-        page.waitForResponse(
-          (resp) =>
-            resp.url().endsWith(resource) &&
-            resp.status() === 201 &&
-            resp.request().method() === 'MOVE'
-        ),
+        page.waitForResponse((resp) => resp.status() === 201 && resp.request().method() === 'MOVE'),
         source.dragTo(target)
       ])
 
@@ -1073,12 +1050,7 @@ export const renameResource = async (args: renameResourceArgs): Promise<void> =>
   await page.locator(util.format(filesContextMenuAction, 'rename')).click()
   await page.locator(fileRenameInput).fill(newName)
   await Promise.all([
-    page.waitForResponse(
-      (resp) =>
-        resp.url().endsWith(resourceBase) &&
-        resp.status() === 201 &&
-        resp.request().method() === 'MOVE'
-    ),
+    page.waitForResponse((resp) => resp.status() === 201 && resp.request().method() === 'MOVE'),
     page.locator(util.format(actionConfirmationButton, 'Rename')).click()
   ])
 }