test(FR-2652): add E2E tests for admin model card deletion with folder option

- Add tests 5.7 and 5.8 to admin-model-card-delete.spec.ts
  - 5.7: delete model card + folder together, verify notification and Go to Trash URL with folder filter
  - 5.8: delete model card only, verify notification and Go to Trash URL without filter
- Add getAlsoDeleteFolderCheckbox() and getFolderNameLinkInDeleteDialog() to AdminModelCardPage POM
- Update E2E_COVERAGE_REPORT.md: Admin Model Store 22→26 features covered

Author: agatha197

e2e/E2E_COVERAGE_REPORT.md

@@ -1,6 +1,6 @@
 # E2E Test Coverage Report
 
-> **Last Updated:** 2026-04-13
+> **Last Updated:** 2026-04-22
 > **Router Source:** [`react/src/routes.tsx`](../react/src/routes.tsx)
 > **E2E Root:** [`e2e/`](.)
 >
@@ -12,7 +12,7 @@
 
 **Scope:** Coverage metrics apply only to the routes listed below and do **not** include all entries from `react/src/routes.tsx`. Routes such as `/admin-dashboard` (not yet exposed in menu) and `/ai-agent` (experimental) are currently out of scope.
 
-**Overall (in-scope routes): 261 / 408 features covered (64%)**
+**Overall (in-scope routes): 265 / 412 features covered (64%)**
 
 | Page | Route | Features | Covered | Status |
 |------|-------|:--------:|:-------:|:------:|
@@ -27,7 +27,7 @@
 | Service Launcher | `/service/start` | 5 | 0 | ❌ 0% |
 | VFolder / Data | `/data` | 45 | 32 | 🔶 71% |
 | Model Store | `/model-store` | 6 | 6 | ✅ 100% |
-| Admin Model Store | `/admin-model-store` | 22 | 22 | ✅ 100% |
+| Admin Model Store | `/admin-model-store` | 26 | 26 | ✅ 100% |
 | Storage Host | `/storage-settings/:hostname` | 3 | 0 | ❌ 0% |
 | My Environment | `/my-environment` | 2 | 2 | ✅ 100% |
 | Environment | `/environment` | 27 | 21 | 🔶 78% |
@@ -47,7 +47,7 @@
 | Chat | `/chat/:id?` | 6 | 6 | ✅ 100% |
 | Plugin System | (config-based) | 12 | 12 | ✅ 100% |
 | RBAC Management | `/rbac` | 22 | 21 | 🔶 95% |
-| **Total** | | **408** | **261** | **64%** |
+| **Total** | | **412** | **265** | **64%** |
 
 ---
 
@@ -420,14 +420,18 @@
 | Cancel edit modal | ✅ | `admin-model-card-edit.spec.ts` |
 | Single delete with confirmation | ✅ | `admin-model-card-delete.spec.ts` |
 | Cancel single delete | ✅ | `admin-model-card-delete.spec.ts` |
+| Delete card + folder together (checkbox) | ✅ | `admin-model-card-delete.spec.ts` |
+| Notification + Go to Trash with folder filter | ✅ | `admin-model-card-delete.spec.ts` |
+| Delete card only, folder kept notification | ✅ | `admin-model-card-delete.spec.ts` |
+| Go to Trash without folder filter | ✅ | `admin-model-card-delete.spec.ts` |
 | Bulk select and delete | ✅ | `admin-model-card-delete.spec.ts` |
 | Cancel bulk delete | ✅ | `admin-model-card-delete.spec.ts` |
 | Clear selection | ✅ | `admin-model-card-delete.spec.ts` |
 | Select all via header checkbox | ✅ | `admin-model-card-delete.spec.ts` |
 | Non-admin access blocked | ✅ | `admin-model-card-access-control.spec.ts` |
 | URL state persistence (filter/sort/pagination) | ✅ | `admin-model-card-url-state.spec.ts` |
 
-**Coverage: ✅ 22/22 features**
+**Coverage: ✅ 26/26 features**
 
 ---
 

e2e/admin-model-card/admin-model-card-delete.spec.ts

@@ -266,5 +266,115 @@ test.describe(
         `${rowCount} selected`,
       );
     });
+
+    // 5.7 Superadmin can delete a model card and its associated folder together
+    test('Superadmin can delete a model card and its associated folder, and navigate to trash with folder filter', async ({
+      page,
+    }) => {
+      const adminModelCardPage = new AdminModelCardPage(page);
+      const cardName = `e2e-test-delete-with-folder-${Date.now()}`;
+
+      // Setup: create a model card (picks first available VFolder automatically)
+      await page.goto(`${webuiEndpoint}/admin-serving?tab=model-store`);
+      await adminModelCardPage.waitForTableLoad();
+      await adminModelCardPage.createModelCard({ name: cardName });
+
+      // Navigate back and filter for the created card
+      await page.goto(`${webuiEndpoint}/admin-serving?tab=model-store`);
+      await adminModelCardPage.waitForTableLoad();
+      await adminModelCardPage.applyNameFilter(cardName);
+
+      // Open the delete confirmation dialog
+      await adminModelCardPage.clickDeleteForRow(cardName);
+
+      // Verify the "Also delete the associated model folder" checkbox is visible
+      await expect(
+        adminModelCardPage.getAlsoDeleteFolderCheckbox(),
+      ).toBeVisible();
+
+      // Read the folder name from the link in the dialog (for URL assertion later)
+      const folderLink = adminModelCardPage.getFolderNameLinkInDeleteDialog();
+      await expect(folderLink).toBeVisible();
+      const folderName = await folderLink.textContent();
+      expect(folderName).toBeTruthy();
+
+      // Check the "Also delete the associated model folder" checkbox
+      await adminModelCardPage.getAlsoDeleteFolderCheckbox().check();
+      await expect(
+        adminModelCardPage.getAlsoDeleteFolderCheckbox(),
+      ).toBeChecked();
+
+      // Confirm deletion
+      await adminModelCardPage.getDeleteConfirmButton().click();
+
+      // Verify the success notification for card + folder deletion
+      await expect(
+        page.getByText('Model card and folder have been moved to trash.'),
+      ).toBeVisible({ timeout: 15000 });
+
+      // Verify "Go to Data > Trash" link is visible in the notification
+      const goToTrashLink = page.getByText('Go to Data > Trash');
+      await expect(goToTrashLink).toBeVisible();
+
+      // Click "Go to Data > Trash" and verify URL includes folder filter
+      await goToTrashLink.click();
+      await page.waitForURL(
+        (url) =>
+          url.pathname === '/data' &&
+          url.searchParams.get('statusCategory') === 'deleted' &&
+          url.searchParams.get('filter') === `name == "${folderName}"`,
+        { timeout: 10000 },
+      );
+    });
+
+    // 5.8 Superadmin deletes card only: notification shows correct message and Go to Trash navigates correctly
+    test('Superadmin can delete a model card only and navigate to trash without folder filter', async ({
+      page,
+    }) => {
+      const adminModelCardPage = new AdminModelCardPage(page);
+      const cardName = `e2e-test-delete-card-only-${Date.now()}`;
+
+      // Setup: create a model card
+      await page.goto(`${webuiEndpoint}/admin-serving?tab=model-store`);
+      await adminModelCardPage.waitForTableLoad();
+      await adminModelCardPage.createModelCard({ name: cardName });
+
+      // Navigate back and filter
+      await page.goto(`${webuiEndpoint}/admin-serving?tab=model-store`);
+      await adminModelCardPage.waitForTableLoad();
+      await adminModelCardPage.applyNameFilter(cardName);
+
+      // Open delete dialog
+      await adminModelCardPage.clickDeleteForRow(cardName);
+
+      // Leave "Also delete folder" checkbox unchecked (default)
+      await expect(
+        adminModelCardPage.getAlsoDeleteFolderCheckbox(),
+      ).not.toBeChecked();
+
+      // Confirm deletion
+      await adminModelCardPage.getDeleteConfirmButton().click();
+
+      // Verify the notification message for card-only deletion
+      await expect(
+        page.getByText(
+          'Model card has been deleted. The model folder was not deleted.',
+        ),
+      ).toBeVisible({ timeout: 15000 });
+
+      // Verify "Go to Data > Trash" link is visible
+      const goToTrashLink = page.getByText('Go to Data > Trash');
+      await expect(goToTrashLink).toBeVisible();
+
+      // Click "Go to Data > Trash" and verify URL (no folder filter)
+      await goToTrashLink.click();
+      await page.waitForURL(
+        (url) =>
+          url.pathname === '/data' &&
+          url.searchParams.get('statusCategory') === 'deleted' &&
+          !url.searchParams.has('filter'),
+        { timeout: 10000 },
+      );
+    });
   },
 );

e2e/utils/classes/AdminModelCardPage.ts

@@ -301,6 +301,14 @@ export class AdminModelCardPage {
     });
   }
 
+  getAlsoDeleteFolderCheckbox(): Locator {
+    return this.getDeleteConfirmDialog().getByRole('checkbox').first();
+  }
+
+  getFolderNameLinkInDeleteDialog(): Locator {
+    return this.getDeleteConfirmDialog().getByRole('link').first();
+  }
+
   // ── Helper: create via UI and return ─────────────────────────────────────
 
   async createModelCard(fields: {