#647 [test] GUI の残り 14 関数を状態待機 helper に統一する

[Kewton]

概要

tests/gui_server.rs に残っていた、状態を待つ独自のループを、#621 の状態待機 helper（tests/support/gui_wait.rs）に統一します。対象は 14 関数・23 箇所です。

変更

• tests/gui_server.rs（+230 −166）だけを変えました。製品コードと gui_wait.rs は変えていません。
• 元の 17 関数・30 箇所について、J と K の対応表を commit message とファイル冒頭のコメントに書きました。漏れはありません。
  • J（[test] GUI の状態待機 helper を整え停止の 3 テストへ適用する #621）：3 関数・7 箇所
  • K（[test] GUI の残り 14 関数を状態待機 helper に統一する #647）：14 関数・23 箇所
• 同時実行で見つかった競合を直すため、3 箇所に待ちを足しました：recovery_auto_run、failure_explanation.projection_status、gate。
  • 3 箇所とも、待った後に 1 回だけサーバーの状態を読んでいました。これを、後の assertion と同じ値になるまで gui_wait::wait_for で待つ形にしました（上限は SESSION_STATE_TIMEOUT）。
  • 待った後の assertion はそのままです。成功するまでの再実行、上限の延長、固定の sleep は足していません。
• 既存の内容の assertion（auth・event bytes・hash・generation・recovery rejection）は 1 つも消していません。
  • 消えたのは待ちのループの中の判定だけです。これらは helper の有限の上限と、Failed の扱いに置き換わりました。

検証

• commandmate verify の 15 ゲート（fmt・clippy -D warnings・cargo test の全体を含む）は、develop c98bdc7c の上の HEAD 26d9bf56 で 15/15 pass でした。
• cargo test --features gui --test gui_server：87 passed でした。
• 同時実行：受け入れ条件の「2 checkout で同時に 3 回」の代わりに、同じビルドのテストバイナリ（87 本の全体）を 2 プロセス同時に起動することを 5 回くり返しました。ビルドは 1 本ずつという運用のためです。
  • SIGINT と SIGQUIT は既定に戻して起動しました。
  • 結果は 10/10 pass で、プロセスは残っていません。
  • 同じ条件で develop c98bdc7c を対照として回すと、10 本中 9 本が FAIL しました（主に unclassified_nextjs_create_is_unmeasured_confirmed_and_delegated）。
• 二点測定は対象外です（待ちの統一のため）。worker は、待ちの条件を届かない値に変えると、helper の「timed out … last state」で FAIL することを確かめています。
• 成長上限の対象（runner.rs・loop_run.rs・baseline）は変えていません。

確かめてほしい点

• 純増は +64 行で、Issue の目安（純増 0 以下）を超えています。大半（+62）は、同時実行の競合を直すために足した 3 箇所の待ちです。
• J の範囲（gui_stop_*・server_ready_wait_*）は、修正の前の同時実行でまれに落ちました。修正後の 10 本では起きていませんが、この PR では直していません。

レビュー

テストの変更なので、merge 前レビューは行いません（利用者承認済み）。

Refs #647

🤖 Generated with Claude Code