PASS IAE: amélioration de la règle autorisant ou non une prolongation - #6872
Conversation
francoisfreitag
left a comment
francoisfreitag
left a comment
There was a problem hiding this comment.
Je ne comprends pas la remarque :
Since IS_OPEN_TO_PROLONGATION_BOUNDARIES_MONTHS_BEFORE_END = 7 months is smaller than WAITING_PERIOD_YEARS = 2 years, it should work fine.
?
|
Dans le cas contraire (si on pouvait prolonger un pass après son délai de carence), l'utilisation de latest_approval poserait soucis car cette property ne renvoie pas de PASS après leur délai de carence... |
| assert approval.number_with_spaces == expected | ||
|
|
||
| def test_is_last_for_user(self): | ||
| def test_can_be_prolonged(self): |
There was a problem hiding this comment.
Dans ce test on peut peut-être garder les assertions à la fin et faire se terminer approval1 il y a moins de 7 mois, pour bien vérifier que can_be_prolonged prend en compte les 2 pass ?
There was a problem hiding this comment.
En fait les 7 mois, c'est avant la date de fin. Un PASS expiré ne peut plus être prolongé par l'utilisateur. Mais j'ai tout de même fait la modif 👍
There was a problem hiding this comment.
Ah oui je me suis emmêlé les pinceaux. Dans le cadre de cette PR le PASS le plus récent forcément est désactivé 👍
it is cheap and might avoid extra queries
|
J'ai supprimé la remarque sur IS_OPEN_TO_PROLONGATION_BOUNDARIES_MONTHS_BEFORE_END & WAITING_PERIOD_YEARS car effectivement elle n'avait pas de sens. |
0fb1476 to
b344960
Compare
With duplicate job seekers it is possible for a job seeker to end up with two approvals. And thanks to suspension/prolongation it is also possible to have a "previous" (start_at sooner) approval still active while the "is_last_for_user" approval being not "open_to_prolongation" anymore. Instead, only allow the latest_approval to be prolonged.
b344960 to
da6597e
Compare
3 participants
🤔 Pourquoi ?
Avec les doublons de candidat, il est possible pour un candidat d'avoir 2 PASS et d'avoir l'ancien PASS encore actif (via des suspensions/prolongations) tandis que le PASS plus récent n'est plus actif.
Dans ce cas, le PASS actif ne pouvait pas être prolongé. Cette PR corrige ce cas.
🍰 Comment ?
🚨 À vérifier
🏝️ Comment tester ?
💻 Captures d'écran