Correctif de la mise à jour des scores de la recherche employeur

[dejafait]

🤔 Pourquoi ?

En manipulant le script en local pour creuser l'évolution en cours de la recherche employeur, j'ai réalisé qu'on avait un gros souci avec l'algorithme actuel. Des entreprises non pertinentes (grand nombre de candidatures) se retrouvent en toute première page en prod et accumulent des candidatures. Probablement car quand leur nouveau score devient None (si elle n'ont plus aucune fiche de poste après en avoir eu) ce None n'est jamais propagé en db et l'ancien score reste ad vitam eternam.

Voir les exemples d'entreprises non pertinentes dans https://www.notion.so/plateforme-inclusion/Tri-des-employeurs-fiches-de-postes-137e8fa5c35b804b95c7f05f68f69b81

🍰 Comment ?

J'ai commencé par rajouter du test cassant (TDD). Ca repasse au vert quand j'enlève toute la clause exclude. Je propose ici une solution moins radicale pour continuer à ne mettre à jour que ce qui doit l'être, mais j'avoue que je ne comprend pas complètement ni la cause, ni ma solution.

[xavfernandez]

Bien vu 👍

L'explication vient du fait que Q(job_app_score=F("computed_job_app_score")) se traduit en SQL en job_app_score=<gros CASE calculant computed_job_app_score> AND job_app_score IS NOT NULL mais autorise quand même la comparaison quand <gros CASE calculant computed_job_app_score> vaut NULL.
Donc ton fix évite bien la comparaison avec NULL qui empêche actuellement la ligne d'être mis à jour:

itou=# select count(*) from companies_company where job_app_score is null;
 count 
-------
  3853
(1 row)

itou=# select count(*) from companies_company where NOT job_app_score=null;
 count 
-------
     0
(1 row)

itou=# select count(*) from companies_company where job_app_score=null;
 count 
-------
     0
(1 row)