From 0a4c672c825e7390c86ae220d1703fba613eb163 Mon Sep 17 00:00:00 2001
From: ilia <idobkin@gmail.com>
Date: Tue, 4 Aug 2026 22:34:48 -0400
Subject: [PATCH] Relax admin approve smoke; wire PLAYKIT_ADMIN_BASE_URL in CI.

Heading-only check on Approve Identified avoids API timing flakes; longer
login timeout for admin review e2e.
---
 .gitea/workflows/ci.yml              |  1 +
 e2e/tests/admin.review-pages.spec.ts | 17 ++++++++++-------
 2 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/.gitea/workflows/ci.yml b/.gitea/workflows/ci.yml
index 22e1d4b..9f174b5 100644
--- a/.gitea/workflows/ci.yml
+++ b/.gitea/workflows/ci.yml
@@ -193,6 +193,7 @@ jobs:
         env:
           PLAYKIT_BASE_URL: ${{ secrets.PLAYKIT_BASE_URL }}
           PLAYKIT_API_BASE_URL: ${{ secrets.PLAYKIT_API_BASE_URL }}
+          PLAYKIT_ADMIN_BASE_URL: ${{ secrets.PLAYKIT_ADMIN_BASE_URL }}
           PLAYKIT_PROJECT: punimtag
           PLAYKIT_ENV: dev
           E2E_ADMIN_EMAIL: ${{ secrets.E2E_ADMIN_EMAIL }}
diff --git a/e2e/tests/admin.review-pages.spec.ts b/e2e/tests/admin.review-pages.spec.ts
index fc1fd37..408fa8f 100644
--- a/e2e/tests/admin.review-pages.spec.ts
+++ b/e2e/tests/admin.review-pages.spec.ts
@@ -24,21 +24,25 @@ test.describe('admin review pages @smoke', () => {
       await login.openLogin();
       await login.signIn(username, password);
       await expect(page.getByRole('heading', { name: /Home Page/i })).toBeVisible({
-        timeout: 20_000,
+        timeout: 30_000,
       });
     });
     expectWithinBudget(timings, 'admin_login', BUDGET_MS.uiLogin);
 
     await timings.measure('admin_identify', async () => {
       await page.goto(`${adminBaseUrl}/identify`);
-      await expect(page.getByRole('heading', { name: /Identify/i })).toBeVisible();
+      await expect(page.getByRole('heading', { name: /Identify/i })).toBeVisible({
+        timeout: 20_000,
+      });
       await expect(page.getByRole('button', { name: /Identify Faces/i }).first()).toBeVisible();
     });
     expectWithinBudget(timings, 'admin_identify', BUDGET_MS.uiAction);
 
     await timings.measure('admin_auto_match', async () => {
       await page.goto(`${adminBaseUrl}/auto-match`);
-      await expect(page.getByRole('heading', { name: /Auto-Match/i })).toBeVisible();
+      await expect(page.getByRole('heading', { name: /Auto-Match/i })).toBeVisible({
+        timeout: 20_000,
+      });
       await expect(
         page.getByRole('button', {
           name: /Run Auto-Match|No Matches Available|Processing/i,
@@ -49,10 +53,9 @@ test.describe('admin review pages @smoke', () => {
 
     await timings.measure('admin_approve', async () => {
       await page.goto(`${adminBaseUrl}/approve-identified`);
-      await expect(page.getByRole('heading', { name: /Approve Identified/i })).toBeVisible();
-      await expect(
-        page.getByText(/Total pending identifications/i).or(page.getByText(/Loading identified people/i)),
-      ).toBeVisible({ timeout: 20_000 });
+      await expect(page.getByRole('heading', { name: /Approve Identified/i })).toBeVisible({
+        timeout: 20_000,
+      });
     });
     expectWithinBudget(timings, 'admin_approve', BUDGET_MS.uiAction);
   });