Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revalorisation du montant de l'Allocation de soutien Familial (ASF) au 1er avril 2018 #1194

Merged
merged 5 commits into from
Oct 29, 2018

Conversation

mtifarine
Copy link
Contributor

@mtifarine mtifarine commented Oct 26, 2018

  • Changement mineur.
  • Périodes concernées : à partir du 01/04/2018.
  • Zones impactées : tests/formulas/asf_2018_04..
  • Détails :
    • Ajoute des tests pour la revalorisation de l'allocation de soutien familial (ASF) au 1er avril 2018.

Fixes #1119

@mtifarine mtifarine added the contrib:msa Identification des sujets MSA label Oct 26, 2018
@mtifarine mtifarine changed the title Msa asf Revalorisation du montant de l'Allocation de soutien Familial (ASF) au 1er avril 2018 Oct 26, 2018
@mtifarine mtifarine requested a review from Morendil October 26, 2018 14:34
- id: "enfant1"
age: 14
output_variables:
asf: 115.30
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

D'après la référence ce montant est en fait de 115.87

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Par ailleurs on a déjà ce test dans tests/formulas/asf.yaml donc il me semble qu'il n'est pas pertinent

output_variables:
asf: 115.30

- name: "Cas N°2 famille ayant droit à l'ASF - Parent isolé avec deux enfant"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A contrario ce test est intéressant puisqu'on n'a pas encore testé la logique ASF = ASF par enfant * nombre d'enfants, par contre ça me semblerait logique, soit qu'on le place dans le même fichier que les précédents (tests/formulas/asf.yaml), soit qu'on regroupe les deux tests de 2018 dans ce nouveau fichier.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Et donc dans ce cas n°2 il convient de corriger le montant (à 231,74)

@@ -0,0 +1,28 @@
- name: "Famille ayant droit à l'ASF - 2015"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ca me va bien comme ça, avec un fichier par période - dans ce cas il faut aussi supprimer le fichier d'origine tests/formulas/asf.yaml qui fait doublon.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai pas doublé le fichier d'origine tests/formulas/asf.yaml, je l'ai juste renommé tests/formulas/asf_2015_01.yaml.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK je vois ! Tu peux utiliser la commande "git mv" pour renommer afin que Git s'y retrouve. En tout cas c'est tout bon maintenant.

@Morendil
Copy link
Contributor

@mtifarine Je prends la mise à jour du Changelog, rebase et merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contrib:msa Identification des sujets MSA kind:evolution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants