test(server-nestjs): parité des chemins d'erreur externes nexus/sonarqube/registry - #2583
test(server-nestjs): parité des chemins d'erreur externes nexus/sonarqube/registry#2583shikanime wants to merge 4 commits into
Conversation
yorha-operator
left a comment
yorha-operator
left a comment
There was a problem hiding this comment.
Verdict : Approuvé — la PR reste en draft dans l'attente de la décision d'atterrissage.
Delta propre par rapport au socle : parité des chemins d'erreur externes sur trois clients, commits tous dans le périmètre. Point de séquencement : nexus.service.spec.ts est aussi réécrit par #2669 (vague MSW) — atterrir la vague parité (#2578 → #2579 → celle-ci) avant #2659/#2669/#2670, sinon restack nécessaire. ✨ Éloge — verrouiller le contrat d'ordre des chemins d'erreur par client externe révélera immédiatement toute divergence de mapping lors des prochaines migrations.
yorha-operator
left a comment
yorha-operator
left a comment
There was a problem hiding this comment.
Verdict : Approuvé — la PR reste en draft dans l'attente de la décision d'atterrissage.
Delta propre par rapport au socle : parité des chemins d'erreur externes sur trois clients, commits tous dans le périmètre. Point de séquencement : nexus.service.spec.ts est aussi réécrit par #2669 (vague MSW) — atterrir la vague parité (#2578 → #2579 → celle-ci) avant #2659/#2669/#2670, sinon restack nécessaire. ✨ Éloge — verrouiller le contrat d'ordre des chemins d'erreur par client externe révélera immédiatement toute divergence de mapping lors des prochaines migrations.
…h parity Co-authored-by: Automata <automata@shikanime.studio> Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr> Change-Id: I7a048db2689f75bd0632dd1e3c23aff16a6a6964
Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr> Change-Id: Iaf48f76267c3288ed1801b7f70327eed6a6a6964
Replace NexusMavenHostedRepository as any with a fully-typed repository object and remove redundant owner as any casts in sonarqube fixtures, per server-nestjs vitest convention. Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr> Change-Id: Ibdf4763351f9871d8e8f912f1201dbb26a6a6964
75f1368 to
9c6a447
Compare
|
0 New Issues
0 Fixed Issues
0 Accepted Issues
No data about coverage (25.00% Estimated after merge)2 participants
Issues liées
#2574
Quel est le comportement actuel ?
Les specs unitaires de nexus, sonarqube et registry ne couvrent pas les chemins d'erreur des appels externes (conflit 409 à la création de dépôt, erreurs transitoires 5xx, cleanup à la suppression) tels qu'ils se présentent côté legacy.
Quel est le nouveau comportement ?
nexus.service.spec.ts: +73 lignes — 409 idempotent à l'upsert, propagation 503, cleanup au delete.sonarqube.service.spec.ts: +72 lignes — verrouillage KO→retry/propagation et suppression projet.registry.service.spec.ts: +55 lignes — conflit/conflict-upsert et nettoyage mirror.84 tests verts (10 fichiers) sur Node 24 ; aucun changement de code produit, uniquement des specs verrouillant le comportement actuel.
Cette PR introduit-elle un breaking change ?
Non.