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