[ERP-78] Refonte taxonomie Catégories : type unique CLIENT + Category.code + RG-1.03/1.29 par code (#42)

Refonte de la taxonomie Catégories (décision produit 01/06) : le modèle est inversé. ## Modèle - **UN SEUL `category_type` : CLIENT**. `Distributeur` / `Courtier` / `Secteur` / `Autre` (+ catégories métier) deviennent des `Category` rattachées à CLIENT. - Filtrage métier sur un **`code` stable porté par `Category`** (NOT NULL, unique partiel `uq_category_code`), slug MAJUSCULE auto-généré du nom (`CategoryCodeGenerator`), figé à la création, exposé en **lecture seule**. ## Contenu - **M0** : `Category.code` (entité + migration corrective `Version20260602100000` au namespace racine + `COMMENT ON COLUMN` + catalogue + ligne `test-db-setup`). Retrofit `Version20260528120000` rendu conscient des colonnes. - **Seed** : type unique CLIENT, catégories codées (`Distributeur→DISTRIBUTEUR`, etc.), anciens types supprimés. Fixtures `CategoryType`/`Category`/`Client` alignées. - **RG-1.03** : `ClientProcessor::hasCategoryCode` — un distributor/broker doit porter la `Category` de code `DISTRIBUTEUR`/`COURTIER`. Filtre liste/export `categoryType` → `categoryCode`. - **RG-1.29** : `ClientAddress::validateCategoryCodes` — denylist des codes `DISTRIBUTEUR`/`COURTIER` sur une adresse (toute autre catégorie autorisée). - **Specs** M0/M1 (back + front) amendées. ## Tests `make php-cs-fixer-allow-risky` OK ; `make db-reset` OK (type unique CLIENT + 11 catégories codées, idempotent) ; `make test` **443 vert**. Ajouts : RG-1.03 courtier, génération/unicité/lecture-seule du code (`CategoryCodeTest`). ## Coordination - #76 (#500) : RG-1.29 réécrite ici sur le nouveau modèle ; #76 ne garde que le gap 2 (mapping CHECK adresse → 422), indépendant de la taxonomie. - ERP-68 (#486) : fixtures démo (déjà mergées via #41) adaptées ici au type unique CLIENT + codes. - Front #480–483 : selects Catégorie / distributeur / courtier basés sur le `code` (`?categoryCode=`), plus le type. Décisions actées avec le PO : `code` NOT NULL auto-généré (slug) ; périmètre complet (réécriture RG + fixtures déjà mergées). --------- Co-authored-by: Matthieu <contact@malio.fr> Reviewed-on: #42 Co-authored-by: THOLOT DECHENE Matthieu <matthieu@yuno.malio.fr> Co-committed-by: THOLOT DECHENE Matthieu <matthieu@yuno.malio.fr>
2026-06-02 08:00:42 +00:00
parent a668a8eb28
commit 00bd02858c
30 changed files with 866 additions and 188 deletions
@@ -83,6 +83,9 @@ abstract class AbstractCatalogApiTestCase extends AbstractApiTestCase
        $suffix   = substr(bin2hex(random_bytes(4)), 0, 8);
        $category = new Category();
        $category->setName($name ?? self::TEST_CATEGORY_PREFIX.$suffix);
+        // ERP-78 : code NOT NULL + unique parmi les actifs (uq_category_code).
+        // Nonce aleatoire -> unicite garantie entre seeds successifs du test.
+        $category->setCode('TEST_'.strtoupper($suffix));
        $category->setCategoryType($type);
        if (null !== $deletedAt) {
            $category->setDeletedAt($deletedAt);
@@ -0,0 +1,73 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Tests\Module\Catalog\Api;
+
+use App\Module\Catalog\Application\Service\CategoryCodeGenerator;
+use App\Shared\Infrastructure\Database\CategoryCodeSql;
+use Doctrine\DBAL\Connection;
+use PHPUnit\Framework\Attributes\DataProvider;
+use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;
+
+/**
+ * Garde-fou ERP-78 : l'expression SQL de slug (CategoryCodeSql, utilisee par le
+ * backfill de la migration Version20260602100000) doit produire EXACTEMENT le
+ * meme code que le generateur applicatif (CategoryCodeGenerator::slugify), sur
+ * tout le domaine de noms francais / Latin-1.
+ *
+ * Verrouille la cause racine du bug initial : deux implementations d'un meme
+ * slug qui derivent silencieusement (« Independant » -> IND_PENDANT cote SQL
+ * faute de translitteration des accents, vs INDEPENDANT cote PHP). On ne couvre
+ * volontairement PAS les ligatures (`Œ`, `ß`) : `translate()` est 1->1 et ne
+ * peut produire `OE`/`SS` ; elles sont hors du domaine des categories CLIENT.
+ *
+ * @internal
+ */
+final class CategoryCodeSqlSlugTest extends KernelTestCase
+{
+    /**
+     * Noms representatifs du domaine reel : accents, cedille, apostrophe,
+     * separateurs varies, parentheses, majuscules accentuees.
+     *
+     * @return iterable<string, array{string}>
+     */
+    public static function nameProvider(): iterable
+    {
+        yield 'sans accent'         => ['Distributeur'];
+        yield 'tiret'               => ['Agro-alimentaire'];
+        yield 'slash'               => ['Transport/Logistique'];
+        yield 'accent aigu'         => ['Indépendant'];
+        yield 'apostrophe + accent' => ["L'Oréal"];
+        yield 'esperluette'         => ['Forêt & Bûcheron'];
+        yield 'cedille majuscule'   => ['Ça va'];
+        yield 'accents multiples'   => ['Naïve façade'];
+        yield 'circonflexe'         => ["Côte d'Azur"];
+        yield 'parentheses'         => ['Zone (Sud)'];
+    }
+
+    #[DataProvider('nameProvider')]
+    public function testSqlSlugMatchesPhpSlug(string $name): void
+    {
+        self::bootKernel();
+        $container = self::getContainer();
+
+        /** @var Connection $conn */
+        $conn = $container->get('doctrine')->getConnection();
+        /** @var CategoryCodeGenerator $generator */
+        $generator = $container->get(CategoryCodeGenerator::class);
+
+        // Evaluation pure de l'expression (aucune table requise) : le nom est
+        // passe en parametre lie a la place de la colonne.
+        $sqlSlug = $conn->fetchOne(
+            'SELECT '.CategoryCodeSql::slugExpression(':name'),
+            ['name' => $name],
+        );
+
+        self::assertSame(
+            $generator->slugify($name),
+            $sqlSlug,
+            sprintf('SQL et PHP doivent produire le meme slug pour "%s".', $name),
+        );
+    }
+}
@@ -0,0 +1,90 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Tests\Module\Catalog\Api;
+
+/**
+ * Tests ERP-78 : le `code` technique stable de Category.
+ *
+ * Cas couverts :
+ *  - POST : le code est auto-genere (slug MAJUSCULE du nom) et expose en lecture ;
+ *  - le code est en lecture seule : un `code` envoye dans le payload est ignore
+ *    (genere depuis le nom) ;
+ *  - deux noms produisant le meme slug recoivent des codes distincts (suffixe).
+ *
+ * @internal
+ */
+final class CategoryCodeTest extends AbstractCatalogApiTestCase
+{
+    public function testPostGeneratesAndExposesCode(): void
+    {
+        $type   = $this->createCategoryType();
+        $client = $this->createAdminClient();
+
+        $response = $client->request('POST', '/api/categories', [
+            'headers' => ['Content-Type' => 'application/ld+json'],
+            'json'    => [
+                'name'         => self::TEST_CATEGORY_PREFIX.'Agro-alimentaire',
+                'categoryType' => '/api/category_types/'.$type->getId(),
+            ],
+        ]);
+        self::assertResponseStatusCodeSame(201);
+
+        $payload = $response->toArray();
+        // Slug MAJUSCULE du nom, separateurs non alphanumeriques -> `_`.
+        self::assertSame(
+            strtoupper(self::TEST_CATEGORY_PREFIX).'AGRO_ALIMENTAIRE',
+            $payload['code'],
+        );
+    }
+
+    public function testCodeIsReadOnlyAndIgnoredFromPayload(): void
+    {
+        $type   = $this->createCategoryType();
+        $client = $this->createAdminClient();
+
+        $response = $client->request('POST', '/api/categories', [
+            'headers' => ['Content-Type' => 'application/ld+json'],
+            'json'    => [
+                'name'         => self::TEST_CATEGORY_PREFIX.'readonly',
+                'categoryType' => '/api/category_types/'.$type->getId(),
+                // Le client tente d'imposer un code : doit etre ignore.
+                'code'         => 'CLIENT_FORGED',
+            ],
+        ]);
+        self::assertResponseStatusCodeSame(201);
+
+        $payload = $response->toArray();
+        self::assertNotSame('CLIENT_FORGED', $payload['code']);
+        self::assertSame(strtoupper(self::TEST_CATEGORY_PREFIX).'READONLY', $payload['code']);
+    }
+
+    public function testCollidingSlugsGetDistinctCodes(): void
+    {
+        $type   = $this->createCategoryType();
+        $client = $this->createAdminClient();
+
+        // Deux noms differents (donc autorises par uq_category_name_type_active)
+        // mais qui produisent le meme slug -> codes distincts (suffixe `_2`).
+        $first = $client->request('POST', '/api/categories', [
+            'headers' => ['Content-Type' => 'application/ld+json'],
+            'json'    => [
+                'name'         => self::TEST_CATEGORY_PREFIX.'Agro Plus',
+                'categoryType' => '/api/category_types/'.$type->getId(),
+            ],
+        ])->toArray();
+
+        $second = $client->request('POST', '/api/categories', [
+            'headers' => ['Content-Type' => 'application/ld+json'],
+            'json'    => [
+                'name'         => self::TEST_CATEGORY_PREFIX.'Agro-Plus',
+                'categoryType' => '/api/category_types/'.$type->getId(),
+            ],
+        ])->toArray();
+
+        self::assertResponseStatusCodeSame(201);
+        self::assertNotSame($first['code'], $second['code']);
+        self::assertStringEndsWith('_2', (string) $second['code']);
+    }
+}
@@ -17,13 +17,17 @@ use DateTimeImmutable;
 * Base des tests fonctionnels du module Commercial (M1 — repertoire clients).
 *
 * Etend la base Core : ajoute des factories pour seeder vite des categories
- * typees (DISTRIBUTEUR / COURTIER / SECTEUR) et des clients, plus un helper
- * d'authentification admin.
+ * codees (DISTRIBUTEUR / COURTIER / SECTEUR...) sous le type unique CLIENT et
+ * des clients, plus un helper d'authentification admin.
+ *
+ * Refonte taxonomie ERP-78 : il n'y a plus qu'un type CLIENT ; le code metier
+ * vit desormais sur la Category. `createCategory($code)` est un fetch-or-create
+ * PAR CODE (idempotent) sous CLIENT — deux clients d'un meme test partagent ainsi
+ * la categorie de meme code sans violer l'index unique partiel uq_category_code.
 *
 * Cleanup : tearDown purge clients, categories `test_cli_cat_*` et users/roles
- * `test_*`. Les category_types business sont fetch-or-create (idempotents) et
- * laisses en place (pas de DELETE pour ne pas entrer en conflit avec d'autres
- * suites). Pas de DAMA en local -> purge manuelle obligatoire.
+ * `test_*`. Le type CLIENT est fetch-or-create (idempotent) et laisse en place.
+ * Pas de DAMA en local -> purge manuelle obligatoire.
 *
 * @internal
 */
@@ -31,6 +35,14 @@ 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).
+     */
+    private const array RG_EXACT_CODES = ['DISTRIBUTEUR', 'COURTIER'];
+
    protected function tearDown(): void
    {
        $this->cleanupCommercialTestData();
@@ -43,20 +55,20 @@ abstract class AbstractCommercialApiTestCase extends AbstractApiTestCase
    }

    /**
-     * Recupere (ou cree) un CategoryType par son code metier. Idempotent : la
+     * Recupere (ou cree) le type unique CLIENT (refonte ERP-78). Idempotent : la
     * contrainte d'unicite sur category_type.code interdit les doublons.
     */
-    protected function createCategoryType(string $code): CategoryType
+    protected function clientCategoryType(): CategoryType
    {
        $em       = $this->getEm();
-        $existing = $em->getRepository(CategoryType::class)->findOneBy(['code' => $code]);
+        $existing = $em->getRepository(CategoryType::class)->findOneBy(['code' => 'CLIENT']);
        if (null !== $existing) {
            return $existing;
        }

        $type = new CategoryType();
-        $type->setCode($code);
-        $type->setLabel(ucfirst(strtolower($code)));
+        $type->setCode('CLIENT');
+        $type->setLabel('Client');
        $em->persist($type);
        $em->flush();

@@ -64,15 +76,38 @@ abstract class AbstractCommercialApiTestCase extends AbstractApiTestCase
    }

    /**
-     * Cree une Category de test rattachee a un type metier donne (code).
+     * Cree une Category de test sous le type unique CLIENT (ERP-78).
+     *
+     * - Code RG (DISTRIBUTEUR / COURTIER) : fetch-or-create par code EXACT — le
+     *   code doit matcher la regle de gestion, et l'appel repete dans un test
+     *   renvoie la meme categorie (pas de violation de uq_category_code).
+     * - Autre code (SECTEUR, AUTRE, ...) : simple libelle generique -> categorie
+     *   a code UNIQUE (suffixe aleatoire). Garantit que deux categories
+     *   « generiques » d'un meme test sont DISTINCTES (ex: detection de
+     *   changement de categorie dans les tests RBAC).
     */
-    protected function createCategory(string $typeCode = 'SECTEUR'): Category
+    protected function createCategory(string $code = 'SECTEUR'): Category
    {
-        $em       = $this->getEm();
-        $suffix   = substr(bin2hex(random_bytes(4)), 0, 8);
+        $em = $this->getEm();
+
+        if (in_array($code, self::RG_EXACT_CODES, true)) {
+            $existing = $em->getRepository(Category::class)->findOneBy(['code' => $code, 'deletedAt' => null]);
+            if (null !== $existing) {
+                return $existing;
+            }
+
+            $effectiveCode = $code;
+            $name          = self::TEST_CATEGORY_PREFIX.strtolower($code);
+        } else {
+            $suffix        = substr(bin2hex(random_bytes(4)), 0, 8);
+            $effectiveCode = strtoupper($code).'_'.strtoupper($suffix);
+            $name          = self::TEST_CATEGORY_PREFIX.strtolower($code).'_'.$suffix;
+        }
+
        $category = new Category();
-        $category->setName(self::TEST_CATEGORY_PREFIX.$suffix);
-        $category->setCategoryType($this->createCategoryType($typeCode));
+        $category->setName($name);
+        $category->setCode($effectiveCode);
+        $category->setCategoryType($this->clientCategoryType());
        $em->persist($category);
        $em->flush();

@@ -81,9 +116,10 @@ abstract class AbstractCommercialApiTestCase extends AbstractApiTestCase

    /**
     * Seede directement un Client en base (sans passer par l'API), pour les
-     * tests de liste / archivage. Le client porte une categorie SECTEUR.
+     * tests de liste / archivage. Le client porte une categorie du code donne
+     * (defaut SECTEUR — categorie generique non interdite sur adresse).
     */
-    protected function seedClient(string $companyName, bool $isArchived = false, string $categoryTypeCode = 'SECTEUR'): ClientEntity
+    protected function seedClient(string $companyName, bool $isArchived = false, string $categoryCode = 'SECTEUR'): ClientEntity
    {
        $em     = $this->getEm();
        $client = new ClientEntity();
@@ -93,7 +129,7 @@ abstract class AbstractCommercialApiTestCase extends AbstractApiTestCase
        $client->setLastName('Seed');
        $client->setPhonePrimary('0102030405');
        $client->setEmail(strtolower(str_replace(' ', '', $companyName)).'@seed.test');
-        $client->addCategory($this->createCategory($categoryTypeCode));
+        $client->addCategory($this->createCategory($categoryCode));
        $client->setIsArchived($isArchived);
        if ($isArchived) {
            $client->setArchivedAt(new DateTimeImmutable());
@@ -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 : seules les categories de type SECTEUR / AUTRE sont autorisees sur
- *    une adresse (DISTRIBUTEUR / COURTIER -> 422).
+ *  - RG-1.29 (ERP-78) : les categories de code DISTRIBUTEUR / COURTIER sont
+ *    interdites sur une adresse (-> 422) ; toute autre categorie est acceptee.
 *
 * Depuis ERP-76, ces regles sont portees par des Assert\Callback sur l'entite
 * ClientAddress (mirror applicatif des CHECK Postgres) : la combinaison invalide
@@ -175,6 +175,49 @@ final class ClientApiTest extends AbstractCommercialApiTestCase
        self::assertResponseStatusCodeSame(201);
    }

+    public function testPostBrokerReferencingNonBrokerReturns422(): void
+    {
+        $client   = $this->createAdminClient();
+        $cat      = $this->createCategory('SECTEUR');
+        $notBroker = $this->seedClient('Pas Un Courtier', false, 'SECTEUR');
+
+        $client->request('POST', '/api/clients', [
+            'headers' => ['Content-Type' => self::LD],
+            'json'    => [
+                'companyName'  => 'Bad Broker Ref',
+                'firstName'    => 'A',
+                'phonePrimary' => '0102030405',
+                'email'        => 'badbroker@test.fr',
+                'categories'   => ['/api/categories/'.$cat->getId()],
+                'broker'       => '/api/clients/'.$notBroker->getId(),
+            ],
+        ]);
+
+        // RG-1.03 (le broker doit porter la categorie de code COURTIER)
+        self::assertResponseStatusCodeSame(422);
+    }
+
+    public function testPostValidBrokerReturns201(): void
+    {
+        $client = $this->createAdminClient();
+        $cat    = $this->createCategory('SECTEUR');
+        $broker = $this->seedClient('Vrai Courtier', false, 'COURTIER');
+
+        $client->request('POST', '/api/clients', [
+            'headers' => ['Content-Type' => self::LD],
+            'json'    => [
+                'companyName'  => 'Client Avec Courtier',
+                'firstName'    => 'A',
+                'phonePrimary' => '0102030405',
+                'email'        => 'okbroker@test.fr',
+                'categories'   => ['/api/categories/'.$cat->getId()],
+                'broker'       => '/api/clients/'.$broker->getId(),
+            ],
+        ]);
+
+        self::assertResponseStatusCodeSame(201);
+    }
+
    public function testListSortedByCompanyNameAscAndExcludesArchived(): void
    {
        $client = $this->createAdminClient();
@@ -74,14 +74,14 @@ final class ClientExportControllerTest extends AbstractCommercialApiTestCase
        self::assertNotContains('OTHER BETA', $names);
    }

-    public function testExportRespectsCategoryTypeFilter(): void
+    public function testExportRespectsCategoryCodeFilter(): void
    {
        $client = $this->createAdminClient();
        $this->seedClient('Distrib Co', false, 'DISTRIBUTEUR');
        $this->seedClient('Secteur Co', false, 'SECTEUR');

        $names = $this->companyNames(
-            $client->request('GET', self::EXPORT_URL.'?categoryType=DISTRIBUTEUR')->getContent(),
+            $client->request('GET', self::EXPORT_URL.'?categoryCode=DISTRIBUTEUR')->getContent(),
        );

        self::assertContains('DISTRIB CO', $names);