• wwwillchen-bot's avatar
    Fix flaky e2e tests with better selectors and waits (#2498) · 90f5ca33
    wwwillchen-bot 提交于
    ## Summary
    - Fix flaky e2e tests across 6 test files by adding proper awaits,
    better CSS class matching, accordion state checks, and explicit waits
    for UI elements
    - Key fixes: missing `await` on expect in delete_app, regex anchoring
    for favorite_app class matching, accordion expansion guards in
    git_collaboration, `pressSequentially` for mention trigger detection,
    and visibility waits in supabase_migrations
    
    ## Test plan
    - Run the full e2e test suite to verify the fixed tests pass reliably
    - Specifically verify: `delete_app`, `favorite_app`, `file_tree_search`,
    `git_collaboration`, `mention_files`, and `supabase_migrations` tests
    
    🤖 Generated with [Claude Code](https://claude.com/claude-code)
    <!-- devin-review-badge-begin -->
    
    ---
    
    <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2498">
      <picture>
    <source media="(prefers-color-scheme: dark)"
    srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1">
    <img
    src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1"
    alt="Open with Devin">
      </picture>
    </a>
    <!-- devin-review-badge-end -->
    
    <!-- This is an auto-generated description by cubic. -->
    ---
    ## Summary by cubic
    Stabilized flaky e2e tests by improving selectors and adding explicit
    waits. Reliability is improved across delete_app, favorite_app,
    file_tree_search, git_collaboration, mention_files, and
    supabase_migrations.
    
    - **Bug Fixes**
      - Added missing await for visibility assertion in delete_app.
      - Anchored favorite star class regex to avoid matching hover:fill.
      - Updated file_tree_search to expect main.tsx.
    - Guarded Git Collaboration flows by checking accordion expansion and
    element visibility; standardized on Timeout.MEDIUM.
    - Used pressSequentially and waits for @ mention menu; verified Supabase
    migrations switch with toBeChecked() after visibility.
    
    <sup>Written for commit a28a3d981e049c023f3a5343ae57b813961a6680.
    Summary will update on new commits.</sup>
    
    <!-- End of auto-generated description by cubic. -->
    
    ---------
    Co-authored-by: 's avatarWill Chen <willchen90@gmail.com>
    Co-authored-by: 's avatarClaude Opus 4.5 <noreply@anthropic.com>
    90f5ca33
名称
最后提交
最后更新
.claude 正在载入提交数据...
.cursor/rules 正在载入提交数据...
.devcontainer 正在载入提交数据...
.github 正在载入提交数据...
.husky 正在载入提交数据...
assets 正在载入提交数据...
docs 正在载入提交数据...
drizzle 正在载入提交数据...
e2e-tests 正在载入提交数据...
makers 正在载入提交数据...
packages/@dyad-sh 正在载入提交数据...
scaffold 正在载入提交数据...
scripts 正在载入提交数据...
shared 正在载入提交数据...
src 正在载入提交数据...
testing 正在载入提交数据...
tools 正在载入提交数据...
worker 正在载入提交数据...
workers/tsc 正在载入提交数据...
.cursorignore 正在载入提交数据...
.env.example 正在载入提交数据...
.eslintrc.json 正在载入提交数据...
.gitattributes 正在载入提交数据...
.gitignore 正在载入提交数据...
.oxfmtrc.json 正在载入提交数据...
.oxlintrc.json 正在载入提交数据...
.prettierignore 正在载入提交数据...
.prettierrc 正在载入提交数据...
AGENTS.md 正在载入提交数据...
CLA.md 正在载入提交数据...
CLAUDE.md 正在载入提交数据...
CONTRIBUTING.md 正在载入提交数据...
LICENSE 正在载入提交数据...
README.md 正在载入提交数据...
SECURITY.md 正在载入提交数据...
biome.json 正在载入提交数据...
components.json 正在载入提交数据...
drizzle.config.ts 正在载入提交数据...
forge.config.ts 正在载入提交数据...
forge.env.d.ts 正在载入提交数据...
index.html 正在载入提交数据...
lint-staged.config.js 正在载入提交数据...
merge.config.ts 正在载入提交数据...
package-lock.json 正在载入提交数据...
package.json 正在载入提交数据...
playwright.config.ts 正在载入提交数据...
tsconfig.app.json 正在载入提交数据...
tsconfig.json 正在载入提交数据...
tsconfig.node.json 正在载入提交数据...
vite.main.config.mts 正在载入提交数据...
vite.preload.config.mts 正在载入提交数据...
vite.renderer.config.mts 正在载入提交数据...
vite.worker.config.mts 正在载入提交数据...
vitest.config.ts 正在载入提交数据...
windowsSign.ts 正在载入提交数据...