developersdo / empleo-dot-net

ASP.NET MVC5 - Iniciativa de aprendizaje mediante ejemplos
https://emplea.do
The Unlicense
211 stars 110 forks source link

Crear proyecto de unit tests #126

Closed EfrainReyes closed 8 years ago

EfrainReyes commented 8 years ago

El src deberia tener una suite de tests que pruebe el código de la aplicación.

Estoy creando un issue para que decidamos las librerias a usar en nuestra test suite. Yo puedo abrir un PR despues con el proyecto creado y algunos tests de ñapa.

Yo propongo:

Por qué NUnit 2.x? La versión 3 salió en Noviembre, pero el soporte de los test runners es relativamente limitado:

Ahora bien si el consenso es que mejor nos vamos con v3 porque es más reciente, no me voy a oponer.

Por qué NSubstitute y no algo como Moq? Llevo un año trabajando con NSubstitute, y me la encuentro mucho más sencilla y fácil de aprender y de usar que Moq. Además está muy bien documentada.

De nuevo, la idea de crear este issue es que haya una conversación en caso de que la gente quiera proponer alternativas.

eatskolnikov commented 8 years ago

Yo no me opongo

luis-ramirez commented 8 years ago

Sería súper cool ir probando porque se están introduciendo muchas cosas nuevas y no se esta probando prácticamente nada.

EfrainReyes commented 8 years ago

Claro luis. Y aclaro, no es que yo este diciendo hey vamos a hacer todos los tests que corresponden ahora mismo, sino, tengamos este proyecto habilitado, y que la gente vaya escribiendo tests para la funcionalidad que no tiene tests todavia.

Ya en un futuro mas a mediano plazo se pudiera decidir si se va a exigir tener tests en los PR.

RaulMonteroC commented 8 years ago

De mi parte lo veo bien.

claudiosanchez commented 8 years ago

@EfrainReyes Algo a tener en cuenta es que si nos mudamos a ASP.NET vNext, no se si eso afecta el package de testing que elijamos.

EfrainReyes commented 8 years ago

Creo que lo unico que afectaria es si se quiere usar .NET Core para correr la app en linux o en osx, pero me parece que de todos modos hay implementaciones multiplatform de NUnit (recuerdo que en Xamarin uno usaba NUnit Lite).

luis-ramirez commented 8 years ago

132