Correction projet
Chablis
Participants:: Saad Mahboubi, Nicolas MATHEY, Teddy Barkou
URL:: https://gitlab.univ-nantes.fr/E186325C/projet-2020.git
MR::
Tickets
- 22 tickets, 7 ouverts et 15 fermés
- Les titres des ticket sont adéquats
- Les tickets sont bien décrits
- Les tickets sont organisés avec des étiquettes.
- Certains tickets ne sont pas attribués à un participant.
- Certains tickets demandent un travail considérable. Pensez à créer des tickets plus granulaires, avec des tâches plus simples.
Validations
Saad Mahboubi seul contributeur à la fin
- 26 validations
- Les titres des validations ne doivent pas être trop génériques (Update, etc.)
- Les corps des messages sont vides.
Tests unitaires
- Après correction des erreurs de compilation, 36 tests exécutés dans
server-main
`, soit 17 classes sur 80, 24% des méthodes et 18% des lignes de code. - Pensez à commenter les méthodes de test
- N'utilisez pas plus d'une assertion par méthode de test.
- N'utilisez pas des assertions à l'intérieur de boucles.
- Apprenez à utiliser les bibliothèques d'assertion de JUnit ou de AssertJ, elles vous feront gagner du temps.
Code source
- Le code source ne compile pas correctement (2 erreurs): certaines classes ne sont pas placées dans les bons fichiers.
- Ajout anecdotique d'un seul fichier
build.gradle
` dans la racine du projet, avec 1 seule dépendance. - Le service Thrift a été implémenté
- Le code respecte majoritairement les conventions de codage de Java.
- Les conditions de garde ne sont pas utilisées.
- Code propre et clair, bien que peu commenté