ClubCedille / jardiniot

Système pour des buckets autonomes servant à faire pousser des plantes
GNU General Public License v3.0
13 stars 11 forks source link

Changer la structure du backend et ajouter des tests #73

Closed AXDOOMER closed 5 years ago

AXDOOMER commented 5 years ago
AXDOOMER commented 5 years ago

@ioioioio Merci pour ta review. Tes suggestions sont pas mal bonnes.

notarock commented 5 years ago

Merci pour vos bonnes suggestions! Je vais regarder pour les appliquer bientôt :+1:

AXDOOMER commented 5 years ago

controller/bucket.py est indenté avec des espaces. ouache.

notarock commented 5 years ago

J'ai corrigé les tabs. Dit moi s’il y a quelque choses que je dois faire avant qu'on puisse enfin fermer ça :) @AXDOOMER

AXDOOMER commented 5 years ago

@notarock Check les tests unitaires SVP. Il en reste deux à fix.

notarock commented 5 years ago

J'ai trouvé le problème. Une requête SQL du type "SELECT" ne peut pas être effectuée avec executemany. J'ai donc ajouté une méthode selectparam qui permet d'exécuter une requête select avec des paramètres.

mikefaille commented 5 years ago

vive les testes unitaires ;-)

notarock commented 5 years ago

J'ai fix les tests unitaires @AXDOOMER