facturascripts icon indicating copy to clipboard operation
facturascripts copied to clipboard

add Tests ApiCreateFacturaClienteTest.php

Open elguitarraverde opened this issue 2 years ago • 0 comments

Descripción

  • Si haces publicas algunas propiedades, podemos testear los controladores.
  • En este caso, se realizan los tests para los errores que puedan darse en la request.
  • Faltaría comprobar que la factura se guarda correctamente en la base de datos cuando ya no existe ningún error.
  • Para probar los controladores correctamente, creo que lo mejor seria poner el metodo runResource como público.
  • Otra forma de poder hacer los tests sin modificar las propiedades a publicas es crear clases Mock y entonces ahí se hacen métodos publicos que devuelvan esas propiedades o metodos protected.

creo que esto sale en los test porque se termina llamando al response->send(). por eso la importancia de poder llamar al metodo runResource(). para no tener que ejecutar toda la logica de la respuesta. image

¿Cómo has probado los cambios?

Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado:

  • [x] He revisado mi código antes de enviarlo.
  • [x] He probado que funciona correctamente en mi PC.
  • [x] He probado que funciona correctamente con una base de datos vacía.
  • [x] He ejecutado los tests unitarios.

elguitarraverde avatar Mar 28 '24 16:03 elguitarraverde