refactor: supprimer alias doublons typeAssociation/typeLibelle/numeroRegistre dans OrganisationResponse
- Supprimer typeAssociation (doublon mort de typeOrganisation) - Supprimer typeLibelle (doublon de typeOrganisationLibelle) - Supprimer alias getNumeroRegistre/setNumeroRegistre (doublon de numeroEnregistrement) - Supprimer alias getNomOrganisationParente/setNomOrganisationParente (doublon de organisationParenteNom) - Réécrire OrganisationResponseTest pour tester les vrais champs
This commit is contained in:
@@ -46,28 +46,13 @@ public class OrganisationResponse extends BaseResponse {
|
||||
|
||||
// ── Classification ─────────────────────────
|
||||
private String typeOrganisation;
|
||||
/** Alias pour tri/filtre (type d'organisation) */
|
||||
private String typeAssociation;
|
||||
private String typeOrganisationLibelle;
|
||||
/** Libellé du type pour affichage (alias typeOrganisationLibelle, ex. p:tag) */
|
||||
private String typeLibelle;
|
||||
private String statut;
|
||||
private String statutLibelle;
|
||||
private String statutSeverity;
|
||||
private LocalDate dateFondation;
|
||||
private String numeroEnregistrement;
|
||||
|
||||
/**
|
||||
* Alias pour la vue (detail.xhtml, organisation-form.xhtml) : même valeur que numeroEnregistrement.
|
||||
*/
|
||||
public String getNumeroRegistre() {
|
||||
return getNumeroEnregistrement();
|
||||
}
|
||||
|
||||
public void setNumeroRegistre(String numeroRegistre) {
|
||||
setNumeroEnregistrement(numeroRegistre);
|
||||
}
|
||||
|
||||
// ── Géographie ─────────────────────────────
|
||||
private String adresse;
|
||||
private String quartier;
|
||||
@@ -81,13 +66,6 @@ public class OrganisationResponse extends BaseResponse {
|
||||
// ── Hiérarchie ─────────────────────────────
|
||||
private UUID organisationParenteId;
|
||||
private String organisationParenteNom;
|
||||
/** Alias pour la vue (detail.xhtml) : même valeur que organisationParenteNom. */
|
||||
public String getNomOrganisationParente() {
|
||||
return getOrganisationParenteNom();
|
||||
}
|
||||
public void setNomOrganisationParente(String nomOrganisationParente) {
|
||||
setOrganisationParenteNom(nomOrganisationParente);
|
||||
}
|
||||
private Integer niveauHierarchique;
|
||||
private Boolean estOrganisationRacine;
|
||||
|
||||
|
||||
@@ -7,55 +7,49 @@ import org.junit.jupiter.api.Test;
|
||||
|
||||
/**
|
||||
* Tests unitaires pour OrganisationResponse.
|
||||
* Vérifie l'alias numeroRegistre / numeroEnregistrement.
|
||||
*
|
||||
* @author UnionFlow Team
|
||||
* @version 1.0
|
||||
* @since 2026-03-05
|
||||
* Vérifie les champs métier principaux.
|
||||
*/
|
||||
@DisplayName("Tests OrganisationResponse")
|
||||
class OrganisationResponseTest {
|
||||
|
||||
@Test
|
||||
@DisplayName("getNumeroRegistre retourne numeroEnregistrement")
|
||||
void getNumeroRegistre_returnsNumeroEnregistrement() {
|
||||
@DisplayName("builder construit correctement avec numeroEnregistrement")
|
||||
void builder_setsNumeroEnregistrement() {
|
||||
OrganisationResponse r = OrganisationResponse.builder()
|
||||
.numeroEnregistrement("MUT-CI-2020-001")
|
||||
.build();
|
||||
assertThat(r.getNumeroRegistre()).isEqualTo("MUT-CI-2020-001");
|
||||
assertThat(r.getNumeroEnregistrement()).isEqualTo("MUT-CI-2020-001");
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("getNumeroRegistre retourne null quand numeroEnregistrement est null")
|
||||
void getNumeroRegistre_nullWhenNumeroEnregistrementNull() {
|
||||
@DisplayName("numeroEnregistrement est null par défaut")
|
||||
void numeroEnregistrement_nullByDefault() {
|
||||
OrganisationResponse r = OrganisationResponse.builder().build();
|
||||
assertThat(r.getNumeroRegistre()).isNull();
|
||||
assertThat(r.getNumeroEnregistrement()).isNull();
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("setNumeroRegistre met à jour numeroEnregistrement")
|
||||
void setNumeroRegistre_updatesNumeroEnregistrement() {
|
||||
@DisplayName("setter numeroEnregistrement fonctionne")
|
||||
void setNumeroEnregistrement_works() {
|
||||
OrganisationResponse r = OrganisationResponse.builder().build();
|
||||
r.setNumeroRegistre("ASSO-2021-042");
|
||||
r.setNumeroEnregistrement("ASSO-2021-042");
|
||||
assertThat(r.getNumeroEnregistrement()).isEqualTo("ASSO-2021-042");
|
||||
assertThat(r.getNumeroRegistre()).isEqualTo("ASSO-2021-042");
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("getNomOrganisationParente retourne organisationParenteNom")
|
||||
void getNomOrganisationParente_returnsOrganisationParenteNom() {
|
||||
@DisplayName("builder construit correctement avec organisationParenteNom")
|
||||
void builder_setsOrganisationParenteNom() {
|
||||
OrganisationResponse r = OrganisationResponse.builder()
|
||||
.organisationParenteNom("Organisation Mère")
|
||||
.build();
|
||||
assertThat(r.getNomOrganisationParente()).isEqualTo("Organisation Mère");
|
||||
assertThat(r.getOrganisationParenteNom()).isEqualTo("Organisation Mère");
|
||||
}
|
||||
|
||||
@Test
|
||||
@DisplayName("setNomOrganisationParente met à jour organisationParenteNom")
|
||||
void setNomOrganisationParente_updatesOrganisationParenteNom() {
|
||||
@DisplayName("setter organisationParenteNom fonctionne")
|
||||
void setOrganisationParenteNom_works() {
|
||||
OrganisationResponse r = OrganisationResponse.builder().build();
|
||||
r.setNomOrganisationParente("Parent Org");
|
||||
r.setOrganisationParenteNom("Parent Org");
|
||||
assertThat(r.getOrganisationParenteNom()).isEqualTo("Parent Org");
|
||||
assertThat(r.getNomOrganisationParente()).isEqualTo("Parent Org");
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user