[TECH] Eviter de re-déclencher les calculs pour déterminer la prochaine épreuve dans assessments/:id/next si la dernière épreuve proposée n'a pas encore été répondue

[laura-bergoens]

🌸 Problème

la route api/assessments/:id/next (la fameuse GetNext) est une des routes les plus appelées.
Certains de ces appels ont lieu alors qu'il n'y a pas besoin de recalculer quelle sera la prochaine épreuve à poser, car la dernière posée n'a pas encore été répondue.
Les cas qui me viennent en tête, et qui sont fréquents :

• Cas de rechargement de page
• Cas de "j'arrête pour aujourd'hui je reprends demain"
• Le front aujourd'hui fait systématiquement deux appels à GetNext avant d'afficher l'épreuve (ce cas là est pas soluble en une aprem tech il faut un peu de temps pour se pencher sur le sujet)

🌳 Proposition

Il se trouve que, lorsqu'on fini de recalculer quelle sera la prochaine épreuve à poser, on enregistre l'id de la prochaine épreuve dans la table assessments, sous lastChallengeId.
On peut utiliser cet id pour déterminer si, en regardant les réponses apportées dans le cadre de l'assessment en cours, cette épreuve a déjà été répondue ou pas. A défaut de réponse, on sait qu'il n'y a pas besoin de redéclencher tout le GetNext (qui est par ailleurs déterministe et qui retournera de toute façon le même ID). Il suffit de retourner l'épreuve pointée par l'id dans lastChallengeId.

Cette PR contient :

• L'ajout d'une petite garde en début de GetNext pour ne pas déclencher les calculs GetNext si l'assessment n'est pas en cours.
• L'ajout de la vérification de si la dernière épreuve posée a déjà été répondue ou pas, auquel cas on retourne directement l'épreuve sans redéclencher tous les calculs

🐝 Remarques

🤧 Pour tester

Non régression de toutes les modalités d'assessment (sauf PixJunior).
Toujours deux appels API consécutifs mais normalement le deuxième appel ne devrait pas refaire tous les calculs.
Une manière de vérifier en local pourrait être de lancer son serveur en mode DEBUG=knex:* et de constater que sur le deuxième appel de GetNext on n'a pas remonté la terre entière dans la BDD

[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

[yaf]

Trop bien, merci ! 🚀

[VincentHardouin]

Je suis d'accord avec le fond, mais j'ai une petite inquiétude, peut-être pas justifiée, sur le fait qu'un challenge soit périmé entre temps ou autres.

[laura-bergoens]

Je suis d'accord avec le fond, mais j'ai une petite inquiétude, peut-être pas justifiée, sur le fait qu'un challenge soit périmé entre temps ou autres.

Bien vu. Je peux ajouter cette vérification je pense que c'est pas cher