[GEN-557] Candidature : préparation à la certification AAH #5481
Conversation
celine-m-s
changed the title
celine-m-s
force-pushed
the
celinems/aah
branch
3 times, most recently
from
cf23844 to
d23e09b
Compare
celine-m-s
force-pushed
the
celinems/aah
branch
from
d23e09b to
7dc5dea
Compare
celine-m-s
added
the
1-recette-jetable
|
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
There was a problem hiding this comment.
Yaay! Un nouveau critère sera bientôt certifié 😀
Cette PR mériterait d’être découpée.
La revue est par conséquent un peu éparpillée et je trouve dommage de retenir des améliorations qui n’ont pas grand chose à voir avec l’AAH le temps de gérer l’intégration de l’AAH.
tests/eligibility/test_iae.py
Outdated
| diagnosis = iae_eligibility_with_criteria_factory(criteria_kind=CRITERIA_KIND) | ||
| admin_criterion = AdministrativeCriteria.objects.get(kind=CRITERIA_KIND) | ||
| diagnosis.administrative_criteria.add(admin_criterion) |
There was a problem hiding this comment.
On pourra même ajouter ce cas aux nouvelles factories, pour passer une liste de critères ?
There was a problem hiding this comment.
Fait dans le dernier commit. Désolée par avance car plusieurs certifiable=True se sont glissés dans le commit. J'ai tenté de découper avec des git add -p mais les changements étaient trop petits. Je sais que ça va te tendre et je comprends. 😬
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
celine-m-s
force-pushed
the
celinems/aah
branch
from
a923e0c to
edb21f0
Compare
celine-m-s
changed the title
|
PR découpée en deux : celle-ci et la #5584 . |
Having `GEIQEligibilityDiagnosisFactory.with_certifiable_criteria` choose randomly from a list of criteria is unpredictable and can cause flaky tests. Better use parametrized tests.
…to api_particulier.certify_criteria
celine-m-s
force-pushed
the
celinems/aah
branch
from
edb21f0 to
3d6dc7c
Compare
|
Voilà ! J'ai pris en compte tous tes retours @francoisfreitag . Je travaille demain (mercredi) mais n'hésite pas à tout squasher si tu veux MEP jeudi ou vendredi. Je peux aussi passer une tête le soir pour MEP par moi-même. |
🤔 Pourquoi ?
A l’instar du critère BRSA on veut interroger l’API Particuliers pour certifier le critère AAH.
AAH est un critère pour lequel on demande un justificatif de < de 3 mois.
🍰 Comment ?
Règles identiques au critère RSA.
🚨 À vérifier
🏝️ Comment tester ?
En tant que SIAE, réaliser une auto-prescription avec un critère AAH.
💻 Captures d'écran