fix: validation niveau de certification dans l'API

Signed-off-by: Maud Royer <[email protected]>
AgenceBio · Jun 28, 2024 · 46998cb · 46998cb
1 parent 23858e9
commit 46998cb
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/lib/outputs/record.js b/lib/outputs/record.js
@@ -39,13 +39,19 @@ function normalizeRecordSummary (record) {
 
 /**
  * @param {String} etat
+ * @param {object} options
+ * @param {boolean} options.strict
  * @returns {EtatProduction|String}
  */
-function normalizeEtatProduction (etat) {
+function normalizeEtatProduction (etat, options = { strict: false }) {
   if (Object.hasOwn(EtatProduction, etat)) {
     return EtatProduction[etat]
   }
 
+  if (options.strict) {
+    throw new Error(`EtatProduction invalide: ${etat}`)
+  }
+
   return etat
 }
 

diff --git a/lib/providers/cartobio.js b/lib/providers/cartobio.js
@@ -1119,6 +1119,13 @@ async function * parseAPIParcellaireStream (stream, { organismeCertificateur })
         const pac = parsePacDetailsFromComment(parcelle.commentaire)
         const numeroIlot = parseInt(String(parcelle.numeroIlot), 10)
         const numeroParcelle = parseInt(String(parcelle.numeroParcelle), 10)
+        let conversionNiveau
+        try {
+          conversionNiveau = parcelle.etatProduction && normalizeEtatProduction(parcelle.etatProduction, { strict: true })
+        } catch (error) {
+          hasFeatureError = new Error('champ etatProduction incorrect')
+          return null
+        }
 
         const properties = {
           id,
@@ -1133,7 +1140,7 @@ async function * parseAPIParcellaireStream (stream, { organismeCertificateur })
           NUMERO_I: Number.isNaN(numeroIlot) ? (pac.numeroIlot ?? '') : String(numeroIlot),
           NUMERO_P: Number.isNaN(numeroParcelle) ? (pac.numeroParcelle ?? '') : String(numeroParcelle),
           PACAGE: record.numeroPacage ? String(record.numeroPacage) : null,
-          conversion_niveau: normalizeEtatProduction(parcelle.etatProduction),
+          conversion_niveau: conversionNiveau,
           engagement_date: parcelle.dateEngagement ?? null,
           auditeur_notes: parcelle.commentaire ?? null
         }