Skip to content

Scanner tests

Xoeseko Yao Nyomi requested to merge scannerTests into master

Résumé de l'implémentation

(Bref résumé de ce qui a été fait et des choix techniques importants) Ajout de tests unitaires pour la fonctionnalité de scanner

Definition of done

  • Tests unitaires
  • Review

Les tests ne passent pas il faut identifier si c'est un danger ou si nous pouvons quand même avancer quand on aura la liste d'évènements on pourra voir si la situation est génante en pratique ou seuelement sur le cas théorique.

Il serait aussi intéressant de printer le QR code que nous générons pour voir si l'erreur est due au QR code utilisé ou à notre implémentation.

L'erreur majeur vient du constructeur d'event pour JSON qui prend une liste d'entiers pour les clés alors que notifyMe donne une chaîne de caractère en hex. L'erreur peut être évitable si on ne travaille pas directement avec la chaîne de caractères mais ça inspire des questions.

Edited by Xoeseko Yao Nyomi

Merge request reports