lionsdev/unionflow-server-impl-quarkus
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
Files
13da0d07bdca496b6ad3a3a922e0999723a62b00
unionflow-server-impl-quarkus/JACOCO_TESTS_MANQUANTS.md
dahoud 75a19988b0 Sync: code local unifié 
Synchronisation du code source local (fait foi).

Signed-off-by: lions dev Team
2026-03-15 16:25:40 +00:00

5.5 KiB
Raw Blame History

JaCoCo 100 % – Tests ajoutés et suites restantes

Ce qui a été fait

1. GlobalExceptionMapper (100 % branches)

  • Fichier : src/main/java/.../exception/GlobalExceptionMapper.java
  • Modifs : @ApplicationScoped pour l’injection en test ; ordre des instanceof dans mapJsonException : InvalidFormatException avant MismatchedInputException (InvalidFormatException étend MismatchedInputException).
  • Tests ajoutés dans GlobalExceptionMapperTest.java :
    • mapRuntimeException : RuntimeException, IllegalArgumentException, IllegalStateException, NotFoundException, WebApplicationException (message non vide, null, vide), fallback 500.
    • mapBadRequestException : message présent, message null.
    • mapJsonException : MismatchedInputException, InvalidFormatException, JsonMappingException, JsonParseException (cas par défaut), avec sous-classes/stubs pour les constructeurs Jackson protégés.
    • buildResponse : délégation 3 args → 4 args ; message null ; details null.

2. IdConverter (package util)

  • Fichier de test : src/test/java/.../util/IdConverterTest.java
  • Couverture : longToUUID (null, membre, organisation, cotisation, evenement, demandeaide, inscriptionevenement, type inconnu, casse), uuidToLong (null, valeur), organisationIdToUUID, membreIdToUUID, cotisationIdToUUID, evenementIdToUUID.

3. UnionFlowServerApplication

  • Fichier de test : src/test/java/.../UnionFlowServerApplicationTest.java
  • Vérification de l’injection du bean (pas de couverture de main() ni run() qui appellent Quarkus.waitForExit()).

4. AuthCallbackResource

  • Les tests REST sur /auth/callback ont été retirés : en environnement test la ressource renvoie 500 (exception dans le bloc try ou en aval). À retester après correction de la cause (ex. config OIDC, format de la réponse, etc.).

État actuel de la couverture (sans exclusions)

  • Instructions : ~44 %
  • Branches : ~32 %
  • Lignes : ~46 %
  • Méthodes : ~55 %
  • Seuils configurés : 1,00 (100 %) pour LINE, BRANCH, INSTRUCTION, METHOD sur le BUNDLE → le check JaCoCo échoue.

Suites de tests à ajouter pour viser 100 %

Les chiffres ci‑dessous sont issus du rapport JaCoCo (index par package). Pour chaque package, il faut ajouter ou compléter des tests jusqu’à couvrir toutes les lignes/branches/méthodes.

Package Instructions Branches À faire
dev.lions.unionflow.server.service 35 % 21 % ~40 classes, couvrir tous les services (DashboardServiceImpl, MembreService, CotisationService, etc.)
dev.lions.unionflow.server.resource 38 % 41 % ~33 resources REST : chaque endpoint et chaque branche (erreurs, paramètres, pagination)
dev.lions.unionflow.server.repository 59 % 46 % ~32 repositories : requêtes personnalisées, critères, cas null
dev.lions.unionflow.server.entity 70 % 50 % ~42 entités : getters/setters, @PrePersist, méthodes métier, listeners
dev.lions.unionflow.server.service.mutuelle.credit 7 % 0 % DemandeCreditService : tous les cas et branches
dev.lions.unionflow.server.service.mutuelle.epargne 18 % 0 % TransactionEpargneService, etc.
dev.lions.unionflow.server.security 30 % - RoleDebugFilter, autres filtres : tests d’intégration (filtre + requête REST)
dev.lions.unionflow.server.mapper (racine + sous-packages) 35–95 % 21–64 % Compléter les branches manquantes dans les mappers MapStruct (null, listes vides, champs optionnels)
de.lions.unionflow.server.auth 0 % 0 % AuthCallbackResource : corriger la 500 en test puis réécrire les tests REST
dev.lions.unionflow.server.util 0 % → couvert - IdConverter : fait
dev.lions.unionflow.server.client 0 % - UserServiceClient, RoleServiceClient : tests avec WireMock ou mock du client + services qui les utilisent
dev.lions.unionflow.server 0 % - UnionFlowServerApplication : main/run non couverts (blocage sur waitForExit)

En pratique, il faut :

  • Services : pour chaque méthode publique, scénarios nominal, erreurs (exceptions, not found), paramètres null/optionnels, et chaque branche (if/else, try/catch).
  • Resources : pour chaque @GET/@POST/…, au moins 200, 404, 400, 401/403 si applicable, et corps de requête/réponse.
  • Repositories : tests avec base H2 et données de test pour chaque requête dérivée ou @Query.
  • Entités : instanciation, setters, callbacks JPA, méthodes métier.
  • Mappers : entité → DTO, DTO → entité, listes, champs null.
  • Filtres / clients : soit tests d’intégration (REST + filtre), soit tests unitaires avec mocks (ContainerRequestContext, client REST mocké).

Recommandation

  • Option A – Build vert avec seuils réalistes :
    Remonter temporairement les seuils JaCoCo (ex. 0,45 en LINE/INSTRUCTION, 0,32 en BRANCH) ou réintroduire des exclusions ciblées (entités, générés MapStruct, *Application) pour que la build passe, puis augmenter progressivement la couverture par packages.

  • Option B – Viser 100 % sans exclusions :
    Continuer à ajouter des tests package par package en s’appuyant sur le rapport HTML JaCoCo (target/site/jacoco/index.html) et sur ce fichier, jusqu’à atteindre 1,00 sur tout le bundle.

Dernière mise à jour : suite aux ajouts GlobalExceptionMapper, IdConverter, UnionFlowServerApplication et correction de l’ordre mapJsonException.