La mises à jour de valeurs précises calculées dans les tests unitaires
n'est pas une solution satisfaisante. Je le fait dans ce commit mais à
terme je pense qu'il faudra déléguer intégralité de ces tests sur des
valeurs précises aux tests de régressions `yarn run test-regressions`
qui peuvent être mis à jour facilement et conserver les tests unitaires
pour des cas où la valeur précise calculée n'importe pas.
J'ai essayé de supprimer complétement la variable entreprise . effectif
(numérique) pour la remplacer par une variable seuil qui soit la "source
de vérité" utilisée dans les formules de calcul. Malheureusement il y a
quelques endroits où persistent des anciens seuil, et où la valeur
précise de l'effectif est utilisée (taxe sur les salaires par exemple,
bien que l'utilisation dans cette formule soit une approximation assez
grossière).
En attendant d'améliorer notre gestion des variables à l’échelle de
l'entreprise, l’implémentation retenue permet de simplifier la saisie
(on clique sur un seuil plutôt que de saisir une effectif précis à
l'unité près) sans cassé les formules existantes.
Gros changements en perspective :
- Supprime la notion de période, au bénéfice de celle d'unité
(`période : mensuelle` devient `unité: €/mois`)
- Améliore les rapports d'erreur avec des messages plus clair
- Ajoute un avertissement lorsque des types ne sont pas compatible
- Ajoute la conversion automatique d'unité dans le moteur
- Ajoute une notion d'unité par défaut de la simulation,
c'est l'unité vers laquelle les règles qui ne spécifient pas
d'unité seront converties
- Ajoute une notion d'unité par défaut des règles, qui spécifie
l'unité de la règle qui prévaut lorsque qu'il n'y a pas
d'unité par défaut de la simulation (utile pour les question ou
pour s'assurer du bon type d'une règle)
implémentation est hacky et il faudrait faire une refacto du système
de distribution pour pouvoir l'intégrer correctement sur l'ensemble des
simulateurs.
On utilise pas tout à fait les même composants que sur les simulateurs
salarié et indépendants, mais il sera certainement possible de les faire
converger.
Et pour faire passer les tests jest au passage.
A noter : il faudra vraiment se pencher sur la notion d'applicable / non applicable
Une variable non applicable a une valeur '0' / 'false', mais une variable
à false n'est pas considérée comme non applicable. Je pense qu'il serait avantageux
de simplifier le modèle en introduisant une symetrie entre applicable si et la valeur
d'une variable.
Johan Girod
Maxime Quandalle