0 Kommentare (0 Kommentare)0 Reaktionen (0 Reaktionen)0 zugewiesene Personen (0 zugewiesene Personen)C#4.257 Forks (4.257 Forks)batch import
Area-CompilersBughelp wanted
Repository-Metriken
- Stars
- 20.414 Sterne (20.414 Sterne)
- PR-Merge-Metriken
- Durchschn. Merge 6T 17h (Durchschn. Merge 6T 17h)256 gemergte PRs in 30 T (256 gemergte PRs in 30 T)
Beschreibung
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.
Contributor Guide
- Research-Richtung
- Beginnen Sie mit der Überprüfung der Diskussion in PR #57050 im verlinkten Kommentar, um das Szenario des bedingten Zugriffs zu verstehen. Untersuchen Sie dann die Methoden `CheckValEscape` und `GetValEscape` in `Binder.cs`, um zu identifizieren, wo ein bedingter Zugriffsknoten in Fehlerpfaden auftreten könnte. Versuchen Sie, einen Testfall zu erstellen, der diesen Codepfad auslöst, wahrscheinlich durch das Schreiben eines Code Snippets mit bedingtem Zugriff, der zu einem Fehler führt, und überprüfen Sie, ob `CheckValEscape` ihn korrekt behandelt.
- Tech Stack
- csharp
- Domain
- testing
- Issue Type
- Test
- Voraussetzungen
- C#TestingGit