🧪 Campaign: full console test coverage vs server-nestjs migration (unit + e2e + Playwright)

[shikanime]

Objectif / Goal

Tester chaque usage console → services externes (GitLab, Keycloak, Vault, ArgoCD, Nexus, Registry, SonarQube, Harbor) contre apps/server-nestjs, pour trouver les bugs et incompatibilités de la migration apps/server → apps/server-nestjs, et couvrir 100 % des features customer-facing touchées.

Contexte / Context

• apps/server (Fastify) est figé — ne pas modifier.
• apps/server-nestjs est le backend cible.
• Le mapping legacy→nestjs repose sur des événements : apps/server fire hook.<entity>.<verb> ; nestjs utilise EventEmitter2 avec @OnEvent → capturePluginResult.
• Checklist de parité : https://github.com/cloud-pi-native/console (cf. MIGRATION-PARITY-MATRIX.md).

Livrables / Deliverables

1. Tests unitaires (vitest) couvrant chaque chemin console→service externe + le contrat emit→consumer.
2. Tests e2e (apps/server-nestjs/test/*.e2e-spec.ts, gated E2E=1) pour chaque ressource migrée.
3. Playwright pour les journeys UI customer-facing (cross-service fragility → socle cahier).
4. Issues une par bug/incoherence trouvée, itérées jusqu'au vert.

Statut / Status

• Baseline unitaire : 578 passés / 59 skipped (Node 24, CI=true).
• Seed validé : app-events.service.spec.ts — parity zone.* (listener sans émetteur). 12/12 ✅
• Blocage infra : les hôtes d'intégration *.dso.cpin-hp.numerique-interieur.fr ne résolvent pas en DNS depuis cette machine dev → les tests e2e sont exécutés sur un host connecté / en CI.

Sous-tâches / Sub-tasks

• Unit: parity matrix emit→consumer (app-events.service.spec.ts)
• Unit: couverture 100 % chemins GitLab (gitlab.service.spec.ts)
• Unit: couverture 100 % chemins Keycloak + admin-role sync (keycloak.service.spec.ts)
• Unit: couverture 100 % chemins Vault y.c. zone.* (vault.service.spec.ts)
• Unit: couverture 100 % chemins ArgoCD/Nexus/Registry/SonarQube
• E2E: vault.e2e-spec.ts zone.* case (gap)
• E2E: keycloak admin-role group sync (gap)
• E2E: argocd/nexus/registry/sonarqube rollback + retry paths (gaps)
• Playwright: create-project full plugin sync journey
• Socle cahier: cross-service scenarios inscrits dans Tests Fonctionnels/

Gates de sortie

pnpm --filter @cpn-console/server-nestjs run lint && pnpm --filter @cpn-console/server-nestjs test vert + chaque sous-tâche clôturée (issue liée + test vert). E2E validé sur host connecté.

[shikanime]

Sous-tâche seed validée: issue zone.* emit gap ci-dessus. Baseline 578→586 (Node 24).

[shikanime]

Vague 1 livrée — 6 PRs draft (tests uniquement)

PR	Unité	Contenu
#2578	seed	Matrice de parité + vault zone unit/e2e (+8 tests)
#2579	u2	projectMember emit parity + garde anti-consommateur fantôme
#2580	u3	e2e gated repository.sync miroir GitLab
#2581	u3b	keycloak admin-role reconcile-only parity + garde émission
#2582	u10	e2e gated argocd ensureDeleteExternal / rollback
#2583	u11	nexus/sonarqube/registry chemins d'erreur externes (+13 tests)

Gate combinée : 600 passed / 64 skipped (arbre fusionné des 5 feuilles + seed, Node 24).

Blocants cutover tracés : #2575 (zone), #2576 (cluster), #2577 (harbor).
E2E gated non exécutés ici (hôtes intégration DNS-inaccessibles depuis le poste de dev) — à jouer sur hôte connecté / CI.

[shikanime]

Mise à jour — exécution de la stratégie de test

• fix(server-nestjs): protège createUser contre les descriptions GitLab non textuelles #2584 : correctif du 4ᵉ site cause.description non gardé (createUser) — même classe que 🐛 [BUG] - GitLab: TypeError sur error.cause.description à la création d'un dépôt #2544, helper existant réutilisé + spec de régression.
• test(server-nestjs): audit de parité de migration + parité unit/e2e vault-zone #2578 : enrichi d'une table de traçabilité cahier fonctionnel ↔ specs automatisées (PROJ/PLUG/GIT/ADMIN/TODO-003 ↔ PRs).

[shikanime]

Vague 2 livrée — couverture adversariale + acceptance utilisateur

PR	Cible	Contenu
#2585	vault	+40 tests : mapping VaultError, approle, secrets récursifs (client 35%→85%)
#2586	registry	+9 tests : caller-decides 409/500, pagination, retention NaN (client 23%→83%)
#2587	observability	+28 tests : plugin lifecycle, YAML round-trip idempotent (11%→92%)
#2588	utils partagés	+60 tests : http/config/members/vault/queries edges, 2 comportements documentés
#2589	playwright UAT	SECR-001 toggle secrets ; SECR-003 skip documenté

Gate combinée : 723 passed / 59 skipped / 0 failed (arbre fusionné des 4 branches nestjs, Node 24).
Couverture globale : 79.3% → 83.5% statements / 80.2% → 84.4% lines — cible ≥80% atteinte.
Merge queue manuel (#32905282108) : success, incluant les 4 shards Playwright sur images construites depuis la PR.

Méthode « don't trust, prove it » : chaque feuille a été revérifiée par le dispatcher dans son workspace avant PR ; deux bugs de spec (mock MSW multi-segments, ordre DFS) et un écart upstream (cron ? Quartz) corrigés/documentés pendant la revérification.