test(server-nestjs): cas limites du module Observability (plugin, client YAML) - #2587
test(server-nestjs): cas limites du module Observability (plugin, client YAML)#2587shikanime wants to merge 3 commits into
Conversation
…nment Co-authored-by: Automata <automata@shikanime.studio>
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
shikanime
changed the title
…ility specs Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr> Change-Id: I35c94382a862ea25935947720c4a05916a6a6964
Annotate the repo const with CondensedProjectSchemaWith<'id'> instead of as unknown as, build the created-repo value via makeProjectSchema instead of as ProjectSchema, and read the commit content arg without as string, per server-nestjs vitest convention. Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr> Change-Id: Ia05cf77e80581a7513da8f83fa1604366a6a6964
|
1 New Issue
0 Fixed Issues
0 Accepted Issues
No data about coverage (39.30% Estimated after merge)
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.
Couverture adversariale autonome du module Observability, trois commits dans le périmètre, aucun chevauchement avec la vague MSW (#2659/#2669/#2670) ni avec le trio parité. ✨ Éloge — les cas limites plugin/envoyeur YAML couvrent les entrées malformées au lieu du seul chemin nominal.
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.
Couverture adversariale autonome du module Observability, trois commits dans le périmètre, aucun chevauchement avec la vague MSW (#2659/#2669/#2670) ni avec le trio parité. ✨ Éloge — les cas limites plugin/envoyeur YAML couvrent les entrées malformées au lieu du seul chemin nominal.
2 participants
Issues liées
#2574
Quel est le comportement actuel ?
Les services plugin et client du module observability étaient à 11% et 12% de lignes couvertes : le cycle de vie Grafana (URLs par type d'environnement, descripteur de config) et le round-trip YAML du fichier de valeurs n'étaient pas testés.
Quel est le nouveau comportement ?
+2 specs / 28 tests :
observability-plugin.service.spec.ts: URLs hprod/prod selon les stages, erreur projet introuvable, contrat du descripteur (switch global admin-writable, instances read-only) ;observability-client.service.spec.ts: répo values existant/créé, fallback init + clone sûr, garde zod sur YAML invalide, idempotence updateProjectConfig (pas de commit si identique), suppression ciblée préservant les projets voisins.36/36 tests verts sur Node 24.
Cette PR introduit-elle un breaking change ?
Non.