Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
Mardiniii opened this issue May 17, 2020 · 2 comments
Open

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

Mardiniii opened this issue May 17, 2020 · 2 comments
Labels
back-end Back-end related issue

Comments

@Mardiniii
Copy link
Contributor

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

  • Add test examples to cover all the actions not tested from the Api::V1::IncidentsController class.
@Mardiniii Mardiniii added the back-end Back-end related issue label May 17, 2020
@jairorjuela
Copy link

Hola para los test quisiera proponer:

  • Rspec, se que con mini test se pueden lograr buenos test pero me parece muy chevere Rspec en documentación y recursos.

  • Complementar RSpec con gemas como FactotyBot para crear fácilmente lo que vayamos a reusar en pruebas, por ejp un "User" sin tener que crearlo dentro de las pruebas, Faker, VCR por si queremos simular Request par ano estar apuntándole a un Api, etc.

  • Para documentar el back con RSpec se facilita bastante(haciendo pruebas claras con descripciones dicientes) y se puede usar Rswag para generar una autodocumentación del Api, esa gema es bien interesante.

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
Copy link
Contributor Author

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! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back-end Back-end related issue
Projects
None yet
Development

No branches or pull requests

2 participants