-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BUGFIX] Gérer une épreuve qui est périmée en cours de test de certification (PIX-14819). #10403
Merged
pix-service-auto-merge
merged 3 commits into
dev
from
pix-14819-bugfix-challenge-not-found-during-v3-certif
Oct 24, 2024
Merged
[BUGFIX] Gérer une épreuve qui est périmée en cours de test de certification (PIX-14819). #10403
pix-service-auto-merge
merged 3 commits into
dev
from
pix-14819-bugfix-challenge-not-found-during-v3-certif
Oct 24, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
6cde5eb
to
63ae0b5
Compare
P-Jeremy
reviewed
Oct 24, 2024
api/src/certification/evaluation/domain/usecases/get-next-challenge.js
Outdated
Show resolved
Hide resolved
P-Jeremy
approved these changes
Oct 24, 2024
523b677
to
2c23f28
Compare
AndreiaPena
reviewed
Oct 24, 2024
api/src/certification/flash-certification/domain/services/algorithm-methods/flash.js
Outdated
Show resolved
Hide resolved
…rithm Co-authored-by: GUL <[email protected]>
Co-authored-by: GUL <[email protected]>
5d8b8f6
to
fb8c1ba
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🍂 Problème
Lors d'un test de certification v3, les épreuves sont sélectionnées au fur et à mesure du test.
Il peut arriver qu'une épreuve à laquelle le candidat a répondu soit périmée plus tard, avant la fin du test de certification.
Cela entraîne une erreur lors de la lecture des paramètres de l'épreuve par l'algorithme qui ne trouve pas l'épreuve associée à la réponse du candidat.
En analysant le usecase
get-next-challenge
, on constate que seules les épreuves validées sont envoyées à l'algorithme.🌰 Proposition
On modifie le usecase
get-next-challenge
afin d'envoyer à l'algorithme l'ensemble des épreuves validées ainsi que les épreuves auxquelles le candidat a répondu.On ajoute également un log d'avertissement si une réponse n'a pu être reliée à une épreuve au cours du test de certification.
🎃 Remarques
En faisant ce correctif, on a constaté un appel en double à
answerRepository.findByAssessmentId
au sein du usecase, on en a profité pour supprimer ce doublon et adapter les tests.🪵 Pour tester
Commencer un test de certification V3 en répondant à une première épreuve.
Modifier le contenu du cache LCMS (voir page confluence https://1024pix.atlassian.net/wiki/spaces/EDTDT/pages/4569202691/Customiser+le+contenu+de+la+release+LCMS+pour+une+Review+App+Pix) pour périmer l'épreuve passée.
Continuer le test de certification.
Constater qu'il n'y a aucun problème.