chore: met à jour Publicodes -> 1.5.1

[JalilArfaoui]

Quelques ajustements ont été nécessaires

[netlify]

✅ Deploy Preview for nostalgic-mahavira-52b682 canceled.

Name	Link
🔨 Latest commit	16b790e
🔍 Latest deploy log	https://app.netlify.com/sites/nostalgic-mahavira-52b682/deploys/673d1e0676843e000836b838

[github-actions]

♿ Checklist accessibilité :

Quelques points à vérifier pour maintenir une bonne accessibilité du site :

• les labels des boutons et des liens sont descriptifs
• un aria-label est ajouté pour préciser l'action d'un bouton ou d'un lien pour les lecteurs d'écran
• les couleurs du texte est suffisamment constratée par rapport à la couleur de background
• la fonctionnalité ajoutée est utilisable en navigant au clavier et avec un lecteur d'écran
• les changements de contexte sont évités ou au minimum annoncés

[github-actions]

🚀 La branche est déployée !

• mon-entreprise : https://3117--mon-entreprise.netlify.app
• mycompanyinfrance : https://3117-en--mon-entreprise.netlify.app
• storybook : https://3117--mon-entreprise.netlify.app/dev/storybook/ (version EN)
• API : https://3117--mon-entreprise.netlify.app/api/v1/doc/

[johangirod]

@JalilArfaoui l'assistant au choix du statut crash et il y a beaucoup de changements dans les tests de non regression, ça inspire pas confiance.

En ajoutant la solution proposée sur mattermost, ça ne fonctionnait pas ?

Edit : ça pourrait être pas mal d'ajouter des tests E2E sur le parcours de choix du statut d'ailleurs :)

[JalilArfaoui]

@JalilArfaoui l'assistant au choix du statut crash et il y a beaucoup de changements dans les tests de non regression, ça inspire pas confiance.

Je viens d’essayer, pas eu de crash sur l’assistant choix de statut… tu pourrais me montrer comment reproduire ? Ça peut être un bug présent en prod 🙈

Pour les snapshots de non-régression, je les ai inspectés fichier par fichier, et je me suis dit que c’était normal qu’il puisse y avoir des différences < 1% , notamment à cause de publicodes/publicodes@ed5b4e9 par exemple…

Toi, ça te parait anormal comme diff @johangirod ?

Il y a aussi des diff dues à quelques ajustements de ma part, comme ne pas afficher la notification de franchise de TVA quand dirigeant = non

En ajoutant la solution proposée sur mattermost, ça ne fonctionnait pas ?

Je n'ai pas essayé, car j’étais déjà parti sur le fait d’essayer de faire les choses « bien ». Toi, ça te parait une meilleure façon d’avoir un choix possible "À définir" dans une règle à choix multiple ? Ce n'est pas bizarre ?

Edit : ça pourrait être pas mal d'ajouter des tests E2E sur le parcours de choix du statut d'ailleurs :)

👍🏾

[johangirod]

Il y a aussi une regression sur le simulateur de coût de création
-> https://3117--mon-entreprise.netlify.app/simulateurs/cout-creation-entreprise

Je n'ai pas essayé, car j’étais déjà parti sur le fait d’essayer de faire les choses « bien ». Toi, ça te parait une meilleure façon d’avoir un choix possible "À définir" dans une règle à choix multiple ? Ce n'est pas bizarre ?

Un poil bizarre, mais explicable (cf mattermost). En fait, je me souviens d'avoir pas mal galéré avant d'arriver à cette solution. Toutes les autres que j'avais essayées aboutissaient à des comportements non voulus sur un des simulateurs concernés (cout de création / choix du statut / indépendant).

Il est peut-être possible de faire mieux, mais cela mériterait de muscler les tests de non régression pour être sûr de ne pas louper quelque chose (notamment au niveaux des questions posées).

Voici comment reproduire le bug sur choix du statut :

Peek.12-09-2024.17-10.mp4

[liliced]

Le bug sur le choix de statut n'est pas en prod. L'erreur affichée est la suivante :

SituationError : [ Erreur lors de la mise à jour de la situation ] ➡️ Dans la règle "entreprise . associés" ✖️ La valeur 'multiple' ne fait pas parti des possibilités listées dans la base de règles.

[liliced]

Une question bonus s'est glissée dans le simulateur salarié :

Mais elle n'apparaît pas dans la liste "Modifier mes réponses". Le seul indice c'est que le nombre total de questions n'est pas le même qu'en prod.

[liliced]

Un peu répétitif tout ce code. Pour des tests c'est moins important mais il n'était peut-être pas nécessaire de répéter toutes les étapes à chaque scénario, ou alors en les factorisant.