(PC-34581) chore(e2e): adapt Android script for IOS support #7699
Conversation
|
|
| - runFlow: subFolder/venue/AccessVenueMapFromSearch.yml | ||
| - runFlow: subFolder/venue/VerifyVenueMap.yml | ||
| - runFlow: subFolder/cloud/commands/SetLocalizationToEverywhere.yml | ||
|
|
||
| - runFlow: subFolder/venue/AccessVenueMapFromSearchResult.yml | ||
| - runFlow: subFolder/venue/VerifyVenueMap.yml | ||
| # - runFlow: subFolder/venue/VerifyVenueMap.yml |
| @@ -5,4 +5,4 @@ appId: ${MAESTRO_APP_ID} | |||
| text: 'République Française - Liberté Égalité Fraternité' | |||
| - tapOn: 'Déconnexion' | |||
|
|
|||
| - assertVisible: 'Identifie-toi pour découvrir des offres' | |||
| - assertVisible: 'Identifie-toi pour découvrir des offres.*' | |||
There was a problem hiding this comment.
Qu'est-ce qui fait que sur iOS on doit faire ça ? *
| --- | ||
| - runFlow: | ||
| when: | ||
| platform: iOS |
There was a problem hiding this comment.
Du coup c'est iOS spécifique ? Est-ce que ça vaut le coup de distinguer dans le nom ?
| when: | ||
| platform: iOS | ||
| commands: | ||
| - tapOn: "Ouvrir la modale de localisation depuis le widget" |
There was a problem hiding this comment.
iOS récupère les accessiblity label plutot que les noms visibles ? Tu as pu voir des différentes avec Maestro studio ?
| text: "Supprimer.*" | ||
| index: 0 |
There was a problem hiding this comment.
Ça fonctionne pas sur Android ça ? Parce que sinon c'est le même script
| - runFlow: # Si affichage de la Home sans player | ||
| when: | ||
| platform: iOS | ||
| visible: "Playlist hybride" | ||
| commands: | ||
| - assertVisible: "Mise à jour requise.*" |
There was a problem hiding this comment.
C'est un teste en lien avec la bannière de mise à jour, en théorie tu ne devrais pas l'avoir dans tes tests E2E il faudrait se baser sur autre chose pour tester le cas dans player vidéo ?
| when: | ||
| platform: iOS | ||
| commands: | ||
| - tapOn: 'Notifications' |
There was a problem hiding this comment.
Ajouter index: 1 ne fonctionne pas en iOS ? Ça permettrait d'éviter la duplication iOS vs Android
| - runFlow: | ||
| when: | ||
| platform: Android | ||
| commands: | ||
| - pressKey: back | ||
|
|
||
| - runFlow: | ||
| when: | ||
| platform: iOS | ||
| commands: | ||
| - tapOn: "Revenir en arrière" | ||
|
|
There was a problem hiding this comment.
Tu as créer un flow spécifique pour ça mais tu ne l'utilise pas ?
| - runFlow: | ||
| when: | ||
| platform: Android | ||
| commands: | ||
| - pressKey: back | ||
|
|
||
| - runFlow: | ||
| when: | ||
| platform: iOS | ||
| commands: | ||
| - tapOn: "Revenir en arrière" |
| when: | ||
| platform: iOS | ||
| commands: | ||
| - tapOn: "Paris Paris" |
There was a problem hiding this comment.
Pourquoi il y a deux fois "Paris" ?
|
Dans cette PR il faudrait que tu ajoutes (si possible) dans la description, les différences entre Android et iOS que tu as remarqué car ça pourrait servir pour plus tard, si on essayer de comprendre les différentes adaptations que tu as du mettre en place |
Link to JIRA ticket: https://passculture.atlassian.net/browse/PC-34581
Flakiness
If I had to re-run tests in the CI due to flakiness, I add the incident [on Notion][2]
Checklist
I have:
Notable Changes for iOS Compatibility