From 53bbd7522d8d30e64a8d45f1374f840bc2ddeeda Mon Sep 17 00:00:00 2001
From: "paseo-bot[bot]" <266920839+paseo-bot[bot]@users.noreply.github.com>
Date: Wed, 28 Oct 2026 06:13:41 +0000
Subject: [PATCH] test(server): wait for pending watchers before testing
 disposal (#8091)

Observation admission precedes asynchronous path resolution. Wait for subscriptions to start before disposal so the test consistently exercises late watcher cleanup.

Co-authored-by: paseo-bot[bot] <266920839+paseo-bot[bot]@users.noreply.github.com>
---
 packages/server/src/server/workspace-git-service.test.ts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/packages/server/src/server/workspace-git-service.test.ts b/packages/server/src/server/workspace-git-service.test.ts
index 1d4d365d95..b3fab7b89a 100644
--- a/packages/server/src/server/workspace-git-service.test.ts
+++ b/packages/server/src/server/workspace-git-service.test.ts
@@ -797,6 +797,8 @@ describe("WorkspaceGitServiceImpl", () => {
 
     cwds.forEach((cwd) => service.registerWorkspace({ cwd }, vi.fn()));
     await vi.waitFor(() => {
+      // Admission precedes async path resolution; disposal must race pending subscriptions.
+      expect(subscribe).toHaveBeenCalledTimes(WORKSPACE_GIT_OBSERVATION_SETUP_CONCURRENCY);
       expect(service.getMetrics()).toMatchObject({
         workspaceRefreshAdmissionActiveCount: WORKSPACE_GIT_REFRESH_CONCURRENCY,
         workspaceRefreshAdmissionPendingCount: 1,