test(FR-2652): fix parallel interference and folder-dialog flakiness in delete spec

[agatha197]

Resolves #6885 (FR-2652)

Stacked on top of #7137.

Summary

Fixes flakiness in e2e/admin-model-card/admin-model-card-delete.spec.ts exposed under load:

• Run delete spec serially. With fullyParallel: true set globally in playwright.config.ts, tests within this file ran in parallel and the server occasionally indexed newly-created cards too slowly to satisfy the name filter (resulting in "No data" assertions). Added test.describe.configure({ mode: 'serial' }) to the describe block so the tests share a deterministic ordering against the same backend.
• Extend post-cancel row visibility timeout (5s → 15s). After closing the bulk-delete confirmation dialog, the table briefly re-renders. The default 5 s toBeVisible was tight enough to flake when the page was busy.
• Make createNewFolderViaPlus robust against the Change Project Popconfirm path. The onConfirm handler in AdminModelCardSettingModal runs inside startTransition and silently no-ops when modelStoreProject data isn't loaded yet. The page object now:
  • Increases the Change Project | folder dialog race timeout (5s → 10s).
  • After clicking Change Project, asserts the Current project changed successfully. message — this fails fast (and clearly) when the data dependency isn't ready, instead of timing out later on the folder dialog.
  • Increases the folder-dialog visibility timeout (15s → 30s) for the slow path where the project change actually triggered a refetch.

Test plan

• pnpm exec playwright test e2e/admin-model-card/admin-model-card-delete.spec.ts — 10/10 passing in ~3.9 minutes
• Re-run under default fullyParallel: true config — no parallel interference within the file

[agatha197]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• test(FR-2652): fix parallel interference and folder-dialog flakiness in delete spec #7162 👈 (View in Graphite)
• test(FR-2622): add E2E tests for bulk vfolder trash on model card bulk deletion #7137
• feat(FR-2622): add bulk vfolder trash support on model card bulk deletion #7135 : 1 other dependent PR (#7250 )
• 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

This PR focuses on stabilizing the Admin Model Card delete E2E suite by eliminating parallel interference under fullyParallel: true and tightening a couple of timing/race conditions around bulk delete and folder-creation flows.

Changes:

• Forces e2e/admin-model-card/admin-model-card-delete.spec.ts to run serially to avoid backend indexing / filter-result races under parallel load.
• Improves delete-flow robustness by switching header checkbox selection to .click() (handles indeterminate state) and waiting for the Delete button to become enabled before clicking.
• Hardens folder creation via the “+” path by explicitly asserting the “Current project changed successfully.” toast before waiting for the folder dialog, and extending relevant timeouts.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File	Description
e2e/utils/classes/AdminModelCardPage.ts	Makes createNewFolderViaPlus resilient to the “Change Project” popconfirm path and slow refetches by asserting the success toast and increasing timeouts.
e2e/admin-model-card/admin-model-card-delete.spec.ts	Runs the suite serially and adjusts interaction/wait patterns to reduce flakiness under load (checkbox selection, enabled-state waits, longer visibility timeouts).
e2e/E2E_COVERAGE_REPORT.md	Updates the report’s “Last Updated” date.

[agatha197]

Closing this PR as its changes have been folded into #7137. Both PRs cover E2E test changes for the same spec, so they've been consolidated into one.