fix(e2e): add ClickHouse polling to userCount metrics test

The /internal/metrics endpoint was migrated to read from ClickHouse,
which has eventual consistency. The test was immediately asserting
total_users after signup and deletion without waiting for ClickHouse
to sync.

CI evidence: Freestyle mock run showed 'expected 1 to be +0' at
projects.test.ts:1597 — the user delete hadn't propagated to
ClickHouse yet.

Fix: Poll with 500ms intervals (30s timeout) for both increment
and decrement assertions.

Co-Authored-By: Konstantin Wohlwend <n2d4xc@gmail.com>

Author: devin-ai-integration[bot]

apps/e2e/tests/backend/endpoints/api/v1/projects.test.ts

@@ -1,6 +1,8 @@
 import { isBase64Url } from "@stackframe/stack-shared/dist/utils/bytes";
+import { wait } from "@stackframe/stack-shared/dist/utils/promises";
 import { it } from "../../../../helpers";
 import { Auth, InternalApiKey, InternalProjectKeys, Project, backendContext, niceBackendFetch } from "../../../backend-helpers";
+import { performance } from "perf_hooks";
 
 
 // TODO some of the tests here test /api/v1/projects/current, the others test /api/v1/internal/projects/current. We should split them into different test files
@@ -1579,10 +1581,20 @@ it("should increment and decrement userCount when a user is added to a project",
   // Create a new user in the project
   await Auth.Password.signUpWithEmail();
 
-  // Check that the userCount has been incremented
-  const updatedProjectResponse = await niceBackendFetch("/api/v1/internal/metrics", { accessType: "admin" });
-  expect(updatedProjectResponse.status).toBe(200);
-  expect(updatedProjectResponse.body.total_users).toBe(1);
+  // The metrics endpoint reads from ClickHouse (eventual consistency).
+  // Poll until the new user is visible.
+  const incrementStart = performance.now();
+  while (true) {
+    const updatedProjectResponse = await niceBackendFetch("/api/v1/internal/metrics", { accessType: "admin" });
+    expect(updatedProjectResponse.status).toBe(200);
+    if (updatedProjectResponse.body.total_users === 1) {
+      break;
+    }
+    if (performance.now() - incrementStart > 30_000) {
+      expect(updatedProjectResponse.body.total_users).toBe(1);
+    }
+    await wait(500);
+  }
 
   // Delete the user
   const deleteRes = await niceBackendFetch("/api/v1/users/me", {
@@ -1591,10 +1603,20 @@ it("should increment and decrement userCount when a user is added to a project",
   });
   expect(deleteRes.status).toBe(200);
 
-  // Check that the userCount has been decremented
-  const finalProjectResponse = await niceBackendFetch("/api/v1/internal/metrics", { accessType: "admin" });
-  expect(finalProjectResponse.status).toBe(200);
-  expect(finalProjectResponse.body.total_users).toBe(0);
+  // The metrics endpoint now reads from ClickHouse, which has eventual
+  // consistency. Poll until the delete has propagated.
+  const startedAt = performance.now();
+  while (true) {
+    const finalProjectResponse = await niceBackendFetch("/api/v1/internal/metrics", { accessType: "admin" });
+    expect(finalProjectResponse.status).toBe(200);
+    if (finalProjectResponse.body.total_users === 0) {
+      break;
+    }
+    if (performance.now() - startedAt > 30_000) {
+      expect(finalProjectResponse.body.total_users).toBe(0);
+    }
+    await wait(500);
+  }
 
 });