-
Notifications
You must be signed in to change notification settings - Fork 4
test(playwright): ⚗️ improve integration tests #1805
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
1fec84e to
c5fd72f
Compare
c5fd72f to
947f0bd
Compare
|
🤖 Hey ! The security scan report for the current pull request is available here. |
947f0bd to
6f0b153
Compare
StephaneTrebel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'est beau. LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
praise: Nickel 👍 Faudra dans le turfu qu'on source un fichier .env quelconque pour éviter la tartine d'export mais c'est un très bon début, et documenté 👏
| .fill(`${faker.internet.url({ appendSlash: true })}myrepository.git`) | ||
| } | ||
| if (infraRepo) { | ||
| await page.getByText('Dépôt contenant du code d\'').click() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thought: *soupir*, ça manque cruellement de data-testid et de id un peu partout…
6f0b153 to
b07e1b5
Compare
b07e1b5 to
61347e9
Compare
Issues liées
Issues numéro: cloud-pi-native/socle#611
Quel est le comportement actuel ?
We have some tests that we could improve.
Quel est le nouveau comportement ?
Cette PR introduit-elle un breaking change ?
No.
Autres informations
No.