e2e(FR-2472): align route table test with current BAIRouteNodes columns

- Replace 'Traffic Ratio' column header expectations with 'Created At'
  (TrafficRatio column is currently commented out in BAIRouteNodes.tsx)
- Mark test 4.6 / 7.2 as test.fixme with TODO(needs-backend)
- Skip accelerator spinbutton edit when it is already disabled at 0

Author: ironAiken2

e2e/E2E_COVERAGE_REPORT.md

@@ -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): 253 / 410 features covered (62%)**
+**Overall (in-scope routes): 228 / 372 features covered (61%)**
 
 | Page | Route | Features | Covered | Status |
 |------|-------|:--------:|:-------:|:------:|
@@ -22,9 +22,9 @@
 | Dashboard | `/dashboard` | 9 | 7 | 🔶 78% |
 | Session List | `/session` | 22 | 14 | 🔶 64% |
 | Session Launcher | `/session/start` | 14 | 3 | 🔶 21% |
-| Serving | `/serving` | 7 | 0 | ❌ 0% |
+| Serving | `/serving` | 7 | 2 | 🔶 29% |
 | Endpoint Detail | `/serving/:serviceId` | 20 | 9 | 🔶 45% |
-| Service Launcher | `/service/start` | 5 | 0 | ❌ 0% |
+| Service Launcher | `/service/start` | 5 | 1 | 🔶 20% |
 | VFolder / Data | `/data` | 45 | 32 | 🔶 71% |
 | Model Store | `/model-store` | 6 | 6 | ✅ 100% |
 | Admin Model Store | `/admin-model-store` | 22 | 22 | ✅ 100% |
@@ -46,7 +46,7 @@
 | App Launcher | (modal) | 18 | 10 | 🔶 56% |
 | Chat | `/chat/:id?` | 6 | 6 | ✅ 100% |
 | Plugin System | (config-based) | 12 | 12 | ✅ 100% |
-| **Total** | | **357** | **203** | **57%** |
+| **Total** | | **372** | **228** | **61%** |
 
 ---
 
@@ -223,7 +223,7 @@
 
 ### 6. Serving / Model Service (`/serving`)
 
-**Test files:** None (visual regression only: [`e2e/visual_regression/serving/serving_page.test.ts`](visual_regression/serving/serving_page.test.ts))
+**Test files:** [`e2e/serving/serving-deploy-lifecycle.spec.ts`](serving/serving-deploy-lifecycle.spec.ts) (integration, `@integration @serving`)
 
 **Filter:** Active | Destroyed (radio)
 **Primary action:** "Start Service" → navigates to `/service/start`
@@ -232,15 +232,15 @@
 
 | Feature | Status | Test |
 | --------------------------------------------------------- | ------ | ---- |
-| Endpoint list rendering | ❌ | - |
+| Endpoint list rendering | ✅ | `Admin can deploy a model service via ServiceLauncher UI` (verifies row visible in serving list) |
 | "Start Service" → navigate to `/service/start` | ❌ | - |
 | Endpoint name click → EndpointDetailPage | ❌ | - |
 | Status filtering (Active/Destroyed) | ❌ | - |
 | Property filtering | ❌ | - |
 | Edit endpoint → navigate to `/service/update/:endpointId` | ❌ | - |
-| Delete endpoint → confirm dialog | ❌ | - |
+| Delete endpoint → confirm dialog | ✅ | `Admin can terminate a deployed service` |
 
-**Coverage: ❌ 0/7 features**
+**Coverage: 🔶 2/7 features**
 
 ---
 
@@ -281,17 +281,17 @@
 
 ### 8. Service Launcher (`/service/start`, `/service/update/:endpointId`)
 
-**Test files:** None
+**Test files:** [`e2e/serving/serving-deploy-lifecycle.spec.ts`](serving/serving-deploy-lifecycle.spec.ts) (integration, `@integration @serving`)
 
 | Feature | Status | Test |
 | ----------------------- | ------ | ---- |
-| Create model service | ❌ | - |
+| Create model service | ✅ | `Admin can deploy a model service via ServiceLauncher UI` |
 | Update existing service | ❌ | - |
 | Resource configuration | ❌ | - |
 | Model folder selection | ❌ | - |
 | Form validation | ❌ | - |
 
-**Coverage: ❌ 0/5 features**
+**Coverage: 🔶 1/5 features**
 
 ---
 

e2e/serving/endpoint-route-table.spec.ts

@@ -141,7 +141,7 @@ test.describe(
         card.getByRole('columnheader', { name: 'Traffic Status' }),
       ).toBeVisible();
       await expect(
-        card.getByRole('columnheader', { name: 'Traffic Ratio' }),
+        card.getByRole('columnheader', { name: 'Created At' }),
       ).toBeVisible();
     });
 
@@ -455,7 +455,7 @@ test.describe(
         card.getByRole('columnheader', { name: 'Traffic Status' }),
       ).toBeVisible();
       await expect(
-        card.getByRole('columnheader', { name: 'Traffic Ratio' }),
+        card.getByRole('columnheader', { name: 'Created At' }),
       ).toBeVisible();
     });
 
@@ -522,7 +522,10 @@ test.describe(
       await expect(inactiveTags.first()).toBeVisible();
     });
 
-    test('4.6 Admin sees the traffic ratio value in the Traffic Ratio column', async ({
+    // TODO(needs-backend): Re-enable when BAIRouteNodes exposes the Traffic Ratio
+    // column. It is currently commented out in BAIRouteNodes.tsx pending backend
+    // support for per-route traffic ratio.
+    test.fixme('4.6 Admin sees the traffic ratio value in the Traffic Ratio column', async ({
       page,
       request,
     }) => {
@@ -720,21 +723,22 @@ test.describe(
       ).toBeVisible();
     });
 
-    test('7.2 Admin can sort routes by Traffic Ratio column', async ({
+    // TODO(needs-backend): Re-enable when BAIRouteNodes exposes the Traffic Ratio
+    // column. It is currently commented out in BAIRouteNodes.tsx pending backend
+    // support for per-route traffic ratio.
+    test.fixme('7.2 Admin can sort routes by Traffic Ratio column', async ({
       page,
       request,
     }) => {
       await setupAndNavigateToDetail(page, request);
 
       const card = getRoutesInfoCard(page);
 
-      // Click the "Traffic Ratio" column header to sort
       const trafficRatioHeader = card.getByRole('columnheader', {
         name: 'Traffic Ratio',
       });
       await trafficRatioHeader.click();
 
-      // Verify a sort indicator is shown
       await expect(
         trafficRatioHeader.locator('.ant-table-column-sorter'),
       ).toBeVisible();

e2e/serving/fixtures/model-definition.yaml

@@ -8,5 +8,4 @@ models:
       port: 8000
       health_check:
         path: /health
-        initial_delay: 5.0
         max_retries: 10

e2e/serving/model-card-drawer.spec.ts

@@ -25,6 +25,38 @@ import { test, expect } from '@playwright/test';
 // Shared helpers
 // ─────────────────────────────────────────────────────────────────────────────
 
+/**
+ * Intercepts the REST endpoints used by `useProjectResourceGroups` so tests
+ * can control which resource groups appear in the Deploy modal selector.
+ * The hook calls `/scaling-groups?group=...` and `/folders/_/hosts` via the
+ * Backend.AI client and is shared by `BAIProjectResourceGroupSelect`.
+ */
+async function setupResourceGroupsRestMock(
+  page: any,
+  resourceGroupNames: ReadonlyArray<string>,
+) {
+  await page.route('**/func/scaling-groups**', async (route: any) => {
+    await route.fulfill({
+      status: 200,
+      contentType: 'application/json',
+      body: JSON.stringify({
+        scaling_groups: resourceGroupNames.map((name) => ({ name })),
+      }),
+    });
+  });
+  await page.route('**/func/folders/_/hosts', async (route: any) => {
+    await route.fulfill({
+      status: 200,
+      contentType: 'application/json',
+      body: JSON.stringify({
+        allowed: [],
+        default: '',
+        volume_info: {},
+      }),
+    });
+  });
+}
+
 /**
  * Shared setup: login, navigate to serving (establishes backendaiclient),
  * inject the model-card-v2 feature flag, then set up GraphQL mocks before
@@ -34,9 +66,15 @@ async function setupModelStorePage(
   page: any,
   request: any,
   mocks: Record<string, (vars: Record<string, any>) => Record<string, any>>,
+  resourceGroupNames: ReadonlyArray<string> = ['default'],
 ) {
   await loginAsAdmin(page, request);
 
+  // Mock the REST endpoints that feed `useProjectResourceGroups` before
+  // anything navigates — the Deploy modal reads resource groups from REST,
+  // not GraphQL, so the GraphQL `scaling_groups` field is not enough.
+  await setupResourceGroupsRestMock(page, resourceGroupNames);
+
   // Set up GraphQL mocks BEFORE any navigation that triggers GQL queries.
   // Mock ServingPageQuery as well to prevent real API calls on the serving page.
   await setupGraphQLMocks(page, {
@@ -288,11 +326,16 @@ test.describe(
     test.describe.configure({ mode: 'serial' });
 
     test.beforeEach(async ({ page, request }) => {
-      await setupModelStorePage(page, request, {
-        ModelStoreListPageV2Query: modelStoreListWithMultiPresetsMock(),
-        ModelCardDeployModalQuery: modelCardDeployModalQueryMock(),
-        ModelCardDeployModalMutation: modelCardDeployModalMutationMock(),
-      });
+      await setupModelStorePage(
+        page,
+        request,
+        {
+          ModelStoreListPageV2Query: modelStoreListWithMultiPresetsMock(),
+          ModelCardDeployModalQuery: modelCardDeployModalQueryMock(),
+          ModelCardDeployModalMutation: modelCardDeployModalMutationMock(),
+        },
+        ['default', 'gpu-cluster'],
+      );
     });
 
     test('admin can open the Deploy Model modal by clicking Deploy in the drawer', async ({

e2e/serving/serving-deploy-lifecycle.spec.ts

@@ -191,6 +191,37 @@ async function createServiceViaUI(
     .first()
     .click({ timeout: 10000 });
 
+  // Set AI Accelerator to 0 to avoid GPU-based allocation presets.
+  // When the resource group has a GPU preset selected by default (e.g. cuda01-small),
+  // service creation would fail if no GPU agents are available.
+  // Setting the accelerator count to 0 ensures CPU-only resource allocation.
+  //
+  // Strategy: Find the AI Accelerator form item by its label text, then target
+  // the spinbutton inside it. Ant Design Form.Item uses a `label` element that
+  // may not have a `for` attribute in all versions, so we use a compound selector.
+  const acceleratorFormItem = page
+    .locator('.ant-form-item')
+    .filter({ hasText: 'AI Accelerator' })
+    .first();
+  const acceleratorSpinbutton = acceleratorFormItem.getByRole('spinbutton');
+  if (
+    await acceleratorSpinbutton.isVisible({ timeout: 5000 }).catch(() => false)
+  ) {
+    // If the spinbutton is already disabled (e.g., no GPU presets available in
+    // the selected resource group), it is already at 0 — skip editing it.
+    const isEditable = await acceleratorSpinbutton
+      .isEditable({ timeout: 1000 })
+      .catch(() => false);
+    if (isEditable) {
+      await acceleratorSpinbutton.scrollIntoViewIfNeeded();
+      await acceleratorSpinbutton.click({ clickCount: 3 });
+      await acceleratorSpinbutton.fill('0');
+      await acceleratorSpinbutton.press('Tab');
+      // Wait for the field to reflect the updated CPU-only allocation value
+      await expect(acceleratorSpinbutton).toHaveValue('0', { timeout: 5000 });
+    }
+  }
+
   // Check "Open To Public"
   const openToPublicCheckbox = page.getByLabel('Open To Public');
   await openToPublicCheckbox.scrollIntoViewIfNeeded();
@@ -205,8 +236,10 @@ async function createServiceViaUI(
   await expect(createButton).toBeEnabled({ timeout: 5000 });
   await createButton.click();
 
-  // Wait for redirect to serving page and verify the service appears
-  await page.waitForURL('**/serving', { timeout: 15000 });
+  // Wait for the service creation to complete.
+  // The form navigates to /serving on success or stays on /service/start on error.
+  await page.waitForURL('**/serving', { timeout: 30000 });
+
   await expect(
     page.getByRole('row').filter({ hasText: serviceName }).first(),
   ).toBeVisible({ timeout: 15000 });