[unit-tests] カバレッジ補強 フェーズ1: AIfCmd パイロット条件バグ修正+テスト追加 - #1093
Open
github-actions[bot] wants to merge 1 commit into
Open
[unit-tests] カバレッジ補強 フェーズ1: AIfCmd パイロット条件バグ修正+テスト追加#1093github-actions[bot] wants to merge 1 commit into
github-actions[bot] wants to merge 1 commit into
Conversation
AIfCmd.PrepareArgs() において、条件式がパイロット名 (terms.Count == 1 or 2) の場合に ExprTermCount が設定されず 0 のままになっていた。 これにより Evaluate() での case 1/2 (パイロット条件分岐) に到達できず、 ヘルプ記載の「パイロット名条件」が機能しない状態だった。 修正: switch の前に ExprTermCount = terms.Count を追加し、 式が確定した場合のみ 0 に上書きする正しい動作に修正した。 テスト: AIfCmdPilotTests.cs を追加 (15件) - パイロット出撃/未出撃/未乗機/格納 の各条件 - Not パイロット名 の逆条件 - ExprTermCount の値確認 - MissingEndIf エラーハンドリング - ElseIf でのパイロット条件 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🧪 カバレッジ補強レポート (2026-09-01)
現在のフェーズ / Current Phase
バグ修正 / Bug Fix
AIfCmd.PrepareArgs() — ExprTermCount 未設定バグ
AIfCmd.PrepareArgs()において、条件式がパイロット名 (terms.Count == 1 or 2) の場合にExprTermCountが設定されず0のままになっていた。修正前:
switch (terms.Count)の前にExprTermCountの初期化なし → パイロット条件が動作しない修正後:
ExprTermCount = terms.Count;を switch の前に追加。式が確定した場合のみ0に上書きする正しい動作に。影響範囲: ヘルプドキュメント記載の以下の機能が修正により正常動作するようになった:
If パイロット名 Then— パイロットのユニットが出撃中なら成立If not パイロット名 Then— パイロットのユニットが出撃していなければ成立追加したテスト / Tests Added
AIfCmdPilotTests.cs: 15件追加Not パイロット名逆条件 3ケース合計: 15件追加
カバレッジサマリー / Coverage Summary
齟齬の報告 / Discrepancies Found
AIfCmd.Evaluate(): 「格納」ステータスがヘルプと不一致 → Issue 作成済みテスト結果 / Test Results
Passed: 7504, Skipped: 0, Failed: 1 (pre-existing: LocalFileConfigTests.LoadTest)
次のステップ: Command.attack.cs / Command.process.cs のテスト追加(GUI依存が強い)
Warning
Firewall blocked 9 domains
The following domains were blocked by the firewall during workflow execution:
crl3.digicert.comcrl4.digicert.comdc.services.visualstudio.comocsp.digicert.coms.symcb.coms.symcd.comtscrl.ws.symantec.comtsocsp.ws.symantec.comwww.microsoft.comSee Network Configuration for more information.