17582: Fixes tests by really testing collection naming. 17582-cypress-test-fix
authorLucas Di Pentima <lucas@di-pentima.com.ar>
Wed, 12 May 2021 14:11:14 +0000 (11:11 -0300)
committerLucas Di Pentima <lucas@di-pentima.com.ar>
Wed, 12 May 2021 14:11:14 +0000 (11:11 -0300)
A mistake was made by adding file naming test cases instead of testing
a collection naming operation.

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas@di-pentima.com.ar>

cypress/integration/collection.spec.js

index e9876c9754384d4ea94341c9d0642a16f7403ae8..c6d29b2c06c8183a651380dc20168e739b940826 100644 (file)
@@ -239,8 +239,6 @@ describe('Collection panel tests', function () {
 
                 const names = [
                     'bar', // initial name already set
-                    '[between brackets]', // JSON-like strings (#17582)
-                    '{between braces}', // JSON-like strings (#17582)
                     '&',
                     'foo',
                     '&amp;',
@@ -570,7 +568,8 @@ describe('Collection panel tests', function () {
         // Create new collection
         cy.get('[data-cy=side-panel-button]').click();
         cy.get('[data-cy=side-panel-new-collection]').click();
-        const collName = `Test collection (${Math.floor(999999 * Math.random())})`;
+        // Name between brackets tests bugfix #17582
+        const collName = `[Test collection (${Math.floor(999999 * Math.random())})]`;
         cy.get('[data-cy=form-dialog]')
             .should('contain', 'New collection')
             .within(() => {