操作
リファクタ #174
完了JudgeManager のテスト追加と責務分割方針を検討する
リファクタ #174:
JudgeManager のテスト追加と責務分割方針を検討する
説明
Redmine #165 のコードベースレビューからの派生タスクです。
背景¶
src/bms/player/beatoraja/play/JudgeManager.java は約 888 行の中核クラスですが、Manager 本体を対象にしたテストが不足しています。判定処理は gameplay 回帰リスクが高いため、まずテスト追加を優先します。
対応内容¶
-
JudgeManagerの主要な入力・判定・状態更新パスを洗い出す。 - 既存の
JudgeAlgorithmTestと重複しない範囲で、Manager 層の回帰テストを追加する。 - テスト容易性を妨げる依存があれば、挙動を変えない小さな責務分割を検討する。
- 大規模な設計変更が必要な場合は別 issue 化する。
確認観点¶
- 判定タイミング、LN、ミス、ゲージ・スコア連携に回帰がないこと。
- テストが実装詳細に過度に依存しないこと。
操作