[SteveTetchoup26]

Quoi et pourquoi

1. Image Oracle 23.26.3, partout à la fois. La PR Dependabot #33 ne modifiait que deploy/compose/docker-compose.yml. La même image est utilisée par Testcontainers dans SystemInfoIT, que Dependabot ne lit pas : fusionnée seule, #33 aurait fait tourner la base locale en 23.26.3 et les tests de la CI en 23.26.2, contraire à l'ADR-003 (« même version d'Oracle partout »). Cette PR monte les trois références ensemble : docker-compose.yml, SystemInfoIT.java et backend/README.md. Correctif (ADR-003, principe 7) : l'ADR n'est pas modifié.

2. Remarque SonarQube java:S5778. Dans application_account_cannot_modify_reference_data, la lambda de assertThatThrownBy contenait deux appels (sql(...) puis update()) : si le premier avait levé une DataAccessException, le test aurait réussi pour une mauvaise raison. La requête est préparée hors de la lambda, qui ne contient plus que l'appel à la base.

Closes #48

Comment vérifier

cd backend && ./mvnw verify   # Testcontainers démarre gvenzl/oracle-free:23.26.3-slim-faststart

Vérifié en local : 4 tests unitaires et 2 tests d'intégration sur Oracle 23.26.3 (ORA-41900 toujours levée pour ITSM_APP). L'étiquette existe en amd64 et arm64 sur Docker Hub.

Base locale : pour passer en 23.26.3, recréer le conteneur. La base ne contient que des données de test :

cd deploy/compose && docker compose down -v && docker compose up -d

Checklist d'auto-revue (Definition of Done)

• Les critères d'acceptation sont couverts par des tests automatisés
• Le pipeline est vert, contrôle qualité compris : en cours
• Le contrat OpenAPI est à jour (si l'API change) : sans objet
• Les migrations Flyway sont versionnées (si le schéma change) : sans objet
• Les logs et métriques utiles au diagnostic sont présents : sans objet
• Aucun secret, aucune donnée personnelle dans le code ni dans les logs
• Un ADR est rédigé si une décision structurante a été prise : sans objet (correctif prévu par l'ADR-003)
• La documentation concernée est à jour (backend/README.md)
• Je peux expliquer chaque ligne de cette PR

Points d'attention pour la revue

• La référence de l'image reste dupliquée (Compose et test). Une source unique, lisible par Dependabot, est à étudier avec I0-06 (images Docker).
• Le filtre du job backend inclut deploy/compose/oracle/** mais pas docker-compose.yml : ici, le job tourne parce que SystemInfoIT.java change.

[sonarqubecloud]

Quality Gate passed for 'itsm-backend'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud