mobility-team / mobility

Mobility, an open-source library for mobility modelisation
MIT License
16 stars 11 forks source link

[Projet] Définir une stratégie de test #27

Closed FlxPo closed 1 year ago

FlxPo commented 1 year ago

Il faut que l'on ajoute des tests à notre code, pour bien encadrer les modifications du code existant. Cela deviendra d'autant plus nécessaire que le code gagne en complexité.

Quelques éléments intéressants ici : https://about.codecov.io/blog/the-10-commandments-of-writing-good-software-tests/

J'ai écrit un premier test ici pour se lancer : https://github.com/mobility-team/mobility/blob/carbon/test/parsers/test_ademe_base_carbone_api.py

Mind-the-Cap commented 1 year ago

Les éléments que tu donnes sont déjà plutôt des bons conseils. Pour moi, l'essentiel est de rattraper les erreurs les plus courantes qui pourraient être faites et qui pourraient modifier les résultats, c'est comme ça que je pense les tests.

Je suis toujours en faveur d'une coverage la plus proche des 100 %, mais on peut déjà commencer et voir ce qu'on arrive à faire.

J'ai modifié https://github.com/mobility-team/mobility/pull/25 pour ajouter pytest à l'action automatisée, il s'avère que les tests qu'on a déjà ne passent pas.

Mind-the-Cap commented 1 year ago

La coverage est à 95 %, les tests passent et ont bien été automatisés !