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

Test : Tracker #96

Closed
salimkanoun opened this issue Oct 25, 2020 · 0 comments
Closed

Test : Tracker #96

salimkanoun opened this issue Oct 25, 2020 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@salimkanoun
Copy link
Collaborator

Dans le test du Tracker, tu fait un appel d'API pour générer des lignes dans le Tracker et ensuite faire des assertions sur l'API Tracker pour retrouver cette ligne.

C'est une mauvaise idée car si ton API qui te sert a générer des lignes dans le Tracker est cassée pour une raison qui n'a rien a voir avec le tracker ton test sur le Tracker tombe aussi (alors que ta fonction tracker sait bien lire ce qu'il y a dans la table tracker).

Ca devient difficile a debeuger, tu pense que c'est l'API tracker qui est cassée alors que c'est une autre API.

Il faut rester unitaire autant que possible, donc au lieu d'appeler une API pour générer des ligne dans la table Tracker, il faut plutot ecrire dans la BDD des lignes dans la table tracker via les factory.
Comme ca ton test reste unitaire et si il tombe c'est uniquement du a la fonction Tracker.

A terme on pourra faire des test pour verifier qu'en executant une API t'a bien un enregistrement dans Tracker que tu retrouve sur l'API Tracker mais ce sont des tests End To End et pas des tests unitaires / integration.
C'est des test utile pour tester tout l'applicatif mais plus long a excuter et plus dificile a debeuger que les tests unitaire (mais utile pour tester la cohérence de l'ensemble)
image

@salimkanoun salimkanoun added the bug Something isn't working label Oct 25, 2020
EmilieOLIVIE added a commit that referenced this issue Oct 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants