feat(api+admin): certification structure update page now also update a certification certification authorities

Author: agarbe

packages/reva-admin-react/cypress/e2e/certifications/update-certificaton-structure-page/fixtures/certification-authorities-educ-nat.json

@@ -0,0 +1,21 @@
+{
+  "data": {
+    "certification_authority_getCertificationAuthorityStructure": {
+      "id": "0ec61d50-a202-4222-95ff-d516b9cae503",
+      "certificationAuthorities": [
+        {
+          "id": "47954f7a-1148-4280-842b-01eecf8ac52d",
+          "label": "Ministère de l'Education Nationale et de la Jeunesse - Auvergne - Rhône-Alpes"
+        },
+        {
+          "id": "39c45c3d-4785-4745-8f24-5cb11c47896e",
+          "label": "Ministère de l'Education Nationale et de la Jeunesse - Bourgogne - Franche-Comté"
+        },
+        {
+          "id": "dd2aaae3-3d59-45a0-8448-804d3f713bda",
+          "label": "Ministère de l'Education Nationale et de la Jeunesse - Bretagne"
+        }
+      ]
+    }
+  }
+}

packages/reva-admin-react/cypress/e2e/certifications/update-certificaton-structure-page/fixtures/certification-bp-boucher.json

@@ -6,7 +6,8 @@
       "certificationAuthorityStructure": {
         "id": "0ec61d50-a202-4222-95ff-d516b9cae503",
         "label": "Ministère de l'Education Nationale et de la Jeunesse"
-      }
+      },
+      "certificationAuthorities": []
     },
     "certification_authority_getCertificationAuthorityStructures": {
       "rows": [

packages/reva-admin-react/cypress/e2e/certifications/update-certificaton-structure-page/update-certification-structure-page.cy.ts

@@ -1,6 +1,7 @@
 import { stubMutation, stubQuery } from "../../../utils/graphql";
 import certificationBPBoucher from "./fixtures/certification-bp-boucher.json";
 import updateCertificationStructureMutationResponse from "./fixtures/update-certification-structure-mutation-response.json";
+import certificationAuthoritiesEducNat from "./fixtures/certification-authorities-educ-nat.json";
 
 function interceptCertification() {
   cy.intercept("POST", "/api/graphql", (req) => {
@@ -19,6 +20,11 @@ function interceptCertification() {
       "getCertificationStructureAndGestionnairesForUpdateCertificationStructurePage",
       certificationBPBoucher,
     );
+    stubQuery(
+      req,
+      "getCertificationAuthoritiesForUpdateCertificationStructurePage",
+      certificationAuthoritiesEducNat,
+    );
   });
 }
 
@@ -44,6 +50,7 @@ context("when i access the update certification page ", () => {
     cy.wait(
       "@getCertificationStructureAndGestionnairesForUpdateCertificationStructurePage",
     );
+    cy.wait("@getCertificationAuthoritiesForUpdateCertificationStructurePage");
 
     cy.get('[data-test="update-certification-structure-page"]')
       .children("h1")
@@ -61,9 +68,11 @@ context("when i access the update certification page ", () => {
     cy.wait(
       "@getCertificationStructureAndGestionnairesForUpdateCertificationStructurePage",
     );
+    cy.wait("@getCertificationAuthoritiesForUpdateCertificationStructurePage");
     cy.get(
       '[data-test="certification-authority-structure-select"] select',
     ).select("IPERIA");
+    cy.wait("@getCertificationAuthoritiesForUpdateCertificationStructurePage");
 
     cy.get("button").contains("Enregistrer").click();
 

packages/reva-admin-react/src/app/(admin)/certifications-v2/[certificationId]/structure/page.tsx

@@ -21,7 +21,12 @@ export default function UpdateCertificationStructurePage() {
 
   const handleFormSubmit = async (data: CertificationStructureFormData) => {
     try {
-      await updateCertificationStructure.mutateAsync(data);
+      await updateCertificationStructure.mutateAsync({
+        ...data,
+        certificationAuthorityIds: data.certificationAuthorities
+          .filter(({ checked }) => checked)
+          .map(({ id }) => id),
+      });
       successToast("modifications enregistrées");
     } catch (e) {
       graphqlErrorToast(e);

packages/reva-admin-react/src/app/(admin)/certifications-v2/[certificationId]/structure/updateCertificationStructure.hook.ts

@@ -26,15 +26,19 @@ const getCertificationStructureAndGestionnairesQuery = graphql(`
   }
 `);
 
-const updateCertificationStructureMutation = graphql(`
-  mutation updateCertificationStructureForUpdateCertificationStructurePage(
-    $input: UpdateCertificationStructureInput!
-  ) {
-    referential_updateCertificationStructure(input: $input) {
-      id
+const updateCertificationStructureAndCertificationAuthoritiesMutation = graphql(
+  `
+    mutation updateCertificationStructureForUpdateCertificationStructurePage(
+      $input: UpdateCertificationStructureAndCertificationAuthoritiesInput!
+    ) {
+      referential_updateCertificationStructureAndCertificationAuthorities(
+        input: $input
+      ) {
+        id
+      }
     }
-  }
-`);
+  `,
+);
 
 export const useUpdateCertificationStructurePage = ({
   certificationId,
@@ -62,12 +66,21 @@ export const useUpdateCertificationStructurePage = ({
   const updateCertificationStructure = useMutation({
     mutationFn: ({
       certificationAuthorityStructureId,
+      certificationAuthorityIds,
     }: {
       certificationAuthorityStructureId: string;
+      certificationAuthorityIds: string[];
     }) =>
-      graphqlClient.request(updateCertificationStructureMutation, {
-        input: { certificationId, certificationAuthorityStructureId },
-      }),
+      graphqlClient.request(
+        updateCertificationStructureAndCertificationAuthoritiesMutation,
+        {
+          input: {
+            certificationId,
+            certificationAuthorityStructureId,
+            certificationAuthorityIds,
+          },
+        },
+      ),
     onSuccess: () =>
       queryClient.invalidateQueries({
         queryKey: [certificationId],

packages/reva-api/modules/referential/features/updateCertificationStructure.ts

@@ -0,0 +1,36 @@
+import { prismaClient } from "../../../prisma/client";
+import { UpdateCertificationStructureAndCertificationAuthoritiesInput } from "../referential.types";
+
+export const updateCertificationStructureAndCertificationAuthorities = async ({
+  certificationId,
+  certificationAuthorityStructureId,
+  certificationAuthorityIds,
+}: UpdateCertificationStructureAndCertificationAuthoritiesInput) =>
+  prismaClient
+    .$transaction([
+      prismaClient.certification.update({
+        where: { id: certificationId },
+        data: {
+          certificationAuthorityOnCertification: { deleteMany: {} },
+        },
+      }),
+      prismaClient.certification.update({
+        where: { id: certificationId },
+        data: {
+          certificationAuthorityOnCertification: {
+            createMany: {
+              data: certificationAuthorityIds.map((id) => ({
+                certificationAuthorityId: id,
+              })),
+            },
+          },
+        },
+      }),
+      prismaClient.certification.update({
+        where: { id: certificationId },
+        data: {
+          certificationAuthorityStructureId,
+        },
+      }),
+    ])
+    .then((r) => r[r.length - 1]);

packages/reva-api/modules/referential/features/updateCertificationStructureAndCertificationAuthorities.ts

@@ -336,9 +336,10 @@ input CreateCompetenceBlocInput {
   competences: [CompetenceInput!]!
 }
 
-input UpdateCertificationStructureInput {
+input UpdateCertificationStructureAndCertificationAuthoritiesInput {
   certificationId: ID!
   certificationAuthorityStructureId: String!
+  certificationAuthorityIds: [String!]!
 }
 
 input SendCertificationToRegistryManagerInput {
@@ -365,8 +366,8 @@ type Mutation {
     certificationCompetenceBlocId: String!
   ): CertificationCompetenceBloc!
   referential_addCertification(input: AddCertificationInput!): Certification!
-  referential_updateCertificationStructure(
-    input: UpdateCertificationStructureInput
+  referential_updateCertificationStructureAndCertificationAuthorities(
+    input: UpdateCertificationStructureAndCertificationAuthoritiesInput
   ): Certification!
   referential_sendCertificationToRegistryManager(
     input: SendCertificationToRegistryManagerInput

packages/reva-api/modules/referential/referential.graphql

@@ -25,7 +25,7 @@ import {
   UpdateCertificationInput,
   UpdateCompetenceBlocInput,
   UpdateCompetenceBlocsInput,
-  UpdateCertificationStructureInput,
+  UpdateCertificationStructureAndCertificationAuthoritiesInput,
   SendCertificationToRegistryManagerInput,
 } from "./referential.types";
 import { RNCPCertification, RNCPReferential } from "./rncp";
@@ -48,7 +48,7 @@ import { getCompetenceBlocsByCertificationIdV2 } from "./features/getCompetenceB
 import { getDomainsByCertificationId } from "./features/getDomainsByCertificationId";
 import { getDomainsByFormacodes } from "./features/getDomainsByFormacodes";
 import { createCertificationCompetenceBloc } from "./features/createCertificationCompetenceBloc";
-import { updateCertificationStructure } from "./features/updateCertificationStructure";
+import { updateCertificationStructureAndCertificationAuthorities } from "./features/updateCertificationStructureAndCertificationAuthorities";
 import { sendCertificationToRegistryManager } from "./features/sendCertificationToRegistryManager";
 
 const unsafeReferentialResolvers = {
@@ -207,10 +207,14 @@ const unsafeReferentialResolvers = {
       _parent: unknown,
       { input }: { input: { codeRncp: string } },
     ) => addCertification(input),
-    referential_updateCertificationStructure: (
+    referential_updateCertificationStructureAndCertificationAuthorities: (
       _parent: unknown,
-      { input }: { input: UpdateCertificationStructureInput },
-    ) => updateCertificationStructure(input),
+      {
+        input,
+      }: {
+        input: UpdateCertificationStructureAndCertificationAuthoritiesInput;
+      },
+    ) => updateCertificationStructureAndCertificationAuthorities(input),
     referential_sendCertificationToRegistryManager: (
       _parent: unknown,
       { input }: { input: SendCertificationToRegistryManagerInput },

packages/reva-api/modules/referential/referential.resolvers.ts

@@ -21,7 +21,8 @@ export const referentialResolversSecurityMap = {
   "Mutation.referential_deleteCertificationCompetenceBloc": [
     hasRole(["admin"]),
   ],
-  "Mutation.referential_updateCertificationStructure": [hasRole(["admin"])],
+  "Mutation.referential_updateCertificationStructureAndCertificationAuthorities":
+    [hasRole(["admin"])],
   "Mutation.referential_sendCertificationToRegistryManager": [
     hasRole(["admin"]),
   ],