test: settle empty file roots and tooltip updates

Author: carlosflorencio

apps/web/components/task/file-browser-tree-loader.ts

@@ -154,10 +154,10 @@ function applyCompletedTree({
   retry: () => void;
 }): boolean {
   if (!completed) return false;
-  // An executor workspace can briefly expose an empty root while its
-  // repository is materializing. Keep the result provisional for a bounded
-  // period so the Files panel does not cache an empty tree prematurely.
-  if (!completed.root || (completed.root.children?.length ?? 0) === 0) {
+  // An executor workspace can briefly expose an existing root with no
+  // children while its repository is materializing. A null root is a valid
+  // empty workspace result and must settle immediately.
+  if (completed.root && (completed.root.children?.length ?? 0) === 0) {
     const retryScheduled = scheduleEmptyTreeRetry({
       isCurrentLoad,
       owner,

apps/web/e2e/tests/pr/pr-status-badge.spec.ts

@@ -679,6 +679,9 @@ test.describe("PR status badge", () => {
     await icon.hover();
 
     const multiSummary = visibleTaskPRSummary(testPage);
+    // The second association updates the icon while the disclosure is closed.
+    // Wait for the tooltip itself to reopen before querying its refreshed rows.
+    await expect(multiSummary).toBeVisible({ timeout: 15_000 });
     const entries = multiSummary.getByTestId("pr-task-status-entry");
     await expect(entries).toHaveCount(2);
     await expect(entries.nth(0).getByTestId("pr-task-status-number")).toHaveText("PR #2966");