Contrôle a posteriori : donner l'accès aux pièces justificatives des contrôles précédents aux employeurs

[tonial]

🤔 Pourquoi ?

Certaines pages n'étaient accessibles que pour les institutions mais le lien était présent dans la page des employeur
et le commentaire des la DDETS manquait

🏝️ Comment tester

Détourner http://127.0.0.1:8000/admin/users/user/511

• Contrôle à posteriori
• Historique
• Campagne de contrôle a posteriori du 01/01/2022 au 31/12/2022
• Revoir les 2 auto-prescriptions
  • -> Plus de 404 👍
• Revoir ses justificatifs
  • -> Présence du commentaire de la DDETS

[notion-workspace]

☠️ [C1] Bug : accès au détail des contrôles précédents

[francoisfreitag]

Ces parcours sont utilisés par la DDETS et par les employeurs.

La DDETS a un champ commentaire (label Raison d'une auto-prescription refusée), donc il ne faut pas lui réafficher ce qu’elle a déjà saisi.

On indique à la DDETS que le message s’affichera pour un auto-prescription refusée, mais en l’état, il s’affiche aussi pour une auto-prescription acceptée. On devriat soit changer le libellé (message accompagnant le contrôle de l’auto-prescription), ou n’afficher que si l’auto-prescription a été refusée.

Autrement ça m’a l’air bien !

[tonial]

Merci pour les retours !
J'ai repris les tests, et modifié la page pour que le comportement soit inchangé pour les DDETS, et n'afficher le commentaire que si le statut est refusé

[tonial]

@francoisfreitag j'ai ajouté un commit pour simplifier les paramètres passés itou/templates/siae_evaluations/includes/job_seeker_infos_for_siae.html

Je me suis rendu compte au passage qu'on utilisait parfois compute_state() ou compute_state_for_siae() alors qu'on est dans un template spécifique aux siae, j'ai donc opté pour n'utiliser que le second.

Par contre, en ajoutant la condition sur final_reviewed_at, et s'assurant que tous le template a systématiquement toutes les variables nécessaires le tests tests/www/siae_evaluations_views/test_siaes_views.py::SiaeJobApplicationListViewTest::test_state_hidden_with_submission_freezed_at cassait.
J'ai donc ajouté un and not evaluated_job_application.evaluated_siae.submission_freezed_at dans le template d'infos.
Est-ce que c'est une bonne idée ?
J'avais dans l'idée que ça aurait du être géré par compute_state_for_siae qui regarde la valeur de submission_freezed_at

[francoisfreitag]

Merci pour le dernier refactor. Il simplifie bien le code !

J'ai donc ajouté un and not evaluated_job_application.evaluated_siae.submission_freezed_at dans le template d'infos.
Est-ce que c'est une bonne idée ?

Presque, on garde le submission_freezed_at à la fin de la campagne, donc il faudrait aussi un and not evaluation_campaign.ended_at.

Rappel du métier :
Après que les SIAE aient soumis leurs documents, la DDETS les évaluent, mais on ne veut donner le résultat de l’évaluation qu’à l’issue de la phase d’évaluation.

Il existe pour cela la propriété evaluation_is_final, qui permet de n’afficher le résultat que si le contrôle est terminé. Mais elle ne prend pas en compte la période où on cache le résultat aux SIAE (submission_freezed_at).

Donc une fois que submission_freezed_at et tant que la campagne n’est pas terminée, on ne veut pas encore rentrer dans le {% elif evaluated_job_application.evaluated_siae.final_reviewed_at %}.

Ce qui m’amène à me demander si on ne devrait pas conditionner l’accès des employeurs aux deux vues que tu as rajouté au fait que les campagnes soient terminées ?
Si c’est le cas, on n’a plus besoin d’utiliser le template job_seeker_infos_for_siae.html et on peut mettre le bout de code suivant dans le template evaluated_job_application.html:

                {% if state == "ACCEPTED" or state == "SUBMITTED" %}
                    <span class="badge badge-sm rounded-pill text-nowrap bg-success text-white">Validé</span>
                {% else %}
                    <span class="badge badge-sm rounded-pill text-nowrap bg-danger text-white">Problème constaté</span>
                {% endif %}

[francoisfreitag]

Je ping Xavier qui aura peut-être aussi son avis sur la question.

[tonial]

Ça devient un serpent de mer ce carte :D

[francoisfreitag]

Tout le contrôle a posteriori est un serpent de mer, bienvenue à bord 😁

[xavfernandez]

J'aurais aussi tendance à penser que compute_state_for_siae devrait également gérer ce cas-là.

[xavfernandez]

Il faudrait que cet assertNotContains utilise une variable validée par un assertContains (comme le refusal_comment_txt)

[tonial]

Sauf que cette chaîne de caractère ne peut jamais être affichée pour les institutions...
Est-ce que je l'importe depuis SiaeEvaluatedJobApplicationViewTest.refusal_comment_text qui est dans un autre fichier ?

[xavfernandez]

Ça me semble bien oui.