fix(backend/indicateurs): ne vérifie pas en plus la permission INDICA…

…TEURS_EDITION pour l'écriture de la trajectoire
incubateur-ademe · Jan 8, 2025 · 64a75c4 · 64a75c4
1 parent 389d414
commit 64a75c4
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/backend/src/indicateurs/trajectoires/trajectoire-snbc.e2e-spec.ts b/backend/src/indicateurs/trajectoires/trajectoire-snbc.e2e-spec.ts
@@ -295,6 +295,13 @@ describe('Calcul de trajectoire SNBC', () => {
       });
   });
 
+  it(`Calcul avec droit suffisant - lecture`, async () => {
+    return request(app.getHttpServer())
+      .get('/trajectoires/snbc?collectiviteId=3895')
+      .set('Authorization', `Bearer ${yoloDodoToken}`)
+      .expect(200);
+  }, 30000);
+
   it(`Verification et calcul avec donnees completes`, async () => {
     // Suppression de la trajectoire snbc existante si le test est joué plusieurs fois
     await request(app.getHttpServer())

diff --git a/backend/src/indicateurs/trajectoires/trajectoires-spreadsheet.service.ts b/backend/src/indicateurs/trajectoires/trajectoires-spreadsheet.service.ts
@@ -324,7 +324,7 @@ export default class TrajectoiresSpreadsheetService {
     const upsertedTrajectoireIndicateurValeurs =
       await this.valeursService.upsertIndicateurValeurs(
         indicateurValeursTrajectoireResultat,
-        tokenInfo
+        undefined // we don't want to check permission, we have already checked it and it's not the same
       );
 
     // Maintenant que les indicateurs ont été créés, on peut ajouter la collectivité au groupement

diff --git a/backend/src/indicateurs/valeurs/crud-valeurs.service.ts b/backend/src/indicateurs/valeurs/crud-valeurs.service.ts
@@ -307,7 +307,7 @@ export default class CrudValeursService {
 
   async upsertIndicateurValeurs(
     indicateurValeurs: IndicateurValeurInsert[],
-    tokenInfo: AuthenticatedUser
+    tokenInfo: AuthenticatedUser | undefined
   ): Promise<IndicateurValeur[]> {
     if (tokenInfo) {
       const collectiviteIds = [
@@ -331,7 +331,7 @@ export default class CrudValeursService {
     }
 
     this.logger.log(
-      `Upsert des ${indicateurValeurs.length} valeurs des indicateurs pour l'utilisateur ${tokenInfo.id} (role ${tokenInfo.role})`
+      `Upsert des ${indicateurValeurs.length} valeurs des indicateurs pour l'utilisateur ${tokenInfo?.id} (role ${tokenInfo?.role})`
     );
     // On doit distinguer les valeurs avec et sans métadonnées car la clause d'unicité est différente (onConflictDoUpdate)
     const [indicateurValeursAvecMetadonnees, indicateurValeursSansMetadonnees] =