CommunityToolkit / CommunityToolkit/dotnet
Fix unit tests assertion parameters for expected and actual values
- Lenguaje dominante
- C#
- Estrellas
- 3.8k
- Forks
- 400
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
## 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)
Guía de contribución
Línea de trabajo
Comienza con tests/CommunityToolkit.Mvvm.UnitTests/Test_ICommandAttribute.cs e inspecciona las llamadas a Assert referenciadas por la issue. Revisa la suite de pruebas unitarias en busca de parámetros expected y actual invertidos y, después, ejecuta las pruebas relevantes para confirmar que el orden de las aserciones se ha corregido en todas partes.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- csharp
- Área
- testing
- Tipo de issue
- Refactorización
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100