API: Ajout d'une API pour rechercher les candidatures pour les CD [GEN-1651] #5588
Conversation
…tmentTokenAuthentication
leo-naeka
added
1-recette-jetable
|
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
|
Comme je vous disais, j'envisage de modifier le dernier test pour instancier des objets stables et comparer avec un snapshot plutôt que faire 36 assertions. Mais je veux bien déjà votre avis sur le reste 🙏 |
| ) | ||
| statut = serializers.ChoiceField( | ||
| source="state", | ||
| choices=JobApplicationState.choices, |
There was a problem hiding this comment.
on ne veut pas mettre le label en valeur dans l'api (pour avoir du texte en français) ?
There was a problem hiding this comment.
Vu avec Zohra, pour tous les enums on préfère la valeur (stable dans le temps) que le label.
J'avais proposé de rajouter les deux. Ils auront l'info dans la doc : https://c1-review-leo-job-seekers-api.cleverapps.io/api/v1/redoc/#tag/candidatures/operation/candidatures_recherche
Par ailleurs dans certains cas (type prescripteur/employeur, on ne voulait que la valeur : l'acronyme)
| source="to_company.email", | ||
| label="Adresse e-mail de l’entreprise", | ||
| ) | ||
| orientation_emetteur_type = serializers.ChoiceField( |
There was a problem hiding this comment.
il faut sans doute distinguer prescripteur habilité et orienteur, non ?
There was a problem hiding this comment.
Vu avec Zohra on n'affiche que les 3 types de base. J'avais proposé de rajouter un autre attribut (bool) pour indiquer si le prescripteur était habilité ou non, a priori pas nécessaire.
| assert len(results) == 3 | ||
|
|
||
| first_application = results[-1] # Reversed sorting | ||
| assert first_application["cree_le"] == _str_with_tz(self.job_application.created_at) |
There was a problem hiding this comment.
à défaut d'un snapshot, si c'est trop pénible de fixer les donnée, du peux tester l'égalité de first_application avec un dictionnaire qui contient les données structurées, ce sera peut être plus lisible
leo-naeka
force-pushed
the
leo/job-seekers-api
branch
from
00f287c to
f3c1942
Compare
| from itou.api.models import DepartmentToken | ||
|
|
||
|
|
||
| class JobApplicationSearcAPIPermission(IsAuthenticated): |
There was a problem hiding this comment.
| class JobApplicationSearcAPIPermission(IsAuthenticated): | |
| class JobApplicationSearchAPIPermission(IsAuthenticated): |
| "cree_le": "2025-02-06T14:48:06.595973+01:00", | ||
| "statut": "obsolete", | ||
| "candidat_nom": "Durand", | ||
| "candidat_prenom": "Nathalie", |
There was a problem hiding this comment.
Supprimer une ligne ici ne casse rien: il faudrait idéalement qu'il y ait un test qui vérifie que les exemples correspondent au schéma ?
🤔 Pourquoi ?
🍰 Comment ?
En mettant une API à disposition de leur éditeur de logiciel
🚨 À vérifier