[TECH] ♻️ Execute le rescoring par script en synchrone (PIX-17242)

[yaf]

🌸 Problème

Le rescoring par script est « over-engineeré ».
Certes, mais surtout il utilise toujours l'event dispatcher.

🌳 Proposition

Appeler le rescoring pas script en synchrone pour se detacher de l'event dispatcer.

🐝 Remarques

🤧 Pour tester

• Faire passer (ou trouver) une certification a un candidat et aller jusqu'a publier
• En base : regarder son dernier assessment-result et
  • Prendre une capture des data pour se souvenir du scoring
  • Modifier (ou dupliquer mais si vous dupliquez pensez a changer la table certification-courses-last-assessment-results) les donnees pour generer "un mauvais resultat" (comprendre par la: ce ne sont pas les donnees qui correspondent au scoring que le candidat aurait du avoir)
    • Recommandation : modifier a minima le pix score, et pourquoi pas le status
    • Verifier sur Pix App l'affichage de vos modifications
• Relever le certification-courses.id
• Lancer le script selon la procedure sur Confluence : api/scripts/certification/rescore-certifications.js sur ce certification-course
• Verifier
  • Pas d'erreur du script
  • Que le assessment-result est revenu a l'etat d'avant vos modifications, mis a part les dates (qui seront celles du script)
  • Que le last-assessment-result pointe bien vers ce nouveau assessment-results
  • Verifier sur Pix App que l'affichage est revenu a la norme

[pix-bot-github]

Une fois les applications déployées, elles seront accessibles via les liens suivants :

• App (.fr)
• App (.org)
• Orga (.fr)
• Orga (.org)
• Certif (.fr)
• Certif (.org)
• Junior
• Admin
• API
• API MaDDo
• Audit Logger

Les variables d'environnement seront accessibles via les liens suivants :

• scalingo front
• scalingo api
• scalingo api-maddo
• scalingo audit-logger

[Steph0]

je n'ai pas fais la tech func, je la ferai demain si personne ne l'a faite d'ici la

[Steph0]

Vu avec @yaf on fait la revue func ensemble jeudi pour que je lui montre 😊

[pix-bot-github]

Une fois les applications déployées, elles seront accessibles via les liens suivants :

• App (.fr)
• App (.org)
• Orga (.fr)
• Orga (.org)
• Certif (.fr)
• Certif (.org)
• Junior
• Admin
• API
• API MaDDo
• Audit Logger

Les variables d'environnement seront accessibles via les liens suivants :

• scalingo front
• scalingo api
• scalingo api-maddo
• scalingo audit-logger

[pix-bot-github]

Une fois les applications déployées, elles seront accessibles via les liens suivants :

• App (.fr)
• App (.org)
• Orga (.fr)
• Orga (.org)
• Certif (.fr)
• Certif (.org)
• Junior
• Admin
• API
• API MaDDo
• Audit Logger

Les variables d'environnement seront accessibles via les liens suivants :

• scalingo front
• scalingo api
• scalingo api-maddo
• scalingo audit-logger

[pix-bot-github]

Une fois les applications déployées, elles seront accessibles via les liens suivants :

• App (.fr)
• App (.org)
• Orga (.fr)
• Orga (.org)
• Certif (.fr)
• Certif (.org)
• Junior
• Admin
• API
• API MaDDo
• Audit Logger

Les variables d'environnement seront accessibles via les liens suivants :

• scalingo front
• scalingo api
• scalingo api-maddo
• scalingo audit-logger

[Steph0]

Test func ⛔

Score avant modification

Je modifie le score

Test KO
node scripts/certification/rescore-certifications.js --file certif.csv

{"level":50,"time":1743697072699,"pid":38,"hostname":"pix-api-review-pr11913-one-off-248","event":"RescoreCertificationScript","err":{"type":"Error","message":"event must be one of types ChallengeNeutralized, ChallengeDeneutralized, CertificationJuryDone, CertificationCourseRejected, CertificationCourseUnrejected, CertificationCancelled, CertificationRescoredByScript, CertificationUncancelled","stack":"Error: event must be one of types ChallengeNeutralized, ChallengeDeneutralized, CertificationJuryDone, CertificationCourseRejected, CertificationCourseUnrejected, CertificationCancelled, CertificationRescoredByScript, CertificationUncancelled\n    at checkEventTypes (file:///app/src/shared/domain/events/check-event-types.js:10:11)\n    at handleCertificationRescoring (file:///app/src/shared/domain/events/handle-certification-rescoring.js:44:3)\n    at Object.handleCertificationRescoring (file:///app/src/shared/infrastructure/utils/dependency-injection.js:4:20)\n    at RescoreCertificationScript.handle (file:///app/scripts/certification/rescore-certifications.js:36:27)\n    at RescoreCertificationScript.run (file:///app/src/shared/application/scripts/script.js:57:18)\n    at ScriptRunner.execute (file:///app/src/shared/application/scripts/script-runner.js:75:20)\n    at async file:///app/scripts/certification/rescore-certifications.js:43:1"},"msg":"event must be one of types ChallengeNeutralized, ChallengeDeneutralized, CertificationJuryDone, CertificationCourseRejected, CertificationCourseUnrejected, CertificationCancelled, CertificationRescoredByScript, CertificationUncancelled"}

[Steph0]

Y'a un soucis lors du test func

[Steph0]

Novueau test func ✅

{"level":30,"time":1743756358555,"pid":41,"hostname":"pix-api-review-pr11913-one-off-2772","event":"RescoreCertificationScript","msg":"Script execution successful."}

on a bien un nouvel assessmentresult

[Steph0]

Suite au fix c'est tout bon niveau test func !