Certification : Déclencher l’appel à l’API Particulier au moment de la création d’un diagnostic et pour les prescripteurs habilités [GEN-2250]

[rsebille]

🤔 Pourquoi ?

La certification actuelle se fait uniquement après embauche et uniquement pour les diagnostics employeurs et Geiq.
On souhaiterait certifier avant l’embauche pour tous les diagnostics d’éligibilité peu importe l’auteur (Employeurs, Geiq ou Prescripteur habilité)

🍰 Comment ?

Deux parties :

1. Afficher les informations de certifications pour les prescripteurs habilités
2. Certifier les critères lors de la création d'un diagnostique

Points de discussion

1. On conserve l’appel API au moment de l’embauche uniquement si l’appel API n’a pas pu être fait au moment du diag

Je suis en hésitation sur le fait de l'implémenter, ce n'est en soi pas compliqué car c'est juste un filtre sur le queryset dans tasks.certify_criteria() mais je me dit que ça fait un système de rattrapage en attendant le rafraîchissement à 92 jours pour les diagnostics des PH.

#### 2. SelectedAdministrativeCriteriaQuerySet.with_is_considered_certified

Actuellement si hiring_start_at=None alors is_certified est toujours False, le commentaire indique c'est voulu mais lors de mes tests j'ai trouvé le comportement extrêmement déconcertant dans le nouveau fonctionnement. Par exemple :
- Je suis PH, je fait le diag et on me dit "Non certifié" alors que les critères ont bien été certifiés et validés. Par contre dès qu'une date de contrat est présente sur la candidature alors on affiche "Certifié", même si celle-ci a depuis été annulée par exemple.
- Même logique coté SIAE où avant l'embauche on me dit "Non certifié" et tout de suite après "Certifié", auparavant on n'affichais pas le badge si le diagnostic venait d'un PH mais maintenant oui.
=> Comportement modifié dans #6126

#### 3. Le badge de certification n'est pas affiché dans le bloc "Diagnostic IAE/GEIQ" de la page de détail du candidat

C'est le comportement actuel mais je m'attendais à le voir apparaître ici aussi, surtout que maintenant on certifie au moment du diagnostique et pas juste à l'embauche.
=> Planifié dans un autre ticket

[notion-workspace]

🌿 [Certification élargie 1/2] Déclencher l’appel à l’API Particulier au moment de la création d’un diagnostic [PILOTAGE / EMPLOIS]

[github-actions]

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

[francoisfreitag]

Ohh les beaux tests tout bien organisés. 🤩
Merci d’avoir pris le temps de faire le nettoyage 👍

[francoisfreitag]

def iae_and_geiq(cls):

Je trouve common un peu flou 🤷

Pour l’implem, ce serait peut-être pas mal d’avoir des sets geiq et iae, pour pouvoir faire iae - geiq?

[rsebille]

Le but est justement d'être un peu flou afin de dire "Ce sont ceux disponible partout", le .iae_and_geiq() poussant à mon sens la création de nouvelles combinaisons plus qu'autre chose, donc ça me va bien de rajouter un .for_iae() et.for_gieq() si ça sert ailleurs dans le code 🤔.

[francoisfreitag]

« Ce sont ceux disponibles partout »... = dans l’IAE et les GEIQ. Lorsqu’arrivera un nouveau dispositif, les common seront à revoir et la certification aussi. 🤷

[rsebille]

Oui, mais il n'y aura pas besoin de repasser sur le code appelant (je pense surtout aux tests ;)) si lors de l'utilisation des méthodes on se pose les bonnes questions.
Et actuellement je ne pense pas qu'on ai besoin d'avoir IAE+GEIQ car soit on test 1 dispositif en particulier soit on veux le sous-ensemble commun.

[francoisfreitag]

😂
Pas sûr qu’on ait besoin de toute cette couverture, mais au moins c’est couvert !

[rsebille]

Oui 😁, c'est mon petit bonbon ce test :P.