TEST: Metadata should be editable in a form rather than in a free text field (see #375).

Author: Dinalrn

frontend/scenarios/edit_metadata.feature

@@ -45,17 +45,19 @@ Scénario: sans être connecté
   Alors je peux lire "Before editing this document, please log in first"
 
 
-
-  Scénario: avec un formulaire dont on est l'auteur
-
+  
+Scénario: avec un formulaire dont on est l'auteur
   Soit un document dont je suis l'auteur affiché comme glose
   Et une session active avec mon compte
   Quand j'ouvre le formulaire de modification des métadonnées
-  Et je remplis "title" avec "Chapitre 1: Contexte historique"
   Et je remplis "creator" avec "Alice Liddell"
+  Et je remplis "title" avec "Chapitre 1: Contexte historique"
   Et je remplis "issued" avec "1932"
+  Et j'ajoute le champ "Language" au formulaire
   Et je remplis "language" avec "french"
+  Et j'ajoute le champ "Translator" au formulaire
   Et je remplis "translator" avec "Charles Beaudelaire"
+  Et j'ajoute le champ "Part of" au formulaire
   Et je remplis "isPartOf" avec "Philosophie Moderne : Une première approche"
   Et je valide le formulaire
   Alors "Chapitre 1: Contexte historique" est la glose ouverte

frontend/src/components/Metadata.jsx

@@ -129,7 +129,7 @@ function Metadata({metadata = {}, editable, user, backend, setLastUpdate}) {
         <span className="edition">
           {formatTranslation(dc_translator, dc_language)}
           {dc_isPartOf ? <i>{dc_isPartOf}, </i> : ''}
-          {dc_issued ? `${new Date(dc_issued.toString()).getFullYear()}` : ''}
+          {dc_issued ? `${dc_issued.toString().slice(0, 4)}` : ''}
         </span>
       </>
     );

frontend/tests/event.js

@@ -18,9 +18,7 @@ Quand("j'ouvre {string} à côté", (title) => {
   cy.contains('span', title).parent().prevAll('a.open').first().click();
 });
 
-Quand("j'essaie de remplacer les métadonnées de la glose par :", (metadata) => {
-  cy.edit_metadata(metadata);
-});
+
 
 Quand("j'essaie de remplacer l'annotation du passage {int} par :", (block_number, markdown) => {
   let element = cy.get(`.lectern>.row:nth-child(${block_number + 1})>.scholium>.content>.formatted-text`);
@@ -156,6 +154,12 @@ Quand("{string} remplace les métadonnées de la glose par :", (username, metada
   cy.request_by_user(username, parseStrToObject(metadata));
 });
 
+
+Quand("j'ouvre le formulaire de modification des métadonnées", () => { cy.click_on_text('metadata'); }); 
+
+
+Quand("je valide le formulaire", () => { cy.get('.metadata-form').submit();
+});
 Quand("le titre de l'onglet est {string}", (title) => {
   cy.title().should("eq", title);
 });
@@ -173,3 +177,38 @@ Quand("j'essaie de créer une glose qui soit découpée en passages", () => {
   cy.click_on_create();
   cy.get('.scholium .focus').click();
 });
+
+//MOFIER PAR MOI
+Quand("je remplis {string} avec {string}", (field, value) => {
+  const input = cy.get(`#dc_${field}`);
+  // dc_issued est de type "date" : on force la valeur via invoke('val') + trigger
+  if (field === 'issued') {
+    const dateValue = value.length === 4 ? `${value}-01-01` : value;
+    input.invoke('val', dateValue).trigger('input').trigger('change');
+  } else {
+    input.clear().type(value);
+  }
+});
+
+Quand("je remplis {string} avec {string}", (field, value) => {
+  const input = cy.get(`#dc_${field}`);
+  if (field === 'issued') {
+    const dateValue = value.length === 4 ? `${value}-01-01` : value;
+    input
+      .focus()
+      .clear()
+      .type(dateValue)
+      .blur()
+      .focus();
+  } else {
+    input.clear().type(value);
+  }
+});
+ 
+Quand("j'ajoute le champ {string} au formulaire", (label) => {
+  cy.get('.add-section-toggle').click();
+  cy.contains('.dropdown-item', label).click();
+});
+
+//
+

frontend/tests/outcome.js

@@ -50,9 +50,6 @@ Alors("l'année de publication est {string}", (year) => {
   cy.get('.metadata > .edition').first().should('contain', year);
 });
 
-Alors("le titre de l'ouvrage est {string}", (title) => {
-  cy.get('.metadata > .edition').first().should('contain', title);
-})
 
 Alors("la langue est {string}", (language) => {
   cy.get('.metadata > .edition').first().should('contain', language);
@@ -222,3 +219,8 @@ Alors("le document apparaît une seule fois dans la liste de ma bibliothèque",
   cy.get('.bookshelf').contains(this.randomName);
   cy.get('.bookshelf .work').filter(`:contains("${this.randomName}")`).should('have.length', 1);
 });
+
+
+Alors("le titre de l'ouvrage est {string}", (title) => {
+  cy.get('.metadata').should('contain', title);
+});