dotnet/roslyn

[Tests] These tests seem to check SyntaxToken inside of SyntaxNode

Open

#38.311 aperta il 27 ago 2019

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-CompilersBugTesthelp wanted

Metriche repository

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

Descrizione

See https://github.com/dotnet/roslyn/blob/master/src/Compilers/Core/CodeAnalysisTest/CommonSyntaxTests.cs#L181-201 Names node, token and trivia suggest that this test was intended to test VisualBasicExtensions.Kind methods for SyntaxNode, SyntaxToken and SyntaxTrivia of C#. In fact, both first and second invocation test the same method Kind(this SyntaxToken token), because node is actually a SyntaxToken. The same applies to VisualBasicSyntax_CSharpKind test as well.

Guida contributor

Direzione di ricerca
Verifica i metodi di test e modificali per testare tutti e tre i tipi utilizzando variabili separate per SyntaxNode, SyntaxToken e SyntaxTrivia, quindi chiama gli overload appropriati di Kind.
Tech stack
csharp
Dominio
backend
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#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75