dotnet/roslyn

Unskip unit test AddImportTests.TestAddImportWithCaseChange

Aperta

#41.484 aperta il 7 feb 2020

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (1 assegnatario)C# (4257 fork)batch import
Area-IDEBughelp wanted

Metriche repository

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

Descrizione

Following unit test is attempting to validate that the CodeActionPriority of the offered code fix is Medium and not High. https://github.com/dotnet/roslyn/blob/aaed1133a5bc0db1387ae8538f0b648520d55241/src/EditorFeatures/VisualBasicTest/Diagnostics/AddImport/AddImportTests.vb#L78-L102

However, due to a bug in test framework, the validation is not even being performed. Attempting to fix the test framework in https://github.com/dotnet/roslyn/pull/41363 causes the unit test to fail as the offered code fix is indeed still at CodeActionPriority.High. I am going to skip the unit test as part of that PR, and use this issue to track fixing the product behavior and unskip the test.

Guida contributor

Direzione di ricerca
Ispeziona il test saltato e il relativo framework di test per capire perché la convalida non viene eseguita. Quindi, indaga sulla logica della correzione del codice per determinare perché restituisce priorità alta invece di media.
Tech stack
csharp
Dominio
testing
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
70