(PC-32595)[PRO] refactor: test e2e #14708
Conversation
Aliochka
force-pushed
the
PC-32595-refacto-e2e
branch
from
fc36021
to
fc75594
Compare
Aliochka
force-pushed
the
PC-32595-refacto-e2e
branch
from
fc75594
to
f414361
Compare
|
Visit the preview URL for this PR (updated for commit f414361): https://pc-pro-testing--pr14708-pc-32595-refacto-e2e-jygx3uob.web.app (expires Thu, 24 Oct 2024 09:50:13 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 032d233ee67e1c50d6af12e29c936c7076770eb1 |
| @@ -178,3 +137,44 @@ describe('Search for collective bookings', () => { | |||
| expectOffersAreFound(expectedResults) | |||
| }) | |||
| }) | |||
|
|
|||
| function expectOffersAreFound(expectedResults: Array<Array<string>>) { | |||
There was a problem hiding this comment.
Cette fonction peut être mutualisée pour toutes les vérifications d'offres affichées dans:
- collectiveBooking
- searchCollectiveOffer
- searchIndividualOffer
Plus on peut vérifier le nombre de lignes et le texte x offres dans cette fonction
| } | ||
| } | ||
|
|
||
| function IGoToCollectivePage(login: string) { |
There was a problem hiding this comment.
Peut-on en faire un fonction pour tous les cas se logguant et allant sur une page précise (également pour accueil):
IConnectAndGoPage(login: string, url: string) soit pour:
- ici collectiveBooking
- createIndividualOffer
- desk
- editDigitalIndividualOffer
- financialManagement
- navigation
- searchCollectiveOffer
- searchIndividualOffer
- signupJourneyCreateOfferer
- venue
| @@ -242,3 +235,10 @@ describe('Cookie management with login', () => { | |||
| cy.get('#orejime-app-item-beamer').should('not.be.checked') | |||
| }) | |||
| }) | |||
|
|
|||
| function homePageLoaded(): void { | |||
There was a problem hiding this comment.
également utilisée par navigation....à intégrer dans IConnectAndGoPage dans le cas où on fait un simple login en restant sur l'accueil.
| @@ -2,7 +2,6 @@ import { addDays, format } from 'date-fns' | |||
|
|
|||
| describe('Desk (Guichet) feature', () => { | |||
| let login: string | |||
| const password = 'user@AZERTY123' | |||
There was a problem hiding this comment.
ce password par défaut est défini dans plusieurs tests, on devrait pouvoir le définir une seule fois pour tous les tests
| @@ -11,6 +11,9 @@ describe('Financial Management - messages, links to external help page, reimburs | |||
| login = response.body.user.email | |||
| }) | |||
| cy.intercept({ method: 'GET', url: '/offerers/*' }).as('getOfferers') | |||
| cy.intercept({ method: 'PATCH', url: 'offerers/*/bank-accounts/*' }).as( | |||
There was a problem hiding this comment.
Attention, ceci a été changé dans le dernier fix financialManagement, à rebaser
| @@ -182,6 +182,14 @@ describe('ADAGE discovery', () => { | |||
| }, | |||
| }) | |||
| ).as('searchOfferTemplate') | |||
| cy.intercept({ | |||
There was a problem hiding this comment.
C'est pas mal de mettre tous les intercept dans le before, ça posait problème avec cucumber, mais là c'est bien!
J'ai vu, dans createIndividualOffer que celle-ci est déclarée 2 fois:
- cy.intercept({ method: 'POST', url: '/stocks/bulk' }).as('postStocks')
Peux-tu en profiter pour éviter ce(s) doublon(s) ?
| @@ -79,6 +79,14 @@ describe('Signup journey with new venue', () => { | |||
| }) | |||
| ).as('search1Address') | |||
| cy.intercept({ method: 'GET', url: '/offerers/names' }).as('getOfferers') | |||
There was a problem hiding this comment.
ces intercept sont déclarés dans les 2 describe On pourrait faire un describe global avec un beforeEach() dans lequel on pourrait ces intercept. Qu'en penses-tu ?
But de la pull request
Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-32595
Vérifications