search

etalab / transport-site

Rendre disponible, valoriser et améliorer les données transports
https://transport.data.gouv.fr
198 stars 30 forks source link

Validateur NeTEx : quelques metadata #4160

Closed ptitfred closed 2 months ago

ptitfred commented 2 months ago

Sur une suggestion d'Antoine, cette PR garde trace du temps passé et du nombre de retries pour une validation NeTEx.

Voir #4153.

ptitfred commented 2 months ago

J'aurais eu un peu plus de temps ce matin, j'aurais fait tourner le coverage (https://github.com/etalab/transport-site?tab=readme-ov-file#measuring-test-coverage), pour avoir une idée plus précise ce qui est couvert par les tests ; si jamais ça te tente ça peut être intéressant !

Une utilisation naïve des commandes listées dans le readme échoue par un "Process killed" chez moi. Cela dit je suis relativement confiant quant à la couverture des tests.

Le plus gros souci à mon goût est surtout que malgrè la présence des typespec, et à cause de l'utilisation d'un mock sur ce wrapper dans les tests du code appelant, un changement du client peut casser le code appelant sans que ça saute aux yeux dans la CI.

ptitfred commented 2 months ago

:warning: Je dois corriger le comportement. Je ne lis pas les bonnes dates pour les validations terminées (success, warning ou error), et je dois vérifier le comportement de l'API pour le cas pending.

Il est possible que le timeout implémenté dans #4157 ne fonctionne pas non plus en pratique à cause de ça.

Ne pas merger donc.

ptitfred commented 2 months ago

⚠️ Je dois corriger le comportement. Je ne lis pas les bonnes dates pour les validations terminées (success, warning ou error), et je dois vérifier le comportement de l'API pour le cas pending.

Il est possible que le timeout implémenté dans #4157 ne fonctionne pas non plus en pratique à cause de ça.

Ne pas merger donc.

C'est fait.