Skip to content

Conversation

@Falltrades
Copy link
Contributor

@Falltrades Falltrades commented Dec 9, 2025

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 ?

  • Separate tests for more atomicity.
  • Add tests for Argocd.
  • Update Playwright Readme.

Cette PR introduit-elle un breaking change ?

No.

Autres informations

No.

@Falltrades Falltrades force-pushed the playwright/integration-test-console branch 7 times, most recently from 1fec84e to c5fd72f Compare December 11, 2025 14:17
@Falltrades Falltrades marked this pull request as ready for review December 11, 2025 14:27
@Falltrades Falltrades force-pushed the playwright/integration-test-console branch from c5fd72f to 947f0bd Compare December 11, 2025 15:23
@github-actions github-actions bot added the built label Dec 11, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025

🤖 Hey !

The security scan report for the current pull request is available here.

@Falltrades Falltrades force-pushed the playwright/integration-test-console branch from 947f0bd to 6f0b153 Compare December 11, 2025 15:34
@Falltrades Falltrades self-assigned this Dec 11, 2025
@Falltrades Falltrades added this to the Testing - QA milestone Dec 11, 2025
Copy link
Collaborator

@StephaneTrebel StephaneTrebel left a 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

Copy link
Collaborator

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()
Copy link
Collaborator

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…

@Falltrades Falltrades force-pushed the playwright/integration-test-console branch from 6f0b153 to b07e1b5 Compare December 11, 2025 16:00
@Falltrades Falltrades force-pushed the playwright/integration-test-console branch from b07e1b5 to 61347e9 Compare December 11, 2025 16:05
@Falltrades Falltrades merged commit 057c300 into main Dec 11, 2025
25 checks passed
@Falltrades Falltrades deleted the playwright/integration-test-console branch December 11, 2025 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants