test(FR-2622): add E2E tests for admin model card management

[agatha197]

Resolves #7249 (FR-2622)

Summary

• Full admin model card management page E2E test suite (8 spec files): page-load, create, edit, delete, filter, sort-refresh, url-state, access-control
• New delete tests from FR-2652: single-delete with/without folder trash, bulk delete with folder trash option (tests 5.7–5.10)
• URL migration throughout: /admin-serving?tab=model-store → /admin-deployments?tab=model-store-management
• POM additions: getAlsoDeleteFolderCheckbox, getFolderNameLinkInDeleteDialog, getAlsoDeleteFoldersBulkCheckbox, getBulkDeleteButton, createNewFolderViaPlus

Flakiness fixes

• createNewFolderViaPlus: retry loop (up to 5 attempts) with short-timeout click on "Change Project" button to fail fast when folder dialog overlay blocks; breaks immediately after successful click so the change-project state update can propagate
• deleteModelCardByName: waits for Delete button to be enabled before clicking — under parallel worker load, React's onChange may not fire synchronously after fill()
• Filter test 10.1: uses testCardName created in beforeEach instead of reading the first table row, avoiding interference from concurrent delete tests emptying the table
• Delete tests: run in serial mode to avoid parallel indexing race where newly created cards are not yet searchable under load

Test results

36 passed, 1 skipped (pagination test requires >1 page of data — expected skip)

[agatha197]

• test(FR-2622): add E2E tests for admin model card management #7250 👈 (View in Graphite)
• feat(FR-2622): add bulk vfolder trash support on model card bulk deletion #7135 : 1 other dependent PR (#7137 )
• fix(FR-2651): list all model-store folders when creating a model card #6888 : 1 other dependent PR (#6887 )
• chore(FR-2622): add i18n translations for delete-folder option #6883
• fix(FR-2622): add delete-folder option and trash notification on model card deletion #6881
• main

---

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

• flow:merge-queue - adds this PR to the back of the merge queue
• flow:hotfix - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

[Copilot]

Pull request overview

Adds a comprehensive Playwright E2E test suite for the Admin Model Card Management flow, updating navigation to the new Admin Deployments tab route and extending the test/POM utilities to support folder-related delete scenarios and flakiness handling.

Changes:

• Added/expanded admin model card management E2E coverage (create/edit/delete/filter/sort/URL-state/access-control), including new “delete card + folder” scenarios.
• Migrated test navigation from /admin-serving?tab=model-store to /admin-deployments?tab=model-store-management.
• Enhanced E2E utilities/POM to handle folder creation via “+”, delete-confirm dialog controls, and admin-data vfolder cleanup; updated E2E coverage report accordingly.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
e2e/utils/test-util.ts	Extended vfolder verification/trash/purge helpers to support alternate data routes (e.g., admin-data).
e2e/utils/classes/AdminModelCardPage.ts	Updated page URL to new route; added helpers for folder creation via “+” and richer delete/bulk-delete dialog interactions.
e2e/E2E_COVERAGE_REPORT.md	Updated coverage counts/date and marked new Admin Model Store features/tests as covered; reformatted multiple tables.
e2e/admin-model-card/admin-model-card-access-control.spec.ts	Updated access-control test to hit the new admin deployments tab URL.
e2e/admin-model-card/admin-model-card-create.spec.ts	Updated create tests/selectors for antd v6 behavior and new route; added stability waits.
e2e/admin-model-card/admin-model-card-delete.spec.ts	Expanded delete coverage (single/bulk + folder-trash options), added serial execution, and integrated vfolder cleanup via shared helpers.
e2e/admin-model-card/admin-model-card-edit.spec.ts	Updated edit flow to new route and stabilized selectors for antd v6 tooltips/selects.
e2e/admin-model-card/admin-model-card-filter.spec.ts	Updated filter tests to new route.
e2e/admin-model-card/admin-model-card-page-load.spec.ts	Updated page-load checks to new route; ensured deterministic table data by creating a card in-test.
e2e/admin-model-card/admin-model-card-sort-refresh.spec.ts	Updated refresh/sort tests to new route; created per-test data and added cleanup.
e2e/admin-model-card/admin-model-card-url-state.spec.ts	Updated URL-state tests to new route; made filter test deterministic by creating a card in beforeEach.

[ironAiken2]

Running 1 test using 1 worker

  ✘  1 [chromium] › e2e/admin-model-card/admin-model-card-delete.spec.ts:581:9 › Admin Model Card Management - Delete › Superadmin can bulk delete model cards and move their associated folders to trash @admin-model-card @admin @crud (4.3m)


  1) [chromium] › e2e/admin-model-card/admin-model-card-delete.spec.ts:581:9 › Admin Model Card Management - Delete › Superadmin can bulk delete model cards and move their associated folders to trash @admin-model-card @admin @crud 

    Error: expect(locator).toBeVisible() failed

    Locator: getByText(/model card\(s\) and their associated folder\(s\) have been moved to trash/i)
    Expected: visible
    Timeout: 240000ms
    Error: element(s) not found

    Call log:
      - Expect "toBeVisible" with timeout 240000ms
      - waiting for getByText(/model card\(s\) and their associated folder\(s\) have been moved to trash/i)


      663 |           /model card\(s\) and their associated folder\(s\) have been moved to trash/i,
      664 |         ),
    > 665 |       ).toBeVisible({ timeout: 240000 });
          |         ^
      666 |
      667 |       // Dialog must be hidden once the notification is shown
      668 |       await expect(bulkDialog).toBeHidden();
        at /Users/sungchulhong/Desktop/github/backend.ai-webui2/e2e/admin-model-card/admin-model-card-delete.spec.ts:665:9

    Error Context: test-results/admin-model-card-admin-mod-9b95f-associated-folders-to-trash-chromium/error-context.md

  1 failed
    [chromium] › e2e/admin-model-card/admin-model-card-delete.spec.ts:581:9 › Admin Model Card Management - Delete › Superadmin can bulk delete model cards and move their associated folders to trash @admin-model-card @admin @crud 

I got some errors.

[ironAiken2]

Furthermore, in scenarios where the vfolder associated with a model card is deleted, I would like the logic to permanently delete the folder once it has been moved to the Recycle Bin. This is to keep the test server clean.

[ironAiken2]

LGTM

[graphite-app]

Merge activity

• May 11, 7:36 AM UTC: ironAiken2 added this pull request to the Graphite merge queue.
• May 11, 7:39 AM UTC: Merged by the Graphite merge queue.