-
Notifications
You must be signed in to change notification settings - Fork 55
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] Reparer l'affichage des habilitations des centres (PIX-12909). #9323
[BUGFIX] Reparer l'affichage des habilitations des centres (PIX-12909). #9323
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
api/src/certification/enrolment/domain/usecases/get-mass-import-template-information.js
Show resolved
Hide resolved
api/src/certification/enrolment/infrastructure/repositories/center-repository.js
Outdated
Show resolved
Hide resolved
b5764ac
to
8239790
Compare
89e86a5
to
07e2dc6
Compare
07e2dc6
to
24d6ea0
Compare
J'ai une erreur lorsque je vais voir les détails du centre flaggué V3 https://admin-pr9323.review.pix.fr/certification-centers/7303 Le centre n'a pas d' ça impacte la modification d'un centre, si on souhaite retirer l'externalId |
api/tests/certification/enrolment/unit/domain/models/Habilitation_test.js
Show resolved
Hide resolved
api/src/certification/enrolment/domain/models/factories/CenterForAdminFactory.js
Show resolved
Hide resolved
api/tests/certification/enrolment/unit/domain/models/Center_test.js
Outdated
Show resolved
Hide resolved
api/tests/certification/enrolment/unit/domain/models/Center_test.js
Outdated
Show resolved
Hide resolved
api/tests/certification/enrolment/unit/domain/models/Center_test.js
Outdated
Show resolved
Hide resolved
api/tests/certification/enrolment/unit/domain/models/Center_test.js
Outdated
Show resolved
Hide resolved
api/tests/certification/enrolment/unit/domain/models/Center_test.js
Outdated
Show resolved
Hide resolved
api/src/certification/enrolment/infrastructure/repositories/center-repository.js
Show resolved
Hide resolved
expect(updatedCertificationCenter.habilitations.length).to.equal(1); | ||
expect(updatedCertificationCenter.habilitations[0].id).to.equal(complementaryCertification.id); | ||
expect(updatedCertificationCenter.habilitations[0].complementaryCertificationId).to.equal(complementaryCertification.id); | ||
expect(updatedCertificationCenter.habilitations[0].key).to.equal(complementaryCertification.key); | ||
expect(updatedCertificationCenter.habilitations[0].label).to.equal(complementaryCertification.label); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
expect(updatedCertificationCenter.habilitations.length).to.equal(1); | |
expect(updatedCertificationCenter.habilitations[0].id).to.equal(complementaryCertification.id); | |
expect(updatedCertificationCenter.habilitations[0].complementaryCertificationId).to.equal(complementaryCertification.id); | |
expect(updatedCertificationCenter.habilitations[0].key).to.equal(complementaryCertification.key); | |
expect(updatedCertificationCenter.habilitations[0].label).to.equal(complementaryCertification.label); | |
expect([updatedCertificationCenter.habilitations]).to.deepEqualInstanceOf(new Center(complementaryCertification)); |
api/tests/integration/domain/usecases/update-certification-center_test.js
Show resolved
Hide resolved
12380a5
to
cdd37a8
Compare
5da3f1e
to
ac0bea5
Compare
@AndreiaPena Merci pour ton test func et ta vigilance, en principe c'est corrigé ! |
api/tests/certification/enrolment/unit/domain/models/Center_test.js
Outdated
Show resolved
Hide resolved
api/tests/certification/enrolment/unit/domain/models/Center_test.js
Outdated
Show resolved
Hide resolved
Tech ok ! |
8be59c7
to
4b23c42
Compare
4b23c42
to
cd7305f
Compare
🦄 Problème
Sur Pix Admin, dans la page de détails d’un centre de certification, pour un centre comportant des habilitations (des certifications complémentaire) on peut voir un message d’erreur dans la console.
Le “type” attendu par ember est celui qui devrait se trouver dans le tableau des habilitations mais celui-ci retourne des tableaux de null.
attendu :
Ce qu’on a obtient actuellement
🤖 Proposition
🌈 Remarques
Creation / Mise de centre
Création de
Habilitation.js
dans le DomainLe but est de rapprocher le language métier et notre code source (Ubiquitous Language)
Or, on parle toujours, avec le métier, pour un Centre de Certification d'habilitations (à quelque chose).
Cela semble être le mot le plus clair pour quand on parle de "ce que le centre à le droit de faire passer en plus de la pix coeur".
Par ailleurs dans cette PR il a pour but d'effacer les passages par référence d'objets de différents types (DTO, complémentaires, etc.) afin de sécuriser ce qui a causé le bug traité ici.
💯 Pour tester
Conseil : faire les tests Pix Certif en parallèle de Pix Admin
Pix Admin
Pix Certif