Correction et refacto de l'affichage des questions

[liliced]

#3417

• Refactorisation du composant Banner

  • déplacement dans le dossier Simulation
  • renommage cohérent
  • déplacement de la gestion de la disparition après la 1ère étape dans les parents

• Refactorisation de SelectSimulationYear

  • déplacement dans le dossier Simulation
  • renommage cohérent

• Refactorisation de SimulationPréremplieBanner

  • déplacement dans le dossier Simulation
  • ne s'affiche que lorsqu'il n'y a pas la sélection d'entreprise (peu pertinent sinon)
  • reste affiché même après le début de la simulation (car toujours pertinent)

• Refactorisation de PreviousSimulationBanner

  • déplacement dans le dossier Simulation
  • déplacement de la gestion de la disparition après la 1ère étape dans le parent Simulation

• Ajout d'une variable exceptions à firstStepCompletedSelector, transmise par Simulation et remplie au besoin par le simulateur (cessation d'activité, dividendes, indépendant, lodeom et RGCP)

• Correction de 2 avertissements console (prop inconnue et absence de key)

• Ajout de tests sur les simulateurs génériques

• Ajout de paramètres aux tests des simulateurs génériques

[netlify]

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

Name	Link
🔨 Latest commit	91b7ed8
🔍 Latest deploy log	https://app.netlify.com/sites/nostalgic-mahavira-52b682/deploys/67af0a54b0334300087615eb

[github-actions]

🚀 La branche est déployée !

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

[JalilArfaoui]

Comme on risque de multiplier les options, préférer peut-être un objet options :

export const runSimulateurTest = (
	simulateur: Simulateur,
	{ avecCharges = false }: RunSimulateurOptions,
	beforeAction = () => {}
) => {

ce qui facilite aussi la lecture à l’utilisation en étant explicite :

runSimulateurTest('profession-liberale/medecin', { avecCharges: true})

[liliced]

Pourquoi ne pas mettre beforeAction dans les options aussi ?

[JalilArfaoui]

Oui, tu as raison, on peut considérer ça comme une option également !

[glopezrios]

C'est tout bon, les questions sont bien revenues sur le simulateur pour artiste-auteur!

[JalilArfaoui]

Je préférerais un truc comme firstStepCompletedRules ou firstStepCompletedConditions … là on pose l’idée que l’usage de cette prop doit être exceptionnel, ce qui ne me plait pas, même si c’est vrai pour le moment…

[liliced]

D'accord sur le principe mais l'inconvénient avec Rules ou Conditions c'est que c'est moins clair que cela représente "les règles qui ne rentrent pas en compte dans la détermination de la 1ère étape", contrairement à Exceptions.
Je vais y réfléchir (et demander à une IA d'y réfléchir pour moi 😄)

[JalilArfaoui]

Mais justement, pour moi ce qu’on envoie c’est juste «les règles à regarder pour savoir si on a fait la première étape» … et si on les envoie pas on a un comportement par défaut … mais elles n’ont pas nécessairement à être exceptionnelles, non ?

[JalilArfaoui]

Je suggère <React.Fragment key={year}> pour ne pas avoir à modifier le DOM.

[liliced]

Ah cool, je ne connaissais pas !

[JalilArfaoui]

<> est un sucre syntaxique pour <React.Fragment> …

[JalilArfaoui]

J’ai enfin fait le tour et tout relu @liliced … désolé pour le délai 🙏🏾

J’ai mis juste quelques commentaires mineurs !