test(trace): Wait for list load without ambiguous findByText

during search, expanding a row retriggers search used within(container).findByText(/transaction-op-0/i), which throws when multiple nodes match the same op label (e.g. virtualized rows plus other UI). Match keyboardNavigationTestSetup by scoping to the virtualized list and using findAllByText for transaction-op- rows.

Fixes CI failure in Jest (1) for trace.spec.

Fixes BROWSE-411.

Made-with: Cursor

Author: JoshuaKGoldberg

static/app/views/performance/newTraceDetails/trace.spec.tsx

@@ -1798,12 +1798,14 @@ describe('trace view', () => {
         }
       );
 
-      const {container} = render(<TraceView />, {
+      render(<TraceView />, {
         initialRouterConfig,
       });
 
-      // Awaits for the placeholder rendering rows to be removed
-      await within(container).findByText(/transaction-op-0/i);
+      const virtualizedContainer = getVirtualizedContainer();
+      await within(virtualizedContainer).findAllByText(/transaction-op-/i, undefined, {
+        timeout: 5000,
+      });
 
       const searchInput = await screen.findByPlaceholderText('Search in trace');
       await userEvent.type(searchInput, 'op-0');