Cypress initial config and tests #90

Merged
jsegarra merged 26 commits from wbuezas/hedera-web-mindshore:feature/CypressInit into 4922-vueMigration 2024-11-12 12:24:41 +00:00
Contributor
No description provided.
wbuezas added 6 commits 2024-10-16 23:32:16 +00:00
wbuezas changed title from Cypress initial config and tests to WIP: Cypress initial config and tests 2024-10-16 23:32:28 +00:00
wbuezas added 1 commit 2024-10-16 23:35:47 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
62349278ed
Move command
wbuezas added 1 commit 2024-10-17 19:34:31 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
693b309e79
Several changes and more tests
wbuezas added 1 commit 2024-10-18 19:22:32 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
ff423ae79b
More tests
wbuezas added 1 commit 2024-10-21 18:05:13 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
cfff9f7aeb
PendingOrdersView and OrdersView tests
wbuezas added 1 commit 2024-10-22 13:41:17 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
cb1542937a
Token changes
wbuezas added 1 commit 2024-10-22 17:49:41 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
186ec60937
Add news view tests
jsegarra reviewed 2024-10-23 22:43:19 +00:00
package.json Outdated
@ -69,1 +71,4 @@
"back": "cd ../vn-database && myvc start && cd ../salix && gulp backOnly",
"cy:open": "cd ../salix && gulp docker && cd ../hedera-web && cypress open",
"test:e2e": "cd ../salix && gulp docker && cd ../hedera-web && cypress run",
"cy:open-mindshore": "cd ../salix && gulp docker && cd ../hedera-web-mindshore && cypress open",
Member

no entiendo la diferencia entre hedera-web y hedera-web-mindshore

no entiendo la diferencia entre hedera-web y hedera-web-mindshore
Author
Contributor

Que uno luego del gulp docker va al proyecto de hedera-web y el otro al proyecto hedera-web-mindshore (fork), eso luego pueden eliminarlo, lo uso para poder desarrollar en el fork

Que uno luego del `gulp docker` va al proyecto de `hedera-web` y el otro al proyecto `hedera-web-mindshore` (fork), eso luego pueden eliminarlo, lo uso para poder desarrollar en el fork
jsegarra reviewed 2024-10-23 22:43:39 +00:00
package.json Outdated
@ -70,0 +73,4 @@
"test:e2e": "cd ../salix && gulp docker && cd ../hedera-web && cypress run",
"cy:open-mindshore": "cd ../salix && gulp docker && cd ../hedera-web-mindshore && cypress open",
"test:e2e-mindshore": "cd ../salix && gulp docker && cd ../hedera-web-mindshore && cypress run",
"resetDatabase": "cd ../salix && gulp docker",
Member

si definimos este script, no podemos usarlo en el resto?

si definimos este script, no podemos usarlo en el resto?
Author
Contributor

Si, claro que se puede, ahí lo apliqué.

Commit: 40444ac46a

Si, claro que se puede, ahí lo apliqué. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/40444ac46a8fe7d223684268a313ca0432af5ecf
jsegarra reviewed 2024-10-23 22:44:38 +00:00
@ -0,0 +18,4 @@
cy.get('[data-testid="addItemQuantityButton"]:first').click();
cy.dataCy('catalogAddToBasketButton').should('exist');
cy.dataCy('catalogAddToBasketButton').click();
cy.dataCy('positiveNotify').should('contain', 'Añadido');
Member

Esta podría ser candidata a irse a un comando global

Esta podría ser candidata a irse a un comando global
Author
Contributor

Command de notify creado y aplicado.

Commit: a142ceb151

Command de notify creado y aplicado. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/a142ceb1514c08ce169d0841e926f26d1c6b65f0
jsegarra reviewed 2024-10-23 22:45:22 +00:00
@ -0,0 +12,4 @@
cy.get('input[data-testid="payAmountInput"]').clear();
if (typeAmount || typeAmount === 0)
cy.get('input[data-testid="payAmountInput"]').type(typeAmount);
cy.dataCy('confirmDialogButton').should('exist');
Member

Estas 2 podrian ser candidatas a irse a un comando global

Estas 2 podrian ser candidatas a irse a un comando global
Author
Contributor

Coincido, aplicado.

Commit: e4bc14ef3b

Coincido, aplicado. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/e4bc14ef3b7b8eb5f4951088c9b077a8b022363f
wbuezas added 1 commit 2024-10-24 13:02:30 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
0bec75b2d3
Some changes and more tests
wbuezas added 1 commit 2024-10-24 17:04:14 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
eea2ad7984
Adjustments
wbuezas added 1 commit 2024-10-24 17:09:35 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
5f0a2f9181
New login test
wbuezas added 1 commit 2024-10-24 17:13:42 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
40444ac46a
Reuse resetDatabase script
wbuezas added 1 commit 2024-10-24 17:21:55 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
e4bc14ef3b
Create confirmDialog command
wbuezas added 1 commit 2024-10-24 17:30:33 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
a142ceb151
Create check notify commands
wbuezas changed title from WIP: Cypress initial config and tests to Cypress initial config and tests 2024-10-24 17:30:55 +00:00
wbuezas requested review from jsegarra 2024-10-24 17:31:03 +00:00
wbuezas added 1 commit 2024-10-24 21:03:49 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
879e3cbb44
Add catalog view tests
jsegarra added 1 commit 2024-10-24 21:58:15 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
1471d323d6
Merge branch '4922-vueMigration' into feature/CypressInit
wbuezas added 2 commits 2024-10-24 22:16:50 +00:00
jsegarra reviewed 2024-10-24 23:33:57 +00:00
@ -49,3 +50,4 @@
v-for="(item, index) in items"
:key="index"
:clickable="false"
data-testid="itemsViewCard"
Member

y si le concatenamos el indice?

y si le concatenamos el indice?
jsegarra requested changes 2024-10-25 22:51:23 +00:00
jsegarra left a comment
Member

Hay que añadir el test para gestionar las direcciones, tanto para crear, modificar, eliminar y cambiar la por defecto.
Añadir el report segun lo comentado en la tarea 7782

Hay que añadir el test para gestionar las direcciones, tanto para crear, modificar, eliminar y cambiar la por defecto. Añadir el report segun lo comentado en la tarea 7782
jsegarra requested changes 2024-10-25 23:38:18 +00:00
@ -0,0 +1,42 @@
describe('NewsView', () => {
before(() => {
cy.resetDB();
Member

Este comando lo veo bien, pero cuando sea un before:run https://docs.cypress.io/api/node-events/before-run-api
No entiendo la necesidad de hacerlo en cada test, e incluso CatalogView lo tiene 2 veces. Es para que al tirarlo de manera individual, se refresque?

Este comando lo veo bien, pero cuando sea un before:run https://docs.cypress.io/api/node-events/before-run-api No entiendo la necesidad de hacerlo en cada test, e incluso CatalogView lo tiene 2 veces. Es para que al tirarlo de manera individual, se refresque?
Author
Contributor

Entiendo lo que comentas, en base a esto, se te ocurre alguna alternativa para poder ejecutar diferentes tests sin que fallen los siguientes por haber creado / modificado data previamente?

Entiendo lo que comentas, en base a esto, se te ocurre alguna alternativa para poder ejecutar diferentes tests sin que fallen los siguientes por haber creado / modificado data previamente?
@ -0,0 +27,4 @@
cy.dataCy('formDefaultSaveButton').should('exist');
cy.dataCy('formDefaultSaveButton').should('be.disabled');
cy.dataCy('newsTitleInput').should('exist');
cy.get('input[data-testid="newsTitleInput"]').type('Test new');
Member

podemos hacer cy.dataCy(lo que sea).find('input')
Lo he tirado y en verde

podemos hacer cy.dataCy(lo que sea).find('input') Lo he tirado y en verde
Author
Contributor

Concuerdo con la propuesta, aplicada.

Commmit: f70b98b514

Concuerdo con la propuesta, aplicada. Commmit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/f70b98b514741336dba895a5900ff45ef6bc68bd
wbuezas added 1 commit 2024-11-04 13:47:37 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
f70b98b514
Apply find input in dataCy
wbuezas added 1 commit 2024-11-06 20:02:50 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
ab7c1d1fe0
Add address tests
wbuezas added 1 commit 2024-11-06 22:41:44 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
dff5820ed4
Improve address details update test
wbuezas added 1 commit 2024-11-07 01:44:09 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
69abcf6870
Create custom title variable
jsegarra approved these changes 2024-11-12 12:18:09 +00:00
@ -1,5 +1,6 @@
import { route } from 'quasar/wrappers';
import { useAppStore } from 'stores/app';
import { useUserStore } from 'stores/user';
Member

IR a remember-password no va,

IR a remember-password no va, ❌
jsegarra merged commit 0fc1fc58e6 into 4922-vueMigration 2024-11-12 12:24:41 +00:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/hedera-web#90
No description provided.