Pilotage: Utiliser une date qui prend en compte la timezone pour EligibilityDiagnosis.created_at

[francoisfreitag]

🤔 Pourquoi ?

Problème d’interprétation de date (naïve), révélé par pytest --randomly-seed=3296184821 tests/metabase/management/test_populate_metabase_emplois.py::test_populate_job_seekers -vv.

[xavfernandez]

Moi, je préfère ce fix mais je laisse @rsebille décider en rapport avec #5311 👼

[francoisfreitag]

Note: j’ai hésité (et j’hésite toujours) à un introduire un petit helper qui gèrerait None (au lieu de renvoyer la date du jour comme le fait localdate()).
Qu’en pensez-vous ?

[xavfernandez]

Pas persuadé de l'utilité de la fonction gérant le None :)

[francoisfreitag]

Je vois juste venir le bug silencieux du copier coller sur un champ nullable, qui amène à timezone.localdate(None) et on envoie la date du jour.

[rsebille]

Note: j’ai hésité (et j’hésite toujours) à un introduire un petit helper qui gèrerait None (au lieu de renvoyer la date du jour comme le fait localdate()). Qu’en pensez-vous ?

Je vois juste venir le bug silencieux du copier coller sur un champ nullable, qui amène à timezone.localdate(None) et on envoie la date du jour.

La version actuelle semble OK, mais oui le bug que tu mentionnes peux arriver assez vite.

Mais plutôt que de faire la modif à chaque utilisation et donc d'avoir besoin d'un helper peut-être faire ce que tu voulais faire ici ?

les-emplois/itou/metabase/db.py

Lines 260 to 262 in faaccfb

	if c["type"] == "boolean":
	c["type"] = "integer"
	c["fn"] = compose(convert_boolean_to_int, c["fn"])

[francoisfreitag]

Excellente suggestion, tout à fait ce que je voulais 👍

[francoisfreitag]

C’est un peu plus magique, mais évite les oublis futurs. Idéalement, on ne déclarerait pas en type date un datetime et ce cast deviendrait inutile.

[rsebille]

Merci ❤️