feat(commercial) : catégories de type Adresse pour les blocs adresse (client + fournisseur) (#147)

## Objectif Introduit un `CategoryType` dédié **ADRESSE** (module Catalog) consommé par le champ « Catégorie » des blocs adresse, en remplacement de la réutilisation détournée des types CLIENT / FOURNISSEUR. ## Changements **Backend** - Migration de seed du type ADRESSE + 6 catégories : Siège, Contact issues, Facturation, Livraison, Approvisionnement, Méthaniseur (idempotente, réversible) ; fixtures alignées. - `ClientAddress` : validation blacklist (DISTRIBUTEUR/COURTIER) remplacée par une whitelist « catégories de type ADRESSE uniquement ». - `SupplierAddress` : type requis FOURNISSEUR → ADRESSE (le bloc principal fournisseur reste en FOURNISSEUR). **Frontend** - Ref dédiée `addressCategories` (`?typeCode=ADRESSE`) dans les composables référentiels client et fournisseur. - Pages new/edit client et fournisseur câblées sur les blocs adresse. **Tests** - `CategoryAdresseSeedTest` (miroir du test PRESTATAIRE). - Adaptation des tests d'adresse client/fournisseur (sémantique whitelist ADRESSE) + helper `createAddressCategory()`. ## Vérifications - Back : suites Catalog + Architecture + adresse/fournisseur vertes (le flake JWT connu du hook est sans rapport, tests verts en isolation). - Front : Vitest vert (composables référentiels + ciblés). - php-cs-fixer : 0 correction ; eslint : OK. Reviewed-on: #147 Co-authored-by: tristan <tristan@yuno.malio.fr> Co-committed-by: tristan <tristan@yuno.malio.fr>
2026-06-25 07:26:21 +00:00
parent 2e50a760c6
commit efded9fd40
22 changed files with 444 additions and 168 deletions
@@ -36,10 +36,10 @@ abstract class AbstractCommercialApiTestCase extends AbstractApiTestCase
    protected const string TEST_CATEGORY_PREFIX = 'test_cli_cat_';

    /**
-     * Codes pilotant les RG (RG-1.03 distributor/broker, RG-1.29 adresse) : ils
-     * doivent matcher exactement, donc createCategory() les fetch-or-create par
-     * code. Les autres codes sont traites comme de simples libelles generiques et
-     * produisent une categorie a code UNIQUE (cf. createCategory).
+     * Codes pilotant les RG (RG-1.03 distributor/broker) : ils doivent matcher
+     * exactement, donc createCategory() les fetch-or-create par code. Les autres
+     * codes sont traites comme de simples libelles generiques et produisent une
+     * categorie a code UNIQUE (cf. createCategory).
     */
    private const array RG_EXACT_CODES = ['DISTRIBUTEUR', 'COURTIER'];

@@ -75,6 +75,47 @@ abstract class AbstractCommercialApiTestCase extends AbstractApiTestCase
        return $type;
    }

+    /**
+     * Recupere (ou cree) le type ADRESSE (categories des blocs adresse). Idempotent
+     * via l'unicite de category_type.code. Laisse en place au tearDown.
+     */
+    protected function addressCategoryType(): CategoryType
+    {
+        $em       = $this->getEm();
+        $existing = $em->getRepository(CategoryType::class)->findOneBy(['code' => 'ADRESSE']);
+        if (null !== $existing) {
+            return $existing;
+        }
+
+        $type = new CategoryType();
+        $type->setCode('ADRESSE');
+        $type->setLabel('Adresse');
+        $em->persist($type);
+        $em->flush();
+
+        return $type;
+    }
+
+    /**
+     * Cree une Category de test de type ADRESSE (autorisee sur un bloc adresse).
+     * Code UNIQUE (suffixe aleatoire) : les categories d'adresse ne pilotent aucune
+     * RG par code, deux appels produisent donc deux categories distinctes.
+     */
+    protected function createAddressCategory(): Category
+    {
+        $em     = $this->getEm();
+        $suffix = substr(bin2hex(random_bytes(4)), 0, 8);
+
+        $category = new Category();
+        $category->setName(self::TEST_CATEGORY_PREFIX.'adresse_'.$suffix);
+        $category->setCode('ADRESSE_'.strtoupper($suffix));
+        $category->addCategoryType($this->addressCategoryType());
+        $em->persist($category);
+        $em->flush();
+
+        return $category;
+    }
+
    /**
     * Cree une Category de test sous le type unique CLIENT (ERP-78).
     *
@@ -134,8 +134,8 @@ abstract class AbstractSupplierApiTestCase extends AbstractCommercialApiTestCase
     * Seede un fournisseur COMPLET (sans passer par l'API — validations
     * applicatives non rejouees mais CHECK BDD respectes) : onglet Information
     * rempli, bloc comptable non nul (SIREN + refs), >= 1 RIB, >= 1 adresse
-     * multi-sites (>= 2 sites, triageProvider=true) avec >= 1 categorie
-     * FOURNISSEUR, >= 1 contact, >= 1 categorie sur le fournisseur. Sert de socle
+     * multi-sites (>= 2 sites, triageProvider=true) avec >= 1 categorie de type
+     * ADRESSE, >= 1 contact, >= 1 categorie FOURNISSEUR sur le fournisseur. Sert de socle
     * au contrat de serialisation et a la DoD (§ 4.0.bis).
     *
     * @param string $paymentTypeCode code du type de reglement a poser (defaut LCR,
@@ -202,7 +202,9 @@ abstract class AbstractSupplierApiTestCase extends AbstractCommercialApiTestCase
        foreach ($sites as $site) {
            $address->addSite($site);
        }
-        $address->addCategory($this->supplierCategory('NEGOCIANT'));
+        // Categorie de bloc adresse : type ADRESSE (et non FOURNISSEUR — celui-ci
+        // reste sur le bloc principal du fournisseur).
+        $address->addCategory($this->createAddressCategory());
        $address->addContact($contact);
        $supplier->addAddress($address);
        $em->persist($address);
@@ -15,8 +15,8 @@ use App\Module\Sites\Domain\Entity\Site;
 *  - RG-1.06 / RG-1.07 / RG-1.08 : exclusivite is_prospect vs
 *    is_delivery / is_billing ;
 *  - RG-1.11 : billing_email obligatoire ssi is_billing ;
- *  - RG-1.29 (ERP-78) : les categories de code DISTRIBUTEUR / COURTIER sont
- *    interdites sur une adresse (-> 422) ; toute autre categorie est acceptee.
+ *  - categorie d'adresse : seules les categories de type ADRESSE sont acceptees
+ *    (-> 422 sinon), au moins une est obligatoire.
 *
 * Depuis ERP-76, ces regles sont portees par des Assert\Callback sur l'entite
 * ClientAddress (mirror applicatif des CHECK Postgres) : la combinaison invalide
@@ -170,7 +170,7 @@ final class ClientAddressTest extends AbstractCommercialApiTestCase
        $this->skipIfSitesModuleDisabled();
        $client   = $this->createAdminClient();
        $seed     = $this->seedClient('Non Billing Empty Email');
-        $category = $this->createCategory('SECTEUR');
+        $category = $this->createAddressCategory();

        $client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
            'headers' => ['Content-Type' => self::LD],
@@ -197,7 +197,7 @@ final class ClientAddressTest extends AbstractCommercialApiTestCase
        $this->skipIfSitesModuleDisabled();
        $client   = $this->createAdminClient();
        $seed     = $this->seedClient('Billing Two Emails');
-        $category = $this->createCategory('SECTEUR');
+        $category = $this->createAddressCategory();

        $client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
            'headers' => ['Content-Type' => self::LD],
@@ -225,7 +225,7 @@ final class ClientAddressTest extends AbstractCommercialApiTestCase
        $this->skipIfSitesModuleDisabled();
        $client   = $this->createAdminClient();
        $seed     = $this->seedClient('Secondary Email Non Billing');
-        $category = $this->createCategory('SECTEUR');
+        $category = $this->createAddressCategory();

        $body = $client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
            'headers' => ['Content-Type' => self::LD, 'Accept' => self::LD],
@@ -246,15 +246,16 @@ final class ClientAddressTest extends AbstractCommercialApiTestCase
    }

    /**
-     * RG-1.29 : poster une categorie de type DISTRIBUTEUR sur une adresse -> 422
-     * avec violation sur le champ `categories`.
+     * Une categorie qui n'est PAS de type ADRESSE (ici une categorie CLIENT) est
+     * refusee sur une adresse -> 422 avec violation sur le champ `categories`.
     */
-    public function testAddressRejectsDistributorCategory(): void
+    public function testAddressRejectsNonAddressCategory(): void
    {
        $this->skipIfSitesModuleDisabled();
-        $client   = $this->createAdminClient();
-        $seed     = $this->seedClient('Address Distributor Cat');
-        $category = $this->createCategory('DISTRIBUTEUR');
+        $client = $this->createAdminClient();
+        $seed   = $this->seedClient('Address Non Address Cat');
+        // Categorie de type CLIENT (et non ADRESSE) -> doit etre refusee sur l'adresse.
+        $category = $this->createCategory('SECTEUR');

        $client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
            'headers' => ['Content-Type' => self::LD],
@@ -270,70 +271,20 @@ final class ClientAddressTest extends AbstractCommercialApiTestCase

        self::assertResponseStatusCodeSame(422);
        self::assertStringContainsString(
-            'Type de catégorie non autorisé sur une adresse.',
+            'Type de catégorie non autorisé (ADRESSE attendu).',
            (string) $client->getResponse()->getContent(false),
        );
    }

    /**
-     * RG-1.29 : poster une categorie de type COURTIER sur une adresse -> 422.
+     * Une categorie de type ADRESSE est acceptee sur une adresse -> 201.
     */
-    public function testAddressRejectsBrokerCategory(): void
+    public function testAddressAcceptsAddressCategory(): void
    {
        $this->skipIfSitesModuleDisabled();
        $client   = $this->createAdminClient();
-        $seed     = $this->seedClient('Address Broker Cat');
-        $category = $this->createCategory('COURTIER');
-
-        $client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
-            'headers' => ['Content-Type' => self::LD],
-            'json'    => [
-                'isDelivery' => true,
-                'postalCode' => '86100',
-                'city'       => 'Châtellerault',
-                'street'     => '1 rue du Test',
-                'sites'      => [$this->firstSiteIri()],
-                'categories' => ['/api/categories/'.$category->getId()],
-            ],
-        ]);
-
-        self::assertResponseStatusCodeSame(422);
-    }
-
-    /**
-     * RG-1.29 : une categorie de type SECTEUR est autorisee sur une adresse.
-     */
-    public function testAddressAcceptsSectorCategory(): void
-    {
-        $this->skipIfSitesModuleDisabled();
-        $client   = $this->createAdminClient();
-        $seed     = $this->seedClient('Address Sector Cat');
-        $category = $this->createCategory('SECTEUR');
-
-        $client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
-            'headers' => ['Content-Type' => self::LD],
-            'json'    => [
-                'isDelivery' => true,
-                'postalCode' => '86100',
-                'city'       => 'Châtellerault',
-                'street'     => '1 rue du Test',
-                'sites'      => [$this->firstSiteIri()],
-                'categories' => ['/api/categories/'.$category->getId()],
-            ],
-        ]);
-
-        self::assertResponseStatusCodeSame(201);
-    }
-
-    /**
-     * RG-1.29 : une categorie de type AUTRE est autorisee sur une adresse.
-     */
-    public function testAddressAcceptsOtherCategory(): void
-    {
-        $this->skipIfSitesModuleDisabled();
-        $client   = $this->createAdminClient();
-        $seed     = $this->seedClient('Address Other Cat');
-        $category = $this->createCategory('AUTRE');
+        $seed     = $this->seedClient('Address Address Cat');
+        $category = $this->createAddressCategory();

        $client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
            'headers' => ['Content-Type' => self::LD],
@@ -385,7 +336,7 @@ final class ClientAddressTest extends AbstractCommercialApiTestCase
        $this->skipIfSitesModuleDisabled();
        $client   = $this->createAdminClient();
        $seed     = $this->seedClient('Address No Type');
-        $category = $this->createCategory('SECTEUR');
+        $category = $this->createAddressCategory();

        $body = $client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
            'headers' => ['Content-Type' => self::LD, 'Accept' => self::LD],
@@ -413,7 +364,7 @@ final class ClientAddressTest extends AbstractCommercialApiTestCase
        $this->skipIfSitesModuleDisabled();
        $client   = $this->createAdminClient();
        $seed     = $this->seedClient('Address Broker Type');
-        $category = $this->createCategory('SECTEUR');
+        $category = $this->createAddressCategory();

        $client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
            'headers' => ['Content-Type' => self::LD],
@@ -435,7 +386,7 @@ final class ClientAddressTest extends AbstractCommercialApiTestCase
        $this->skipIfSitesModuleDisabled();
        $client   = $this->createAdminClient();
        $seed     = $this->seedClient('Address Distributor Type');
-        $category = $this->createCategory('SECTEUR');
+        $category = $this->createAddressCategory();

        $client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
            'headers' => ['Content-Type' => self::LD],
@@ -462,7 +413,7 @@ final class ClientAddressTest extends AbstractCommercialApiTestCase
        $this->skipIfSitesModuleDisabled();
        $client   = $this->createAdminClient();
        $seed     = $this->seedClient('Address Broker Mix');
-        $category = $this->createCategory('SECTEUR');
+        $category = $this->createAddressCategory();

        $body = $client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
            'headers' => ['Content-Type' => self::LD, 'Accept' => self::LD],
@@ -203,7 +203,7 @@ final class ClientSubResourceApiTest extends AbstractCommercialApiTestCase
        $client   = $this->createAdminClient();
        $seed     = $this->seedClient('Address Host');
        $siteIri  = $this->firstSiteIri();
-        $category = $this->createCategory('SECTEUR');
+        $category = $this->createAddressCategory();

        $data = $client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
            'headers' => ['Content-Type' => self::LD],
@@ -276,7 +276,7 @@ final class ClientSubResourceApiTest extends AbstractCommercialApiTestCase
        $client   = $this->createAdminClient();
        $seed     = $this->seedClient('Addr Multi');
        $siteIri  = $this->firstSiteIri();
-        $category = $this->createCategory('SECTEUR');
+        $category = $this->createAddressCategory();
        $this->seedAddress($seed, 'Bordeaux');
        $this->seedAddress($seed, 'Lyon');

@@ -305,7 +305,7 @@ final class ClientSubResourceApiTest extends AbstractCommercialApiTestCase
        $this->skipIfSitesModuleDisabled();
        $client   = $this->createAdminClient();
        $siteIri  = $this->firstSiteIri();
-        $category = $this->createCategory('SECTEUR');
+        $category = $this->createAddressCategory();

        $client->request('POST', '/api/clients/999999/addresses', [
            'headers' => ['Content-Type' => self::LD, 'Accept' => self::LD],
@@ -106,7 +106,7 @@ final class SupplierSubResourceApiTest extends AbstractSupplierApiTestCase
        $this->skipIfSitesModuleDisabled();
        $client   = $this->createAdminClient();
        $seed     = $this->seedSupplier('Address Host');
-        $category = $this->supplierCategory('NEGOCIANT');
+        $category = $this->createAddressCategory();

        $data = $client->request('POST', '/api/suppliers/'.$seed->getId().'/addresses', [
            'headers' => ['Content-Type' => self::LD],
@@ -174,7 +174,7 @@ final class SupplierSubResourceApiTest extends AbstractSupplierApiTestCase
        $this->skipIfSitesModuleDisabled();
        $client   = $this->createAdminClient();
        $seed     = $this->seedSupplier('Address Incoherent');
-        $category = $this->supplierCategory('NEGOCIANT');
+        $category = $this->createAddressCategory();

        // RG-2.05 : pas de controle strict de coherence CP/ville cote serveur.
        $client->request('POST', '/api/suppliers/'.$seed->getId().'/addresses', [
@@ -222,7 +222,7 @@ final class SupplierSubResourceApiTest extends AbstractSupplierApiTestCase
        $client   = $this->createAdminClient();
        $seed     = $this->seedSupplier('Address Types');
        $siteIri  = $this->firstSiteIri();
-        $category = $this->supplierCategory('NEGOCIANT');
+        $category = $this->createAddressCategory();

        foreach (['PROSPECT', 'DEPART', 'RENDU'] as $type) {
            $client->request('POST', '/api/suppliers/'.$seed->getId().'/addresses', [
@@ -240,12 +240,12 @@ final class SupplierSubResourceApiTest extends AbstractSupplierApiTestCase
        }
    }

-    public function testPostAddressWithNonFournisseurCategoryReturns422(): void
+    public function testPostAddressWithNonAddressCategoryReturns422(): void
    {
        $this->skipIfSitesModuleDisabled();
        $client = $this->createAdminClient();
        $seed   = $this->seedSupplier('Address Bad Cat');
-        // categorie de type CLIENT -> interdite sur une adresse fournisseur.
+        // categorie de type CLIENT (et non ADRESSE) -> interdite sur une adresse.
        $clientTypedCategory = $this->createCategory('SECTEUR');

        $response = $client->request('POST', '/api/suppliers/'.$seed->getId().'/addresses', [
@@ -260,7 +260,7 @@ final class SupplierSubResourceApiTest extends AbstractSupplierApiTestCase
            ],
        ]);

-        // RG-2.10 -> 422 rattachee a categories.
+        // Categorie hors type ADRESSE -> 422 rattachee a categories.
        self::assertResponseStatusCodeSame(422);
        self::assertArrayHasKey('categories', $this->violationsByPath($response->toArray(false)));
    }