Skip to content

docs(compliance): 收斂測試與合規規範#32

Merged
perditavojo merged 1 commit into
mainfrom
dev
Apr 28, 2026
Merged

docs(compliance): 收斂測試與合規規範#32
perditavojo merged 1 commit into
mainfrom
dev

Conversation

@perditavojo
Copy link
Copy Markdown
Contributor

補齊測試 XML summary 並修正 SystemHelperTests 編碼。
更新 MTP dotnet test 指令為 --project 格式。
收斂 README 的手動剪貼簿中轉描述,並補強 JS 條款頁抓取要求。

補齊測試 XML summary 並修正 SystemHelperTests 編碼。
更新 MTP dotnet test 指令為 --project 格式。
收斂 README 的手動剪貼簿中轉描述,並補強 JS 條款頁抓取要求。
Copilot AI review requested due to automatic review settings April 28, 2026 14:29
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

這個 PR 針對測試與合規相關文件做一致性收斂:補齊測試的 XML summary、修正單一測試檔案的編碼,並更新文件中 Microsoft Testing Platform (MTP) 的 dotnet test 指令格式,同時強化合規條款頁抓取在遇到 JS shell 時的處置說明。

Changes:

  • 補齊多個測試案例的 /// <summary> XML 文件註解,讓測試意圖更明確。
  • 將測試執行文件中的 dotnet test 指令統一更新為 dotnet test --project ... 格式(含 detailed logger 與 coverage 範例)。
  • 更新合規工作流文件:若條款頁只回傳 JavaScript shell,要求改用可執行 JS 的工具或記錄無法擷取並停止高風險變更;並收斂 README 的剪貼簿中轉描述措辭。

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/InputBox.Tests/SystemHelperTests.cs 修正檔案開頭編碼/格式(BOM)以符合專案現況,避免測試檔案編碼不一致。
tests/InputBox.Tests/README.md 更新 MTP 測試執行指令為 --project 形式,維持與 repo 測試說明一致。
tests/InputBox.Tests/GamepadShoulderShortcutArbiterTests.cs 為測試方法補上 XML summary,強化測試行為意圖可讀性。
tests/InputBox.Tests/GamepadCalibrationVisualizerMapperTests.cs 補齊多個測試的 XML summary,讓校準/播報/導覽守衛的回歸意圖更清楚。
tests/InputBox.Tests/DialogLabelStabilityTests.cs 補齊測試方法 XML summary,清楚描述固定寬度避免版面抖動的驗證目的。
docs/engineering/testing.md 文件中的 dotnet test 指令改為 --project 格式並同步 coverage 範例。
docs/engineering/git-commit-safety.md 強化條款頁抓取遇到 JS shell 時的流程要求與停止高風險異動的規範描述。
README.md 收斂對「手動觸發流程」的描述,明確區分輸入暫存/剪貼簿中轉與實際遊戲內貼上操作責任。

@perditavojo perditavojo merged commit c7667de into main Apr 28, 2026
13 of 14 checks passed
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.

2 participants