Technique: Correction d'une regression apportée par #5541

[tonial]

🤔 Pourquoi ?

Correction de #5541 (https://inclusion.sentry.io/issues/28983753/?project=4508410589020240)

Indiquez le problème que nous sommes en train de résoudre et les objectifs métiers ou techniques qui sont visés par ces changements.

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

• Mettre à jour le CHANGELOG_breaking_changes.md ?
• Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

[rsebille]

Le changement dans les tests me fait dire qu'effectivement maintenant on teste le cas mais j'ai du mal à comprendre le self.steps.current == self.steps.first qui me semble ultra-spécifique, j'ai l'impression qu'avec ça on ne gère pas le cas où la personne passe la 1ère étape d'un wizard et essaye ensuite d'aller à une étape suivante.

[tonial]

Damn... en fait il y a 2 problèmes

• essayer d'aller à une étape random sans commencer par la première (c'est le self.steps.current == self.steps.first)
• essayer d'aller à une mauvaise étape. le reste du temps (step_url != self.steps.current sauf quand step_url == "done" et que self.steps.current == last)

[xavfernandez]

🍿 Moi j'ai mon avis sur les wizard formtools 🧌

[tonial]

@rsebille j'ai améliorer le code pour mieux gérer les cas.
Je vais passer à la solution maison de @xavfernandez qui marche mieux dans un second temps