Assertions: inversion des champs "expected" et "actual" - Githubissues

loicsteinmetz / alternbot-app

Bot Discord de gestion de l'emploi du temps - IUTM

https://alternbot-web.herokuapp.com/

0 stars 2 forks source link

Assertions: inversion des champs "expected" et "actual" #91

Open ghost opened 3 years ago

ghost commented 3 years ago

SPOILER: Ça concerne un détail insignifiant et on est vraiment en plein dans le pinaillage.

J'ai déjà remarqué à plusieurs endroits dans les tests que les champs expected et actual des certaines assertions étaient inversés. Un exemple ici: https://github.com/B4va/alternbot-app/blob/d7ba548f2a49eef9b9a34f39f55bab995024100c/src/test/java/process/schedule/data/TestIcalMappingProcess.java#L24 https://github.com/B4va/alternbot-app/blob/d7ba548f2a49eef9b9a34f39f55bab995024100c/src/test/java/process/schedule/data/TestIcalMappingProcess.java#L128

Ça donne une sortie "inversée" lors de l'exécution du test:

expected: <sessions.get(0).getEnd()> but was: <10:00>
Comparison Failure: 
Expected :sessions.get(0).getEnd()
Actual   :10:00

loicsteinmetz commented 3 years ago

J'ai ajouté une carte sur Trello ;)