dotnet/roslyn

Missing test case for conditional access in `CheckValEscape`

Aperta

#57.653 aperta il 9 nov 2021

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-CompilersBughelp wanted

Metriche repository

Star
 (20.414 stelle)
Metriche merge PR
 (Merge medio 6g 17h) (256 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Inizia esaminando la discussione nella PR #57050 al commento collegato per comprendere lo scenario di accesso condizionale. Quindi esamina i metodi `CheckValEscape` e `GetValEscape` in `Binder.cs` per identificare dove un nodo di accesso condizionale potrebbe apparire nei percorsi di errore. Tenta di creare un caso di test che attivi quel percorso di codice, probabilmente scrivendo uno snippet di codice con accesso condizionale che porta a un errore, e verifica che `CheckValEscape` lo gestisca correttamente.
Tech stack
csharp
Dominio
testing
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#TestingGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75