From 5bc279f829052e7b62828053617fbe7ab01be809 Mon Sep 17 00:00:00 2001 From: Fabien Lamarque Date: Fri, 24 May 2024 13:55:57 +0000 Subject: [PATCH] Supprime l'identifiant eIDAS de la trame ebms --- .env.oots.template | 1 - src/adaptateurs/adaptateurEnvironnement.js | 3 --- src/api/pieceJustificative.js | 2 -- src/ebms/requeteJustificatif.js | 3 --- src/routes/routesRequete.js | 1 - test/api/pieceJustificative.spec.js | 25 ---------------------- test/ebms/requeteJustificatif.spec.js | 11 ---------- test/routes/serveurTest.js | 1 - 8 files changed, 47 deletions(-) diff --git a/.env.oots.template b/.env.oots.template index 932a08f..269f331 100644 --- a/.env.oots.template +++ b/.env.oots.template @@ -1,5 +1,4 @@ AVEC_REQUETE_PIECE_JUSTIFICATIVE= # active l'API /requete/pieceJustificative avec valeur true -IDENTIFIANT_EIDAS= # identifiant eIDAS injecté dans les requêtes (tant qu'on ne sait pas le récupérer) URL_OOTS_FRANCE= # URL Serveur OOTS-France, ex. https://oots.gouv.fr DELAI_MAX_ATTENTE_DOMIBUS= # délai maximum d'attente d'une réponse Domibus à une requête envoyée (en millisecondes) diff --git a/src/adaptateurs/adaptateurEnvironnement.js b/src/adaptateurs/adaptateurEnvironnement.js index 0cff015..4e16ac4 100644 --- a/src/adaptateurs/adaptateurEnvironnement.js +++ b/src/adaptateurs/adaptateurEnvironnement.js @@ -1,8 +1,5 @@ const avecRequetePieceJustificative = () => process.env.AVEC_REQUETE_PIECE_JUSTIFICATIVE === 'true'; -const identifiantEIDAS = () => process.env.IDENTIFIANT_EIDAS; - module.exports = { avecRequetePieceJustificative, - identifiantEIDAS, }; diff --git a/src/api/pieceJustificative.js b/src/api/pieceJustificative.js index 6b983d7..ac3256e 100644 --- a/src/api/pieceJustificative.js +++ b/src/api/pieceJustificative.js @@ -16,7 +16,6 @@ const estErreurMetier = (e) => estErreurAbsenceReponse(e) || estErreurReponseReq const pieceJustificative = ( { adaptateurDomibus, - adaptateurEnvironnement, adaptateurUUID, depotPointsAcces, }, @@ -37,7 +36,6 @@ const pieceJustificative = ( codeDemarche, destinataire, idConversation, - identifiantEIDAS: adaptateurEnvironnement.identifiantEIDAS(), typeJustificatif, previsualisationRequise: (previsualisationRequise === 'true' || previsualisationRequise === ''), })) diff --git a/src/ebms/requeteJustificatif.js b/src/ebms/requeteJustificatif.js index 3c39133..3b7ebfa 100644 --- a/src/ebms/requeteJustificatif.js +++ b/src/ebms/requeteJustificatif.js @@ -11,7 +11,6 @@ class RequeteJustificatif extends Message { codeDemarche = 'T1', destinataire = {}, idConversation = config.adaptateurUUID.genereUUID(), - identifiantEIDAS = 'DK/DE/123123123', typeJustificatif = new TypeJustificatif({}), previsualisationRequise = false, } = {}, @@ -19,7 +18,6 @@ class RequeteJustificatif extends Message { super(config, { destinataire, idConversation }); this.codeDemarche = codeDemarche; - this.identifiantEIDAS = identifiantEIDAS; this.typeJustificatif = typeJustificatif; this.previsualisationRequise = previsualisationRequise; } @@ -113,7 +111,6 @@ class RequeteJustificatif extends Message { High - ${this.identifiantEIDAS} Smith Jonas 1999-03-01 diff --git a/src/routes/routesRequete.js b/src/routes/routesRequete.js index 1209cfd..60b97bc 100644 --- a/src/routes/routesRequete.js +++ b/src/routes/routesRequete.js @@ -17,7 +17,6 @@ const routesRequete = (config) => { pieceJustificative( { adaptateurDomibus, - adaptateurEnvironnement, adaptateurUUID, depotPointsAcces, }, diff --git a/test/api/pieceJustificative.spec.js b/test/api/pieceJustificative.spec.js index 7e76e57..88260be 100644 --- a/test/api/pieceJustificative.spec.js +++ b/test/api/pieceJustificative.spec.js @@ -25,7 +25,6 @@ describe('Le requêteur de pièce justificative', () => { adaptateurDomibus.envoieMessageRequete = () => Promise.resolve(); adaptateurDomibus.pieceJustificativeDepuisReponse = () => Promise.resolve(); adaptateurDomibus.urlRedirectionDepuisReponse = () => Promise.resolve(); - adaptateurEnvironnement.identifiantEIDAS = () => 'FR/BE/123456789'; adaptateurUUID.genereUUID = () => ''; depotPointsAcces.trouvePointAcces = () => Promise.resolve({}); @@ -82,30 +81,6 @@ describe('Le requêteur de pièce justificative', () => { return pieceJustificative(config, requete, reponse); }); - describe('quand `process.env.IDENTIFIANT_EIDAS` vaut `FR/BE/123456789`', () => { - let identifiantTemporaire; - - beforeEach(() => { - identifiantTemporaire = process.env.IDENTIFIANT_EIDAS; - process.env.IDENTIFIANT_EIDAS = 'FR/BE/123456789'; - }); - - afterEach(() => { - process.env.IDENTIFIANT_EIDAS = identifiantTemporaire; - }); - - it('utilise cet identifiant', () => { - adaptateurDomibus.envoieMessageRequete = ({ identifiantEIDAS }) => { - try { - expect(identifiantEIDAS).toEqual('FR/BE/123456789'); - return Promise.resolve(); - } catch (e) { return Promise.reject(e); } - }; - - return pieceJustificative(config, requete, reponse); - }); - }); - it('utilise un identifiant de conversation', () => { adaptateurUUID.genereUUID = () => '11111111-1111-1111-1111-111111111111'; diff --git a/test/ebms/requeteJustificatif.spec.js b/test/ebms/requeteJustificatif.spec.js index 2389bd9..b83efcf 100644 --- a/test/ebms/requeteJustificatif.spec.js +++ b/test/ebms/requeteJustificatif.spec.js @@ -66,17 +66,6 @@ describe("La vue du message de requête d'un justificatif", () => { expect(codeDemarche.LocalizedString['@_value']).toBe('T3'); }); - it("injecte l'identifiant eIDAS spécifié en variable d'environnement", () => { - const requeteJustificatif = new RequeteJustificatif( - configurationRequete, - { identifiantEIDAS: 'BE/FR/123456789' }, - ); - const xml = parseXML(requeteJustificatif.corpsMessageEnXML()); - - const personne = valeurSlot('NaturalPerson', xml.QueryRequest.Query); - expect(personne.Person.Identifier['#text']).toBe('BE/FR/123456789'); - }); - it("injecte l'identifiant de type de justificatif demandé", () => { const requeteJustificatif = new RequeteJustificatif( configurationRequete, diff --git a/test/routes/serveurTest.js b/test/routes/serveurTest.js index d21ddb0..ebb1cde 100644 --- a/test/routes/serveurTest.js +++ b/test/routes/serveurTest.js @@ -24,7 +24,6 @@ const serveurTest = () => { adaptateurEnvironnement = { avecRequetePieceJustificative: () => true, - identifiantEIDAS: () => 'FR/BE/123456789', }; adaptateurUUID = {