GEIQ : meilleure gestion du type de la colonne SIRET du xls importé#5297
GEIQ : meilleure gestion du type de la colonne SIRET du xls importé#5297
Conversation
EwenKorr
added
no-changelog
xavfernandez
left a comment
xavfernandez
left a comment
There was a problem hiding this comment.
Je sais qu'il n'y a pas de tests sur cette commande, mais ça vaudrait peut-être le coup de commencer à en mettre, en commençant par tester juste la fonction get_geiq_df .
Tu as la fonction generate_excel_sheet pour créer des fichiers excel et la fixture tmp_path_factory pour stocker des fichiers temporaires 👼
EwenKorr
force-pushed
the
ewen/import_geiq
branch
from
f367586 to
9e6753e
Compare
|
Je commence à ajouter des tests donc. Et grâce à ça j'ai pu un peu mieux isoler le problème. Avec le fichier actuel ( Quand au moins 1 SIRET manque, la colonne prend le format |
EwenKorr
force-pushed
the
ewen/import_geiq
branch
2 times, most recently
from
128c70f to
5415f1a
Compare
When replacing NaN elements with None, if the siret column is not explicitely defined as integer, it is converted to float. These values later converted to string, need to be integers, otherwise they are suffixed with `.0`.
EwenKorr
force-pushed
the
ewen/import_geiq
branch
from
5415f1a to
0b84479
Compare
xavfernandez
left a comment
xavfernandez
left a comment
There was a problem hiding this comment.
Merci pour le fix et l'ajout de tests 🙏
|
Merci d'avoir pointé les éventuelles conséquences d'une valeur/date par défaut dans la définition d'une fonction ! |
2 participants
🤔 Pourquoi ?
Parfois les SIRET sont des float dans le tableur importé. Une fois casté en chaîne de caractères, le SIRET devient 'XXXXXXXXXXXXXX.0'.
cf https://inclusion.sentry.io/issues/15011873/
C'est la solution la plus courte que j'ai trouvée, mais je suis preneur d'amélioration le cas échéant !
Label
bugou pas ? 🤔