Projet

Général

Profil

Development #40407

Permettre de tourner le code agent authentic de provisioning en python3

Ajouté par Paul Marillonnet il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
04 mars 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Ça pourrait nous être utile :)


Fichiers


Demandes liées

Bloque Publik Installation Développeur - Development #40403: Tourner authentic2 en python3Fermé04 mars 2020

Actions

Révisions associées

Révision 5fe3fcb6 (diff)
Ajouté par Benjamin Dauvergne il y a environ 4 ans

tests_authentic: format test data (#40407)

Révision 0ca228fa (diff)
Ajouté par Paul Marillonnet il y a environ 4 ans

python3: migrate authentic (#40407)

Historique

#1

Mis à jour par Paul Marillonnet il y a environ 4 ans

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Paul Marillonnet
#2

Mis à jour par Paul Marillonnet il y a environ 4 ans

#3

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Statut changé de Solution proposée à Solution validée

Ok, "permettre de" tourner.

#4

Mis à jour par Paul Marillonnet il y a environ 4 ans

  • Sujet changé de Tourner le code agent authentic de provisioning en python3 à Permettre de tourner le code agent authentic de provisioning en python3

Oui, au temps pour moi. J'attends le résultat des tests — par acquit de conscience un peu superficiel, parce qu'on me souffle dans l'oreillette que cette partie du code n'est pas du tout testée en python3.

#5

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

  • Statut changé de Solution validée à En cours

Les tests ne passent pas.

#6

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

  • Assigné à changé de Paul Marillonnet à Benjamin Dauvergne
#7

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

Ça devrait passer maintenant.

#8

Mis à jour par Frédéric Péters il y a environ 4 ans

            return with_lists_sorted(input.sort(
                key=lambda l: sorted(tuple(d.items()) if isinstance(d, dict) else d for d in l)))

Oui le code était déjà là; on en discutait hier sur le salon, input.sort(...) va retourner None. La construction comme elle est là me semble quand même pouvoir passer mais elle sonne tout à fait bizarre; plus clair serait :

input.sort(...)
return input
#10

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

En essayant de réfléchir avant d'écrire.

#11

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

J'ai du mettre à jour les données de test, les tests étant cassés avant et l'export ayant évolué ça ne testait plus grand chose.

#12

Mis à jour par Paul Marillonnet il y a environ 4 ans

  • Statut changé de Solution proposée à Solution validée

Les deux patches 0001 et 0002 touchent successivement au même fichier de test data_authentic_export_site.json. J'ai l'impression que c'est des modifs de même nature et qu'il y a intérêt à les regrouper toutes dans un même patch.
Sinon c'est ok pour moi, merci d'avoir repris l'affaire.

#13

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

Paul Marillonnet a écrit :

Les deux patches 0001 et 0002 touchent successivement au même fichier de test data_authentic_export_site.json. J'ai l'impression que c'est des modifs de même nature et qu'il y a intérêt à les regrouper toutes dans un même patch.
Sinon c'est ok pour moi, merci d'avoir repris l'affaire.

C'est fait exprès pour qu'on puisse "lire" les changements aux données de test; sinon c'est juste impossible.

#14

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 0ca228fa967902aba5949038a5afc807fad838bf
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Wed Mar 4 14:31:36 2020 +0100

    python3: migrate authentic (#40407)

commit 5fe3fcb6670caed9042ec434d24bf09769bc8356
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Fri Mar 6 07:52:40 2020 +0100

    tests_authentic: format test data (#40407)
#15

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

#16

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF