chore: add test on TruncateMessageCommand #180 - #586
guillaumedc wants to merge 2 commits into
Conversation
Test Results171 tests 171 ✅ 33s ⏱️ Results for commit c4c7ce3. ♻️ This comment has been updated with latest results. |
602a4e1 to
f2f66c1
Compare
1a38b5a to
b75352e
Compare
c65fbd8 to
9a29aa7
Compare
b75352e to
d3d99d4
Compare
9a29aa7 to
2466d6b
Compare
d3d99d4 to
195f415
Compare
2466d6b to
2865288
Compare
195f415 to
78efad6
Compare
2865288 to
cb65447
Compare
78efad6 to
cd79b33
Compare
cb65447 to
473c554
Compare
cd79b33 to
b214462
Compare
| @@ -0,0 +1,68 @@ | |||
| <?php | |||
There was a problem hiding this comment.
Cette review ajoute une entité Doctrine et modifie donc du code de production. Je serais d'avis soit d'extraire ces changements dans une PR dédiée, puisque celle-ci est centrée sur les tests, soit d'ajouter des tests autour de ce mapping, notamment sur la relation entre OutMessage et OutQuarantine, ainsi que sur la suppression des quarantaines associées.
There was a problem hiding this comment.
Les tests PHPUnit dans Symfony ne lancent pas les migrations pour créer les tables.
Ils se basent sur les entités et leur mapping Doctrine.
J'ai l'impression qu'il n'est ainsi pas possible d'écrire ces tests sans introduire ces nouvelles entités.
À noter que c'est exactement le même mapping pour les entités In et Out.
Pour vérifier que les entités correspondent au schéma existant en base, on peut exécuter la commande make migration et vérifier qu'aucune migration ne soit générée.
473c554 to
3df21f6
Compare
b214462 to
02e737c
Compare
02e737c to
9e6366d
Compare
f0093d9 to
c4c7ce3
Compare
Related issue(s)
#180
How to test manually
make testReviewer checklist