fix(publish): les réponses spécifiques ne doivent pas être searchable (…

…#1203)
SocialGouv · Dec 20, 2023 · fbdb7d0 · fbdb7d0
1 parent 50496fd
commit fbdb7d0
Show file tree

Hide file tree

Showing 3 changed files with 132 additions and 3 deletions.
diff --git a/targets/frontend/src/modules/contribution/__tests__/mapContributionToDocument.test.ts b/targets/frontend/src/modules/contribution/__tests__/mapContributionToDocument.test.ts
@@ -131,6 +131,7 @@ describe("mapContributionToDocument", () => {
         info_random: "random",
       },
       is_available: true,
+      is_searchable: true,
     };
 
     const outputDoc: Document<ContributionDocumentJson> = {
@@ -209,6 +210,7 @@ describe("mapContributionToDocument", () => {
       },
       initial_id: "effee3b9-84fb-4667-944b-4b1e1fd14eb5",
       is_available: true,
+      is_searchable: false,
       meta_description: "meta",
       slug: "hospitalisation-du-nouveau-ne-quelles-consequences-sur-le-conge-de-maternite",
       source: "contributions",
@@ -224,4 +226,131 @@ describe("mapContributionToDocument", () => {
     );
     expect(result).toEqual(outputDoc);
   });
+
+  describe("flag is_searchable", () => {
+    it("devrait être à false pour une réponse personnalisée", async () => {
+      const inputContribution: ContributionsAnswers = {
+        id: "effee3b9-84fb-4667-944b-4b1e1fd14eb5",
+        content: "<p>Texte de la réponse</p>",
+        content_type: "ANSWER",
+        agreement: {
+          id: "0016",
+          name: "Convention collective nationale des transports routiers et activités auxiliaires du transport",
+          kali_id: "KALICONT000005635624",
+        },
+        question: {
+          id: "3384f257-e319-46d1-a4cb-8e8294da337b",
+          content:
+            "Quelles sont les conditions d’indemnisation pendant le congé de maternité ?",
+          order: 43,
+        },
+        kali_references: [],
+        legi_references: [],
+        other_references: [],
+        cdtn_references: [],
+        content_fiche_sp: null,
+      };
+
+      const inputDoc: Document<any> = {
+        cdtn_id: "cdtn_id",
+        initial_id: "effee3b9-84fb-4667-944b-4b1e1fd14eb5",
+        title:
+          "Quelles sont les conditions d’indemnisation pendant le congé de maternité ?",
+        meta_description: "meta",
+        source: "contributions",
+        slug: "hospitalisation-du-nouveau-ne-quelles-consequences-sur-le-conge-de-maternite",
+        text: " texte",
+        document: {
+          info_random: "random",
+        },
+        is_available: true,
+        is_searchable: true,
+      };
+
+      const result = await mapContributionToDocument(
+        inputContribution,
+        inputDoc,
+        jest.fn()
+      );
+      expect(result.is_searchable).toEqual(false);
+    });
+
+    it("devrait être à true pour une réponse générique", async () => {
+      const inputContribution: ContributionsAnswers = {
+        id: "effee3b9-84fb-4667-944b-4b1e1fd14eb5",
+        content: "<p>Texte de la réponse</p>",
+        content_type: "ANSWER",
+        agreement: {
+          id: "0000",
+          name: "Code du travail",
+          kali_id: "CDT",
+        },
+        question: {
+          id: "3384f257-e319-46d1-a4cb-8e8294da337b",
+          content:
+            "Quelles sont les conditions d’indemnisation pendant le congé de maternité ?",
+          order: 43,
+        },
+        kali_references: [],
+        legi_references: [],
+        other_references: [],
+        cdtn_references: [],
+        content_fiche_sp: null,
+      };
+
+      const result = await mapContributionToDocument(
+        inputContribution,
+        undefined,
+        jest.fn()
+      );
+      expect(result.is_searchable).toEqual(true);
+    });
+
+    it("devrait garder la même valeur du document pour une réponse générique", async () => {
+      const inputContribution: ContributionsAnswers = {
+        id: "effee3b9-84fb-4667-944b-4b1e1fd14eb5",
+        content: "<p>Texte de la réponse</p>",
+        content_type: "ANSWER",
+        agreement: {
+          id: "0000",
+          name: "Code du travail",
+          kali_id: "CDT",
+        },
+        question: {
+          id: "3384f257-e319-46d1-a4cb-8e8294da337b",
+          content:
+            "Quelles sont les conditions d’indemnisation pendant le congé de maternité ?",
+          order: 43,
+        },
+        kali_references: [],
+        legi_references: [],
+        other_references: [],
+        cdtn_references: [],
+        content_fiche_sp: null,
+      };
+
+      const inputDoc: Document<any> = {
+        cdtn_id: "cdtn_id",
+        initial_id: "effee3b9-84fb-4667-944b-4b1e1fd14eb5",
+        title:
+          "Quelles sont les conditions d’indemnisation pendant le congé de maternité ?",
+        meta_description: "meta",
+        source: "contributions",
+        slug: "hospitalisation-du-nouveau-ne-quelles-consequences-sur-le-conge-de-maternite",
+        text: " texte",
+        document: {
+          info_random: "random",
+        },
+        is_available: true,
+        is_searchable: false,
+      };
+
+      const result = await mapContributionToDocument(
+        inputContribution,
+        inputDoc,
+        jest.fn()
+      );
+      expect(result.is_searchable).toEqual(false);
+    });
+  });
 });
diff --git a/targets/frontend/src/modules/contribution/mapContributionToDocument.ts b/targets/frontend/src/modules/contribution/mapContributionToDocument.ts
@@ -82,9 +82,8 @@ export const mapContributionToDocument = async (
     slug:
       document?.slug ??
       generateContributionSlug(data.agreement.id, data.question.content),
-    is_searchable: document
-      ? document.is_searchable
-      : data.agreement.id === "0000",
+    is_searchable:
+      data.agreement.id === "0000" ? document?.is_searchable ?? true : false,
     is_available: true,
     document: doc,
   };

diff --git a/targets/frontend/src/modules/documents/api/documents.query.ts b/targets/frontend/src/modules/documents/api/documents.query.ts
@@ -17,6 +17,7 @@ const query = `query documents($source: String!, $initialId: String) {
         title
         meta_description
         is_available
+        is_searchable
   }
 }`;
-Original file line number
+Diff line change
@@ Expand Up @@
             title
             meta_description
             is_available
+            is_searchable
       }
     }`;
@@ Expand Down @@