[shikanime]

Issues liées

Aucune issue ouverte ne couvre ces manques de couverture ; la justification est documentée ci-dessous et dans l'audit de la suite Playwright. (fermeture délibérée après la fusion : néant)

---

Quel est le comportement actuel ?

Sept surfaces de la console n'ont aucun test e2e : l'onglet Services externes d'un projet (aucune référence à test-tab-services dans la suite), l'UI de déploiement (deux tests test.skip depuis l'ajout de la fonctionnalité), l'onglet Clusters du projet, l'onglet Configuration du projet, la page Plugins admin, la page Réglages console et la page Status des services — plus la page 404. Par ailleurs, trois tests de clusters.spec.ts ne portent pas le tag @e2e et ne sont donc jamais sélectionnés par le grep CI, et le cleanup Keycloak (tag @integ dans le répertoire e2e) est injoignable dans les deux modes documentés.

Quel est le nouveau comportement ?

• 7 nouveaux specs e2e (8 tests) : Services externes (état vide, vue utilisateur puis admin via Administration → Projets), clusters du projet, configuration du projet (bascule limitless + quotas hprod), Plugins admin, Réglages système (lecture seule, aucun clic sur les toggles), Status des services, page 404.
• deployment.spec.ts dé-skippé et réécrit en 3 tests (prérequis sans environnement/dépôt, création, suppression) ; helpers/deployment.ts aligné sur l'UI réelle (radio riche cliqué via son label, sélecteur de dépôt, DeleteForm) et attend les lignes environnement/dépôt avant d'ouvrir le modal — sinon le bouton agit sur des données non chargées (race observée).
• Les 3 tests clusters non tagués portent désormais @e2e et entrent dans la sélection CI.
• Keycloak : describe en mode serial, cleanup converti de test @integ (injoignable) en afterAll exécuté en mode e2e.

Cette PR introduit-elle un breaking change ?

Non.

Autres informations

Basée sur #2940 (fiabilisation), désormais fusionné : la branche est rebasée sur main en un commit unique (e723868). Vérification sur stack CI locale (Node 26.7.0, Chromium, 1 worker, retries 0) : les 11 nouveaux/réécrits passent (27 s), clusters 10/10, keycloak 9/9 avec cleanup vérifié (compteur de groupes stable avant/après relance, 0 utilisateur résiduel), directory complet 95/96 — l'unique échec est la race produit documentée du formulaire de création de cluster (voir #2940), non liée à cette PR. tsc --noEmit et eslint : exit 0 ; listing 192 tests (24 fichiers) contre 176 (17 fichiers) avant.

[StephaneTrebel]

Les ajouts couvrent les surfaces annoncées et le nettoyage Keycloak est désormais raccordé à la fin de suite ; aucun défaut bloquant identifié dans le diff. Collecte de 192 tests/24 fichiers, typage (avec neutralisation de la dépréciation TypeScript 6 locale) et ESLint réussis ; E2E non réexécutés localement, avec une remarque non bloquante sur la preuve de sauvegarde.

[shikanime]

Verdict : Commentaire

Sept nouvelles surfaces couvertes, le déploiement dé-skippé avec une garde explicite contre la course au chargement des tableaux environnement/dépôt, et le cleanup Keycloak enfin joignable : le diff tient chacune des promesses du corps de la PR. Scan de sécurité des lignes ajoutées propre, helpers et imports cohérents avec la suite existante, CI verte sur e723868. Les parcours e2e ne sont pas réexécutés ici — l'exécution locale documentée dans la PR fait foi.