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

add Tests ApiCreateFacturaClienteTest.php #1547

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

elguitarraverde
Copy link
Contributor

@elguitarraverde elguitarraverde commented Mar 28, 2024

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:

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant