search

ign-packo / PackO

Outil pour le contrôle et la retouche du mosaïquage d'ortho
Other
6 stars 2 forks source link

Amélioration couverture de test (gdal_processing.js, db.js, middelwares/wmts.js et middelwares/branch.js) #218

Open ftoromanoff opened 2 years ago

ftoromanoff commented 2 years ago

Pour améliorer la couverture de test sur gdal_processing.js, il faudrait pouvoir tester des erreurs qui ne se produisent jamais parce que l'on gère les cas limites en amont dans la route ou le middleware. Peut-être faut-il ajouter un fichier regress spécifique à gdal_processing pour tester:

C'est la même chose pour le fichier db/db.js qui contient nos requête SQL. Il faudrait éventuellement un fichier regress dédié qui vérifie le comportement en cas d'erreur?

Originally posted by @gmaillet in https://github.com/ign-packo/PackO/issues/204#issuecomment-934332899

ftoromanoff commented 2 years ago

Il y a aussi la couverture du fichier middelwares/wmts.js pour les erreurs du GetFeatureInfo a revoir.

Originally posted by @gmaillet in https://github.com/ign-packo/PackO/issues/204#issuecomment-934267877 Dans le fichier middelwares/wmts.js il y a deux groupes de lignes concernant la gestion d'erreur sur le GetFeatureInfo qui ne sont pas couverts par les tests (326-332,335-342). On a pourtant bien des tests pour vérifier les erreurs de type "out of bounds". Soit on supprime les catchs si on pensent qu'il n'y a aucun risque d'erreur, soit on les conservent mais il ne faut pas indiquer un "out of bounds" mais plutôt un "unexpected error" ou quelque chose dans le genre.

ftoromanoff commented 2 years ago

Lors du merge de la PR #309 , on a pu remarquer une perte de couverture de 0.08% ce qui montre une couverture incomplète de la version avant changement.