kimata / rasp-shutter / 29177425660

84%

Build Type

push

github

Committed by kimata

Commit Message

fix(test): スケジュール適用前の時刻操作で発生する flaky を根本修正

E2E テスト test_schedule_run が CI で稀に失敗していた要因を確定して修正。

要因: スケジュール保存(multiprocessing.Queue への put)とスケジューラスレッドの
ジョブ登録は非同期で、既存の同期(ループシーケンス 2 周待ち)はキュー処理を
保証しない。put の可視化遅延で適用が時刻進行後にずれると、schedule ライブラリが
next_run を datetime.now() 基準で「翌日」に計算し、ジョブが発火しなくなる。
また advance_mock_time の traveler stop/start 再作成の隙間には実時刻が露出し
(実測: advance 2000 回で露出 2665 回)、この間の登録でも同じ結果になる。

対策:
- scheduler にスケジュール適用世代番号を追加し、キュー取り込み・ジョブ登録の
  完了ごとにインクリメント
- テスト API に schedule_generation / wait_schedule_applied を追加
- E2E は save_schedule_and_wait() で、integration は ScheduleAPI.update() 内で
  適用完了を待ってから時刻を操作するように変更
- 時刻モック API を traveler の stop/start 再作成から Traveller.move_to()/shift()
  に変更し、実時刻の露出窓を排除(実測: 露出 0 回)
- 保証にならない待機ヘルパー wait_for_schedule_update_seq /
  wait_for_schedule_update を削除

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Coverage Stats

27 of 44 new or added lines in 3 files covered. (61.36%)

10 existing lines in 2 files now uncovered.

1673 of 2002 relevant lines covered (83.57%)

0.84 hits per line