|
1 | 1 | // spec: e2e/.agent-output/test-plan-admin-model-card.md |
2 | 2 | // section: 5. Delete Model Card |
3 | 3 | import { AdminModelCardPage } from '../utils/classes/AdminModelCardPage'; |
4 | | -import { loginAsAdmin, webuiEndpoint } from '../utils/test-util'; |
| 4 | +import { |
| 5 | + createVFolderAndVerify, |
| 6 | + loginAsAdmin, |
| 7 | + moveToTrashAndVerify, |
| 8 | + webuiEndpoint, |
| 9 | +} from '../utils/test-util'; |
5 | 10 | import { test, expect } from '@playwright/test'; |
6 | 11 |
|
7 | 12 | test.describe( |
@@ -266,5 +271,135 @@ test.describe( |
266 | 271 | `${rowCount} selected`, |
267 | 272 | ); |
268 | 273 | }); |
| 274 | + |
| 275 | + // 5.7 Superadmin can delete a model card and its associated folder together |
| 276 | + test('Superadmin can delete a model card and its associated folder, and navigate to trash with folder filter', async ({ |
| 277 | + page, |
| 278 | + }) => { |
| 279 | + test.setTimeout(90000); |
| 280 | + const adminModelCardPage = new AdminModelCardPage(page); |
| 281 | + const timestamp = Date.now(); |
| 282 | + const folderName = `e2e-test-delete-folder-${timestamp}`; |
| 283 | + const cardName = `e2e-test-delete-with-folder-${timestamp}`; |
| 284 | + |
| 285 | + // Setup: create a dedicated test VFolder to avoid affecting real folders |
| 286 | + await createVFolderAndVerify(page, folderName, 'model'); |
| 287 | + |
| 288 | + // Create a model card linked to the dedicated test folder |
| 289 | + await page.goto(`${webuiEndpoint}/admin-serving?tab=model-store`); |
| 290 | + await adminModelCardPage.waitForTableLoad(); |
| 291 | + await adminModelCardPage.createModelCard({ |
| 292 | + name: cardName, |
| 293 | + vfolderTitle: folderName, |
| 294 | + }); |
| 295 | + |
| 296 | + // Navigate back and filter for the created card |
| 297 | + await page.goto(`${webuiEndpoint}/admin-serving?tab=model-store`); |
| 298 | + await adminModelCardPage.waitForTableLoad(); |
| 299 | + await adminModelCardPage.applyNameFilter(cardName); |
| 300 | + |
| 301 | + // Open the delete confirmation dialog |
| 302 | + await adminModelCardPage.clickDeleteForRow(cardName); |
| 303 | + |
| 304 | + // Verify the "Also delete the associated model folder" checkbox is visible |
| 305 | + await expect( |
| 306 | + adminModelCardPage.getAlsoDeleteFolderCheckbox(), |
| 307 | + ).toBeVisible(); |
| 308 | + |
| 309 | + // Verify the folder link shows the expected folder name |
| 310 | + const folderLink = adminModelCardPage.getFolderNameLinkInDeleteDialog(); |
| 311 | + await expect(folderLink).toBeVisible(); |
| 312 | + await expect(folderLink).toHaveText(folderName); |
| 313 | + |
| 314 | + // Check the "Also delete the associated model folder" checkbox |
| 315 | + await adminModelCardPage.getAlsoDeleteFolderCheckbox().check(); |
| 316 | + await expect( |
| 317 | + adminModelCardPage.getAlsoDeleteFolderCheckbox(), |
| 318 | + ).toBeChecked(); |
| 319 | + |
| 320 | + // Confirm deletion |
| 321 | + await adminModelCardPage.getDeleteConfirmButton().click(); |
| 322 | + |
| 323 | + // Verify the success notification for card + folder deletion |
| 324 | + await expect( |
| 325 | + page.getByText('Model card and folder have been moved to trash.'), |
| 326 | + ).toBeVisible({ timeout: 15000 }); |
| 327 | + |
| 328 | + // Verify "Go to Data > Trash" link is visible in the notification |
| 329 | + const goToTrashLink = page.getByText('Go to Data > Trash'); |
| 330 | + await expect(goToTrashLink).toBeVisible(); |
| 331 | + |
| 332 | + // Click "Go to Data > Trash" and verify URL includes folder filter |
| 333 | + await goToTrashLink.click(); |
| 334 | + await page.waitForURL( |
| 335 | + (url) => |
| 336 | + url.pathname === '/data' && |
| 337 | + url.searchParams.get('statusCategory') === 'deleted' && |
| 338 | + url.searchParams.get('filter') === `name == "${folderName}"`, |
| 339 | + { timeout: 10000 }, |
| 340 | + ); |
| 341 | + }); |
| 342 | + |
| 343 | + // 5.8 Superadmin deletes card only: notification shows correct message and Go to Trash navigates correctly |
| 344 | + test('Superadmin can delete a model card only and navigate to trash without folder filter', async ({ |
| 345 | + page, |
| 346 | + }) => { |
| 347 | + test.setTimeout(90000); |
| 348 | + const adminModelCardPage = new AdminModelCardPage(page); |
| 349 | + const timestamp = Date.now(); |
| 350 | + const folderName = `e2e-test-keep-folder-${timestamp}`; |
| 351 | + const cardName = `e2e-test-delete-card-only-${timestamp}`; |
| 352 | + |
| 353 | + // Setup: create a dedicated test VFolder to avoid affecting real folders |
| 354 | + await createVFolderAndVerify(page, folderName, 'model'); |
| 355 | + |
| 356 | + // Create a model card linked to the dedicated test folder |
| 357 | + await page.goto(`${webuiEndpoint}/admin-serving?tab=model-store`); |
| 358 | + await adminModelCardPage.waitForTableLoad(); |
| 359 | + await adminModelCardPage.createModelCard({ |
| 360 | + name: cardName, |
| 361 | + vfolderTitle: folderName, |
| 362 | + }); |
| 363 | + |
| 364 | + // Navigate back and filter |
| 365 | + await page.goto(`${webuiEndpoint}/admin-serving?tab=model-store`); |
| 366 | + await adminModelCardPage.waitForTableLoad(); |
| 367 | + await adminModelCardPage.applyNameFilter(cardName); |
| 368 | + |
| 369 | + // Open delete dialog |
| 370 | + await adminModelCardPage.clickDeleteForRow(cardName); |
| 371 | + |
| 372 | + // Leave "Also delete folder" checkbox unchecked (default) |
| 373 | + await expect( |
| 374 | + adminModelCardPage.getAlsoDeleteFolderCheckbox(), |
| 375 | + ).not.toBeChecked(); |
| 376 | + |
| 377 | + // Confirm deletion |
| 378 | + await adminModelCardPage.getDeleteConfirmButton().click(); |
| 379 | + |
| 380 | + // Verify the notification message for card-only deletion |
| 381 | + await expect( |
| 382 | + page.getByText( |
| 383 | + 'Model card has been deleted. The model folder was not deleted.', |
| 384 | + ), |
| 385 | + ).toBeVisible({ timeout: 15000 }); |
| 386 | + |
| 387 | + // Verify "Go to Data > Trash" link is visible |
| 388 | + const goToTrashLink = page.getByText('Go to Data > Trash'); |
| 389 | + await expect(goToTrashLink).toBeVisible(); |
| 390 | + |
| 391 | + // Click "Go to Data > Trash" and verify URL (no folder filter) |
| 392 | + await goToTrashLink.click(); |
| 393 | + await page.waitForURL( |
| 394 | + (url) => |
| 395 | + url.pathname === '/data' && |
| 396 | + url.searchParams.get('statusCategory') === 'deleted' && |
| 397 | + !url.searchParams.has('filter'), |
| 398 | + { timeout: 10000 }, |
| 399 | + ); |
| 400 | + |
| 401 | + // Cleanup: move the kept test folder to trash |
| 402 | + await moveToTrashAndVerify(page, folderName); |
| 403 | + }); |
269 | 404 | }, |
270 | 405 | ); |
0 commit comments