PROCESS: Tests should always wait on creating documents.

Author: benel

.github/workflows/tests.yml

@@ -49,7 +49,7 @@ jobs:
           args: http://localhost/ 200 30000 500
       - name: Run tests
         run: |
-          cd frontend; npx cypress run --config baseUrl=http://localhost
+          cd frontend; npx cypress run --config "baseUrl=http://localhost, defaultCommandTimeout=6000"
       - name: Measure carbon footprint
         run: |
           cd requirements

frontend/tests/context.js

@@ -105,7 +105,7 @@ Soit("un document en deux passages affiché comme document principal", () => {
 
 Soit("une glose dont je suis l'auteur faisant référence uniquement au premier passage", () => {
   cy.sign_in('alice');
-  cy.get('.create-document').click();
+  cy.click_on_create();
   Step(this, "j'essaie de remplacer le contenu de la glose par :", '{1} First side passage');
   cy.contains('.formatted-text', 'First side passage');
   cy.sign_out();
@@ -131,7 +131,7 @@ Soit("{string} la glose ouverte", (title) => {
 
 Soit("un autre document, en plusieurs passages, affiché comme glose et dont je suis l'auteur", () => {
   cy.sign_in('alice');
-  cy.get('.create-document').click();
+  cy.click_on_create();
   cy.click_on_contextual_menu_item('.runningHead .scholium', 'Break into numbered passages');
   cy.get('.lectern .row:not(.runningHead) .scholium').should('have.length.greaterThan', 1);
   cy.sign_out();

frontend/tests/event.js

@@ -6,7 +6,7 @@ Quand("j'essaie de créer un nouveau document", () => {
 
 Quand("j'essaie de créer une glose de type {string}", (option) => {
   cy.get('#select-dropdown').select(option);
-  cy.get('.create-document').click();
+  cy.click_on_create();
 });
 
 Quand("je me focalise sur {string}", (title) => {
@@ -96,25 +96,25 @@ Quand("je souhaite modifier le contenu du document principal", () => {
 Quand("j'essaie de créer une glose en gardant {string} comme éditeur", (userName) => {
   cy.get('.open-editor-list').click();
   cy.get(`#editor-${userName}`).click();
-  cy.get(".create-document").click();
+  cy.click_on_create();
 });
 
 Quand("j'essaie de créer une glose en gardant tous les éditeurs", () => {
   cy.get('.open-editor-list').click();
   cy.get(`#select-all-editor`).click();
-  cy.get(".create-document").click();
+  cy.click_on_create();
 });
 
 Quand("j'essaie de créer une glose en gardant les métadonnées du document source", () => {
   cy.get('.open-metadata-list').click();
   cy.get(`#select-all-metadata`).click();
-  cy.get(".create-document").click();
+  cy.click_on_create();
 });
 
 Quand("j'essaie de créer une glose en gardant la {string} du document source", (metadata) => {
   cy.get('.open-metadata-list').click();
   cy.get(`#metadata-${metadata}`).click();
-  cy.get(".create-document").click();
+  cy.click_on_create();
 });
 
 Quand("je consulte l'historique du document", function () {

frontend/tests/support.js

@@ -33,24 +33,22 @@ Cypress.Commands.add('click_on_text', (type, text) => {
 });
 
 Cypress.Commands.add('create_glose', (random = false) => {
-  cy.get('.create-document').click();
-  cy.url().should('match', /\/[0-9a-f]+\/[0-9a-f]+$/);
+  cy.click_on_create();
   if (random) {
     cy.set_random_name();
   }
 });
 
 Cypress.Commands.add('create_glose_of_type', (random = false, option = "Adaptation") => {
   cy.get('#select-dropdown').select(option);
-  cy.get('.create-document').click();
-  cy.url().should('match', /\/[0-9a-f]+\/[0-9a-f]+$/);
+  cy.click_on_create();
   if (random) {
     cy.set_random_name();
   }
 });
 
 Cypress.Commands.add('create_document_from_scratch', () => {
-  cy.get('.create-document').click();
+  cy.click_on_create();
   cy.get('.lectern').should('exist');
 });
 
@@ -78,6 +76,11 @@ Cypress.Commands.add('click_on_contextual_menu_item', (context, item_name) => {
   cy.contains(item_name).click({force: true});
 });
 
+Cypress.Commands.add('click_on_create', () => {
+  cy.get('.create-document').click();
+  cy.url().should('match', /\/[0-9a-f]+\/[0-9a-f]+$/);
+});
+
 // From: https://github.com/decaporg/decap-cms/blob/a4b7481a99f58b9abe85ab5712d27593cde20096/cypress/support/commands.js#L374
 
 Cypress.Commands.add('setSelection', { prevSubject: true }, (subject, query, endQuery) => {