dotnet/roslyn

Missing test case for conditional access in `CheckValEscape`

Ouverte

#57 653 ouverte le 9 nov. 2021

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)C# (4 257 forks)batch import
Area-CompilersBughelp wanted

Métriques du dépôt

Stars
 (20 414 étoiles)
Métriques de merge PR
 (Merge moyen 6j 17h) (256 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Commencez par examiner la discussion dans la PR #57050 au commentaire lié pour comprendre le scénario d'accès conditionnel. Ensuite, examinez les méthodes `CheckValEscape` et `GetValEscape` dans `Binder.cs` pour identifier où un nœud d'accès conditionnel pourrait apparaître dans les chemins d'erreur. Essayez de créer un cas de test qui déclenche ce chemin de code, probablement en écrivant un extrait de code avec un accès conditionnel qui conduit à une erreur, et vérifiez que `CheckValEscape` le gère correctement.
Stack technique
csharp
Domaine
testing
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#TestingGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75