-
Notifications
You must be signed in to change notification settings - Fork 9
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
Ajout du dispositif Revenu Solidarite Jeune de la métropole de Lyon #189
Conversation
1d152d2
to
dc26a77
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Une remarque pour le nom du fichier du rsj, les autres remarques sont des suggestions optionnelles.
Fonctionnel en local !
openfisca_france_local/metropoles/lyon/eligibilie_residentielle.py
Outdated
Show resolved
Hide resolved
label = "Éligibilité géographique de la métropole de Lyon" | ||
|
||
def formula(menage, period): | ||
return menage("menage_dans_epci_siren_200046977", period) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ca pourrait être bien de tester menage_dans_epci_siren_200046977
un peu à la manière de ce test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je suis un peu mitigé, le test en question test la epci_test_factory
. On peut effectivement lui reprocher de ne vérifier que cela fonctionne que pour une seule EPCI
.
Ici on ne fait qu'utiliser cette réforme, tester cette formule revient à tester la réforme epci_test_factory
et ce n'est pas vraiment le sujet ici.
Je trouve que la réforme est effectivement un peu légèrement testée mais je n'ai pas pris le temps de voir comment tester mieux et je ne suis pas convaincu par la valeur de dupliquer un test existant en changeant juste la valeur.
Je propose de créer un ticket pour investiguer la question en dehors de cette PR, qu'en penses-tu ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je propose de créer un ticket pour investiguer la question en dehors de cette PR, qu'en penses-tu ?
Oui bonne idée. J'avais pas compris d'où ça provenait lors de ma review, ça vaudrait effectivement le coup d'ajouter un petit test sur cette méthode dans une autre PR :)
openfisca_france_local/metropoles/lyon/revenu_solidaire_jeune.py
Outdated
Show resolved
Hide resolved
44aff62
to
b62f77e
Compare
b62f77e
to
3f50a98
Compare
3f50a98
to
e1d2890
Compare
Ticket qui explique les conditions
Added