From 08498f98287ea79dcecfd8262fc620338003b0fb Mon Sep 17 00:00:00 2001
From: Peter Steinberger <peter@steipete.me>
Date: Sat, 11 Jul 2026 15:54:44 -0700
Subject: [PATCH] ci: pass frozen UI test options to Vitest (#104758)

---
 .github/workflows/ci.yml                | 2 +-
 test/scripts/ci-workflow-guards.test.ts | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 609018ebb0cf..dde0d49320f3 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -1052,7 +1052,7 @@ jobs:
           if [[ "$COMPATIBILITY_TARGET" == "true" ]]; then
             # Frozen targets can contain timing-sensitive tests fixed on current main.
             # Preserve one retry, but let their browser helpers use their 10-second waits.
-            pnpm --dir ui test -- --retry=1 --testTimeout=10000
+            pnpm --dir ui test --retry=1 --testTimeout=10000
           else
             pnpm --dir ui test
           fi
diff --git a/test/scripts/ci-workflow-guards.test.ts b/test/scripts/ci-workflow-guards.test.ts
index bbd67fcda103..22aae9722c27 100644
--- a/test/scripts/ci-workflow-guards.test.ts
+++ b/test/scripts/ci-workflow-guards.test.ts
@@ -1875,7 +1875,8 @@ describe("ci workflow guards", () => {
       "${{ needs.preflight.outputs.compatibility_target }}",
     );
     expect(uiTest.run).toContain('if [[ "$COMPATIBILITY_TARGET" == "true" ]]');
-    expect(uiTest.run).toContain("pnpm --dir ui test -- --retry=1 --testTimeout=10000");
+    expect(uiTest.run).toContain("pnpm --dir ui test --retry=1 --testTimeout=10000");
+    expect(uiTest.run).not.toContain("pnpm --dir ui test -- --retry");
     expect(uiTest.run).toContain("pnpm --dir ui test");
   });