CommunityToolkit / CommunityToolkit/dotnet

Fix unit tests assertion parameters for expected and actual values

Abierto
#55 0 comentarios 0 reacciones 0 asignados Ver en GitHub
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

Abrir la 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.