CommunityToolkit / CommunityToolkit/dotnet
Fix unit tests assertion parameters for expected and actual values
- Langage dominant
- C#
- Étoiles
- 3.8k
- Forks
- 400
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
## Expected behavior
In unit tests, `Assert` methods such as `Assert.AreEqual(value, otherValue)` should have the first parameter be the expected value, second parameter the actual value.
## Current behavior
Some correct usage, mostly backwards. [Example](https://github.com/CommunityToolkit/dotnet/blob/main/tests/CommunityToolkit.Mvvm.UnitTests/Test_ICommandAttribute.cs#L20)
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez par tests/CommunityToolkit.Mvvm.UnitTests/Test_ICommandAttribute.cs et examinez les appels à Assert référencés par l’issue. Vérifiez la suite de tests unitaires pour repérer les paramètres expected et actual inversés, puis exécutez les tests concernés afin de confirmer que l’ordre des assertions a été corrigé partout.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- csharp
- Domaine
- testing
- Type d'issue
- Refactorisation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100