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

Bugfix #80

Open
wants to merge 27 commits into
base: master
Choose a base branch
from

Conversation

Caromarmachi
Copy link

Projet 5 Testez l'implémentation d'une nouvelle fonctionnalité Java

- le methode getHours était obsolète et ne convenait pas pour le calcul.
- Nous sommes passés par la méthode getTime de l'objet Date (heure de
sortie et entree du ticket), pour calculer une difference en
millisecondes
- puis nous divisons pour obtenir une fraction d'heures pour calculer le
prix
Correction de la requete Get_Ticket pour prendre la bonne ligne quand il
y a plusieurs lignes avec le même reg_number
compte le nombre de vehicule garé, non sorti, pour un meme reg number)
pour vérifier si le vehicule n'est pas déjà garé et non sorti
- ajout d'un formatage du prix (2 décimales)
- Calcul du prix seulement à partir de la 2eme demi heure
- Affichage de la durée calculée
- Affichage de la durée prise en compte pour le calcul
pour calculer le prix d'un vehicule qui sort avant 30mn
- Pour cela nous avons ajouté également une nouvelle requete et donc une
modification de TicketDAO
pour generer une exception en cas de mauvais type de véhicule
l'objet parkingSpot
- déplacement du contenu de la méthode setUpPerTest dans le test
processExitingVehicleTest pour éviter l'erreur des "when" (mockito)
inutiles
double du meme vehicule)
- Ajout du test de le remise des 5% testParking5Percent
objet TicketDAO en plus, pour permettre le test des 5%
- Ajout du test de remise des 5% (bike) testParking5PercentBIKE
l'ancien test des 5% déplacé et corrigé dans la classe
FareCalculatorService
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant