diff --git a/src/main/java/dev/lions/unionflow/server/api/dto/organisation/response/OrganisationResponse.java b/src/main/java/dev/lions/unionflow/server/api/dto/organisation/response/OrganisationResponse.java index 363d3a9..eb7cf19 100644 --- a/src/main/java/dev/lions/unionflow/server/api/dto/organisation/response/OrganisationResponse.java +++ b/src/main/java/dev/lions/unionflow/server/api/dto/organisation/response/OrganisationResponse.java @@ -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; diff --git a/src/test/java/dev/lions/unionflow/server/api/dto/organisation/response/OrganisationResponseTest.java b/src/test/java/dev/lions/unionflow/server/api/dto/organisation/response/OrganisationResponseTest.java index e22abdb..ab6c069 100644 --- a/src/test/java/dev/lions/unionflow/server/api/dto/organisation/response/OrganisationResponseTest.java +++ b/src/test/java/dev/lions/unionflow/server/api/dto/organisation/response/OrganisationResponseTest.java @@ -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"); } }