From 529ac12ba5850fc097877dfd1e9d56d503fdbed0 Mon Sep 17 00:00:00 2001 From: Yuneng Jiang Date: Tue, 1 Sep 2026 13:01:46 -0700 Subject: [PATCH] test(ui): drop the helper docblock The repo does not take explanatory comments. The reason the helper queries by role lives in the commit that introduced it and in the PR description. --- ui/litellm-dashboard/tests/test-utils.tsx | 9 --------- 1 file changed, 9 deletions(-) diff --git a/ui/litellm-dashboard/tests/test-utils.tsx b/ui/litellm-dashboard/tests/test-utils.tsx index 31af51f3f32..553726faff0 100644 --- a/ui/litellm-dashboard/tests/test-utils.tsx +++ b/ui/litellm-dashboard/tests/test-utils.tsx @@ -44,15 +44,6 @@ const pointerBlocked = (element: HTMLElement): boolean => { return false; }; -/** - * Opens a Base UI popup and picks an entry by its accessible name. - * - * Querying the entry by text or by a title attribute matches the moment the node exists, which is - * one render before the popup finishes entering. Until then the positioner still carries - * `pointer-events: none` and user-event refuses to click, so that shape is a race a fast machine - * loses. The role query only matches once the popup is open to the accessibility tree, which is - * what makes this wait correct rather than lucky. - */ export const chooseSelectOption = async ( user: Pick, "click">, trigger: HTMLElement,