dotnet/roslyn

Unskip unit test AddImportTests.TestAddImportWithCaseChange

Ouverte

#41 484 ouverte le 7 févr. 2020

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (1 personne assignée)C# (4 257 forks)batch import
Area-IDEBughelp 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

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.

Guide contributeur

Direction de recherche
Inspectez le test ignoré et le cadre de test associé pour comprendre pourquoi la validation n'est pas effectuée. Ensuite, étudiez la logique de correction de code pour déterminer pourquoi elle renvoie une priorité élevée au lieu d'une priorité moyenne.
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.
3
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#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70