feat: 1103 confirmation demande agrement (retours revue)

Author: olivier-rabot

packages/backend/src/__tests__/usagers/agrements.test.ts

@@ -26,6 +26,10 @@ jest.mock("../../middlewares/common/checkJWT", () => {
 beforeAll(async () => await createTestContainer());
 afterAll(async () => await removeTestContainer());
 
+beforeEach(() => {
+  jest.clearAllMocks();
+});
+
 describe("GET /agrements/organisme/:organismeId", () => {
   it("devrait retourner un agrément par ID de l'organisme avec succès", async () => {
     authUser = await createUsagersUser();

packages/backend/src/usagers/agrements/agrements.controller.ts

@@ -63,11 +63,7 @@ export const AgrementController = {
   ) {
     log.i("PATCH statut IN");
 
-    if (!req.decoded) {
-      throw new AppError("Utilisateur non authentifié", { statusCode: 401 });
-    }
-    const { id: usagerUserId } = req.decoded;
-
+    const { id: usagerUserId } = req.decoded!;
     const agrementId = Number(req.validatedParams!.agrementId);
     const { statut } = req.validatedBody!;
     try {

packages/backend/src/usagers/agrements/agrements.repository.ts

@@ -253,6 +253,33 @@ export const AgrementsRepository = {
     return result.rows;
   },
 
+  /**
+   * Récupère un agrément par son ID. Retourne agrement enrichi avec l'email du user responsable.
+   */
+  async getById(
+    agrementId: number,
+  ): Promise<AgrementEntity & { user_mail: string | null }> {
+    const client = await getPool().connect();
+    try {
+      const result = await client.query(
+        `SELECT a.*, u.mail AS user_mail
+          FROM front.agrements a
+          JOIN front.organismes o ON a.organisme_id = o.id
+          JOIN front.user_organisme uo ON uo.org_id = o.id
+          JOIN front.users u ON uo.use_id = u.id
+          WHERE a.id = $1
+          LIMIT 1;`,
+        [agrementId],
+      );
+      if (result.rows.length === 0) {
+        throw new Error("Aucun agrément trouvé avec cet ID");
+      }
+      return result.rows[0] as AgrementEntity & { user_mail: string | null };
+    } finally {
+      client.release();
+    }
+  },
+
   /**
    * Récupère un agrément par organisme ID (avec ou sans détails liés)
    */
@@ -416,31 +443,6 @@ export const AgrementsRepository = {
     }
   },
 
-  /**
-   * Récupère l'email du user lié à un agrément id
-   */
-  async getUserMailByAgrementId(agrementId: number): Promise<string | null> {
-    const client = await getPool().connect();
-    try {
-      const query = `
-        SELECT u.mail
-        FROM front.agrements a
-        JOIN front.organismes o ON a.organisme_id = o.id
-        JOIN front.user_organisme uo ON uo.org_id = o.id
-        JOIN front.users u ON uo.use_id = u.id
-        WHERE a.id = $1
-        LIMIT 1;
-      `;
-      const result = await client.query(query, [agrementId]);
-      if (result.rows.length > 0) {
-        return result.rows[0].mail;
-      }
-      return null;
-    } finally {
-      client.release();
-    }
-  },
-
   async insertHistoryEvent({
     source,
     agrementId,

packages/backend/src/usagers/agrements/agrements.service.ts

@@ -10,6 +10,7 @@ import {
 import { getById } from "../../services/adresse";
 import { getFileMetaData } from "../../services/Document";
 import { mailService } from "../../services/mail";
+import AppError from "../../utils/error";
 import logger from "../../utils/logger";
 import MailUtils from "../../utils/mail";
 import { AgrementsRepository } from "./agrements.repository";
@@ -97,11 +98,23 @@ export const AgrementService = {
     statut: AGREMENT_STATUT;
     usagerUserId: string;
   }): Promise<boolean> {
+    let agrement;
+    try {
+      agrement = await AgrementsRepository.getById(agrementId);
+    } catch (err) {
+      log.w("Erreur lors de la récupération de l'agrément", err);
+      throw new AppError("Agrement non trouvé", { statusCode: 404 });
+    }
+
     const updated = await AgrementsRepository.updateStatut({
       agrementId,
       statut,
     });
-    if (!updated) return false;
+    if (!updated) {
+      throw new AppError("Échec de la mise à jour du statut", {
+        statusCode: 500,
+      });
+    }
 
     await AgrementService.trackEvent({
       agrementId,
@@ -111,20 +124,18 @@ export const AgrementService = {
       usagerUserId: Number(usagerUserId),
     });
 
-    // Envoi d'email si le statut devient TRANSMIS
     if (statut === AGREMENT_STATUT.TRANSMIS) {
-      try {
-        const email =
-          await AgrementsRepository.getUserMailByAgrementId(agrementId);
-        if (email) {
+      const email = agrement.user_mail;
+      if (email) {
+        try {
           await mailService.send(
             MailUtils.usagers.agrement.sendStatutTransmisMail({ email }),
           );
-        } else {
-          log.w("Aucun email trouvé pour l'agrément", agrementId);
+        } catch (e) {
+          log.w("Erreur lors de l'envoi de l'email de transmission", e);
         }
-      } catch (e) {
-        log.w("Erreur lors de l'envoi de l'email de transmission", e);
+      } else {
+        log.w("Aucun email trouvé pour l'agrément", agrementId);
       }
     }
     return true;

packages/frontend-usagers/src/components/agrement/personne-morale.vue

@@ -116,17 +116,13 @@ const validationSchema = yup.object({
     yup.string().email("Format d'email invalide").nullable(),
   ),
   adresse: requiredUnlessBrouillon(yup.string().nullable()),
-  // prenom: requiredUnlessBrouillon(yup.string().nullable()),
-  // nom: requiredUnlessBrouillon(yup.string().nullable()),
 });
 
 const initialValues = {
   statut: props.initAgrement.statut || AGREMENT_STATUT.BROUILLON,
   telephone: props.initOrganisme?.personneMorale?.telephone || "",
   email: props.initOrganisme?.personneMorale?.email || "",
   adresse: props.initOrganisme?.personneMorale?.adresse || "",
-  // prenom: props.initOrganisme?.personneMorale?.prenom || "",
-  // nom: props.initOrganisme?.personneMorale?.nom || "",
 };
 
 const { handleSubmit, setValues } = useForm({

packages/frontend-usagers/src/components/agrement/synthese.vue

@@ -108,7 +108,7 @@
   </div>
 </template>
 
-<script setup>
+<script setup lang="ts">
 import { storeToRefs } from "pinia";
 import { AGREMENT_STATUT } from "@vao/shared-bridge";
 import { useToaster } from "@vao/shared-ui";
@@ -120,11 +120,11 @@ const props = defineProps({
   cdnUrl: { type: String, required: true },
 });
 
-const expandedIndex = ref(-1);
-const coordonneesValid = ref(false);
-const dossierValid = ref(false);
-const bilanValid = ref(false);
-const projetValid = ref(false);
+const expandedIndex = ref<number>(-1);
+const coordonneesValid = ref<boolean>(false);
+const dossierValid = ref<boolean>(false);
+const bilanValid = ref<boolean>(false);
+const projetValid = ref<boolean>(false);
 
 const agrementStore = useAgrementStore();
 const { agrementCourant } = storeToRefs(agrementStore);

packages/frontend-usagers/src/stores/agrement.ts

@@ -147,11 +147,7 @@ export const useAgrementStore = defineStore("agrement", {
           agrementId,
           statut,
         );
-        if (
-          success &&
-          this.agrementCourant &&
-          this.agrementCourant.id === agrementId
-        ) {
+        if (success && this.agrementCourant?.id === agrementId) {
           this.agrementCourant.statut = statut;
         }
         log.i("changeStatutAgrement - DONE", { success });