boucle-de-verification-springboot

Boucle de Vérification Spring Boot

À exécuter avant les PR, après des changements majeurs et en pré-déploiement.

Quand l'activer

• Avant d'ouvrir une pull request pour un service Spring Boot
• Après une refactoring majeure ou des mises à jour de dépendances
• Vérification pré-déploiement pour les environnements de staging ou de production
• Exécution complète du pipeline : build → lint → test → scan de sécurité
• Validation de la couverture de tests par rapport aux seuils

Phase 1 : Construction

mvn -T 4 clean verify -DskipTests
# ou
./gradlew clean assemble -x test

Si la construction échoue, arrêtez-vous et corrigez.

Phase 2 : Analyse Statique

Maven (plugins courants) :

mvn -T 4 spotbugs:check pmd:check checkstyle:check

Gradle (si configuré) :

./gradlew checkstyleMain pmdMain spotbugsMain

Phase 3 : Tests + Couverture

mvn -T 4 test
mvn jacoco:report   # vérifier 80%+ de couverture
# ou
./gradlew test jacocoTestReport

Rapport :

• Nombre total de tests, réussis/échoués
• % de couverture (lignes/branches)

Tests Unitaires

Tester la logique métier en isolation avec des dépendances mockées :

@ExtendWith(MockitoExtension.class)
class UserServiceTest {

  @Mock private UserRepository userRepository;
  @InjectMocks private UserService userService;

  @Test
  void createUser_validInput_returnsUser() {
    var dto = new CreateUserDto("Alice", "[email protected]");
    var expected = new User(1L, "Alice", "[email protected]");
    when(userRepository.save(any(User.class))).thenReturn(expected);

    var result = userService.create(dto);

    assertThat(result.name()).isEqualTo("Alice");
    verify(userRepository).save(any(User.class));
  }

  @Test
  void createUser_duplicateEmail_throwsException() {
    var dto = new CreateUserDto("Alice", "[email protected]");
    when(userRepository.existsByEmail(dto.email())).thenReturn(true);

    assertThatThrownBy(() -> userService.create(dto))
        .isInstanceOf(DuplicateEmailException.class);
  }
}

Tests d'Intégration avec Testcontainers

Tester avec une vraie base de données plutôt que H2 :

@SpringBootTest
@Testcontainers
class UserRepositoryIntegrationTest {

  @Container
  static PostgreSQLContainer<?> postgres = new PostgreSQLContainer<>("postgres:16-alpine")
      .withDatabaseName("testdb");

  @DynamicPropertySource
  static void configureProperties(DynamicPropertyRegistry registry) {
    registry.add("spring.datasource.url", postgres::getJdbcUrl);
    registry.add("spring.datasource.username", postgres::getUsername);
    registry.add("spring.datasource.password", postgres::getPassword);
  }

  @Autowired private UserRepository userRepository;

  @Test
  void findByEmail_existingUser_returnsUser() {
    userRepository.save(new User("Alice", "[email protected]"));

    var found = userRepository.findByEmail("[email protected]");

    assertThat(found).isPresent();
    assertThat(found.get().getName()).isEqualTo("Alice");
  }
}

Tests API avec MockMvc

Tester la couche contrôleur avec le contexte Spring complet :

@WebMvcTest(UserController.class)
class UserControllerTest {

  @Autowired private MockMvc mockMvc;
  @MockBean private UserService userService;

  @Test
  void createUser_validInput_returns201() throws Exception {
    var user = new UserDto(1L, "Alice", "[email protected]");
    when(userService.create(any())).thenReturn(user);

    mockMvc.perform(post("/api/users")
            .contentType(MediaType.APPLICATION_JSON)
            .content("""
                {"name": "Alice", "email": "[email protected]"}
                """))
        .andExpect(status().isCreated())
        .andExpect(jsonPath("$.name").value("Alice"));
  }

  @Test
  void createUser_invalidEmail_returns400() throws Exception {
    mockMvc.perform(post("/api/users")
            .contentType(MediaType.APPLICATION_JSON)
            .content("""
                {"name": "Alice", "email": "not-an-email"}
                """))
        .andExpect(status().isBadRequest());
  }
}

Phase 4 : Scan de Sécurité

# CVE des dépendances
mvn org.owasp:dependency-check-maven:check
# ou
./gradlew dependencyCheckAnalyze

# Secrets dans le code source
grep -rn "password\s*=\s*\"" src/ --include="*.java" --include="*.yml" --include="*.properties"
grep -rn "sk-\|api_key\|secret" src/ --include="*.java" --include="*.yml"

# Secrets (historique Git)
git secrets --scan  # si configuré

Problèmes de Sécurité Courants

# Vérifier les System.out.println (utiliser un logger à la place)
grep -rn "System\.out\.print" src/main/ --include="*.java"

# Vérifier les messages d'exception bruts dans les réponses
grep -rn "e\.getMessage()" src/main/ --include="*.java"

# Vérifier le CORS avec wildcard
grep -rn "allowedOrigins.*\*" src/main/ --include="*.java"

Phase 5 : Formatage/Lint (optionnel)

mvn spotless:apply   # si le plugin Spotless est utilisé
./gradlew spotlessApply

Phase 6 : Revue des Différences

git diff --stat
git diff

Liste de vérification :

• Pas de logs de débogage laissés (System.out, log.debug sans gardes)
• Erreurs et codes HTTP pertinents
• Transactions et validation présentes là où nécessaire
• Modifications de configuration documentées

Modèle de Sortie

RAPPORT DE VÉRIFICATION
=======================
Construction : [SUCCÈS/ÉCHEC]
Statique :     [SUCCÈS/ÉCHEC] (spotbugs/pmd/checkstyle)
Tests :        [SUCCÈS/ÉCHEC] (X/Y réussis, couverture Z%)
Sécurité :     [SUCCÈS/ÉCHEC] (CVE trouvées : N)
Diff :         [X fichiers modifiés]

Global :       [PRÊT / PAS PRÊT]

Problèmes à corriger :
1. ...
2. ...

Mode Continu

• Réexécuter les phases lors de changements significatifs ou toutes les 30 à 60 minutes lors de longues sessions
• Garder une boucle courte : mvn -T 4 test + spotbugs pour un retour rapide

Rappelez-vous : Un retour rapide vaut mieux que des surprises tardives. Gardez la porte stricte — traitez les avertissements comme des défauts dans les systèmes de production.