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

Ajoute un nouveau test pour l'ASI #1168

Merged
merged 4 commits into from
Oct 17, 2018
Merged

Ajoute un nouveau test pour l'ASI #1168

merged 4 commits into from
Oct 17, 2018

Conversation

aguillouzouic
Copy link
Contributor

@aguillouzouic aguillouzouic commented Oct 16, 2018

  • Changement mineur.
  • Périodes concernées : toutes.
  • Zones impactées : tests/formulas/asi.yaml
  • Détails :
    • Ajout de tests pour l'ASI afin de vérifier que la formule prend bien en compte la dégressivité au delà d'un seuil.

Ces changements (effacez les lignes ne correspondant pas à votre cas) :

  • Corrigent ou améliorent un calcul déjà existant.

Quelques conseils à prendre en compte :

Fix #1073

asi: 311.9


- name: "Cas test de l'ASI pour vérifier que dégressivité est bien codée (3)"
Copy link
Contributor

Choose a reason for hiding this comment

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

J'ai du mal à comprendre l'intention derrière les cas 3 et 4, c'est un seul individu en couple ?…

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Les cas 3 et 4 sont pour un individu bénéficiaire en couple, afin de vérifier que le montant de l'aide individuelle n'est pas différent entre couples mariés et non-mariés (puisque c'est le cas pour le calcul du plafond de ressources de l'ASPA et que les deux aides sont liées).

@Morendil
Copy link
Contributor

Salut @aguillouzouic ! Merci pour cette contribution.

J'ai du mal à voir le lien qui est fait dans le descriptif de la PR avec #1073 et notamment je ne vois pas comment l'ajout de ce seul test corrige l'issue; est-ce que ces paramètres ont déjà été intégrés et il n'y a plus rien à faire ? (Auquel cas l'ajout d'un test a bien sûr de la valeur, et on peut le décorréler de la fermeture de l'issue avec un simple commentaire "rien à corriger".) Est-ce que d'autres évolutions vont venir s'ajouter à cette PR ? Dans ce cas il vaut mieux attendre qu'elle soit complète avant de mettre à jour le CHANGELOG.

Merci de tes lumières là dessus. :)

@aguillouzouic
Copy link
Contributor Author

Bonjour !
Par rapport à #1073, il n'y avait en définitive pas d'erreur dans le calcul de l'ASI, les tests ont simplement servi à s'en assurer.
Je pense qu'on peut donc en effet ajouter "rien à corriger" sur l'issue, et il n'y aura pas d'ajouts supplémentaires à la PR.

@Morendil
Copy link
Contributor

Alors c'est OK pour moi, merci !

@aguillouzouic aguillouzouic merged commit 6176cee into master Oct 17, 2018
@aguillouzouic aguillouzouic deleted the test-asi branch October 23, 2018 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants