dotnet/roslyn

Missing test case for conditional access in `CheckValEscape`

開放

#57,653 建立於 2021年11月9日

在 GitHub 查看貢獻者指南
 (0 則留言) (0 個反應) (0 位負責人)C# (4,257 個分叉)batch import
Area-CompilersBughelp wanted

倉庫指標

星標
 (20,414 顆星)
PR 合併指標
 (平均合併 6天 17小時) (30 天內合併 256 個 PR)

描述

https://github.com/dotnet/roslyn/pull/57050#discussion_r726383228 adds a test case that hits a conditional access receiver in Binder.GetValEscape. This suggests that CheckValEscape should also be able to see this node in an error scenario, like GetValEscape, but we don't have a test case covering this so we haven't added it to the switch there. We should investigate to see if a test case can be created where that node is hit in CheckValEscape as well.

貢獻者指南

研究方向
首先檢視 PR #57050 中連結評論的討論，以理解條件存取情境。接著檢查 `Binder.cs` 中的 `CheckValEscape` 和 `GetValEscape` 方法，以識別條件存取節點可能出現在錯誤路徑的位置。嘗試建立一個觸發該程式碼路徑的測試案例，可能是透過撰寫一個包含條件存取並導致錯誤的程式碼片段，並驗證 `CheckValEscape` 是否正確處理。
技術棧
csharp
領域
testing
議題類型
測試
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
2
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-3 小時
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
活躍
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
清晰
前置要求
C#TestingGit
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
75