From 46da690cdd9bb7f50c84442faa760a3507c73cd5 Mon Sep 17 00:00:00 2001
From: Pranav C <pranavxc@gmail.com>
Date: Fri, 17 May 2024 09:35:48 +0000
Subject: [PATCH] test: delete table - add class to avoid ambiguity

---
 packages/nc-gui/components/dashboard/TreeView/TableNode.vue | 2 +-
 tests/playwright/pages/Dashboard/TreeView.ts                | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/nc-gui/components/dashboard/TreeView/TableNode.vue b/packages/nc-gui/components/dashboard/TreeView/TableNode.vue
index dede1f8c00..6b8844c24b 100644
--- a/packages/nc-gui/components/dashboard/TreeView/TableNode.vue
+++ b/packages/nc-gui/components/dashboard/TreeView/TableNode.vue
@@ -358,7 +358,7 @@ const deleteTable = () => {
                 <NcMenuItem
                   v-if="isUIAllowed('tableDelete', { roles: baseRole })"
                   :data-testid="`sidebar-table-delete-${table.title}`"
-                  class="!text-red-500 !hover:bg-red-50"
+                  class="!text-red-500 !hover:bg-red-50 nc-table-delete"
                   @click="deleteTable"
                 >
                   <div v-e="['c:table:delete']" class="flex gap-2 items-center">
diff --git a/tests/playwright/pages/Dashboard/TreeView.ts b/tests/playwright/pages/Dashboard/TreeView.ts
index cc226a0537..bb11ea9fef 100644
--- a/tests/playwright/pages/Dashboard/TreeView.ts
+++ b/tests/playwright/pages/Dashboard/TreeView.ts
@@ -182,7 +182,7 @@ export class TreeViewPage extends BasePage {
     await this.waitForTableOptions({ title });
 
     await this.get().locator(`.nc-base-tree-tbl-${tableTitle}`).locator('.nc-tbl-context-menu').click();
-    await this.rootPage.locator('.ant-dropdown').locator('.nc-menu-item:has-text("Delete")').click();
+    await this.rootPage.locator('.ant-dropdown').locator('.nc-menu-item.nc-table-delete:has-text("Delete")').click();
 
     await this.waitForResponse({
       uiAction: async () => {