Skip to content

Conversation

gakigaki
Copy link
Contributor

@gakigaki gakigaki commented Sep 8, 2025

概要

E2E(Laravel Dusk)のフレーク低減のため、テスト内の pause()waitFor* に置換し、DOMの描画やダイアログ表示を待ってから操作するよう統一しました。

  • テストの安定化(今回の対象)
    • Manage: Page / User / Theme / Log / API / Code / Reservation で任意待機を排除し、waitFor / waitForText / waitForLocation を追加
    • Common: LoginLock / PasswordPage で失敗メッセージや遷移の待機を追加
    • Top: スライドショーで .carousel の描画待ち
    • Wysiwyg: TinyMCEエディタ・ドロップダウン・ダイアログの描画待ち(.tox-tinymce / .tox-collection / .tox-dialog
  • 既定待機秒数は 10s(前PRで導入済み)を活用し、個別の同期点を明示

本PRは、前回PR(#2258)での基盤対応に続く「テストコード側の待機標準化(第2弾)」です。CI設定やドライバ起動整理は含みません。

レビュー完了希望日

CIの安定運用のため、可能であれば今週中のレビューをお願いできると助かります。(急ぎではありません)

関連Pull requests/Issues

参考

DB変更の有無

無し

チェックリスト

…waits for dropdowns in AdminLink/LoginLogout; increase default waitSeconds already applied
…anage flows; wait for selectors/text before interactions
…ge flows; wait for forms, panels, and location/text
@gakigaki gakigaki changed the title test(dusk): pause()置換の第2弾(waitFor系で同期を明示) ci(dusk): 安定化対応(pause()のwaitFor化・同期待機の明示) Sep 8, 2025
@gakigaki gakigaki changed the title ci(dusk): 安定化対応(pause()のwaitFor化・同期待機の明示) [E2Eテスト] 安定化対応(pause()のwaitFor化・同期待機の明示) Sep 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant