makeitrealcamp / crown

1 stars 8 forks source link

Include tests for the `Api::V1::IncidentsController` class #15

Open Mardiniii opened 4 years ago

Mardiniii commented 4 years ago

Description

We didn't include tests for the Api::V1::IncidentsController class and we should do it. Give us a hand adding a few examples to make sure everything is working correctly.

Expected Outcome

jairorjuela commented 4 years ago

Hola para los test quisiera proponer:

Por ahora solo sería instalar Rspec, FactoryBot y Faker, y dejar todo testeado ya que no hay tantas cosas aún para testear.

Mardiniii commented 4 years ago

Hola @jairorjuela, gracias por tu comment. De una, vamonos con toda con RSpec. La documentación del API la estamos manejando por ahora en la wiki. Veamos como evoluciona pero lo de usar una herramienta que nos automatice este proceso suena genial. Si tienes alguna pregunta me cuentas, en cuanto subas un PR, le doy un vistazo. Muchísimas gracias! :)