0c9b563cae
Auto Tag Develop / tag (push) Successful in 9s
**MR stackée sur ERP-54** — cible = `feature/ERP-54-creer-entites-client-m1` (PAS `develop`). Tristan validera le stack en fin de chaîne. Branche l'API REST du répertoire clients (M1) sur l'entité `Client` d'ERP-54. ## Périmètre - **ClientProvider** : liste paginée (Paginator ORM aligné ERP-72, `?pagination=false`), exclusion archives+soft-delete par défaut (RG-1.24), `?includeArchived=true` (RG-1.25), tri `companyName ASC` (RG-1.26), filtres `?search` (fuzzy) + `?categoryType`, détail 404 si soft-deleted + embarque contacts/adresses/ribs. - **ClientProcessor** : normalisation (RG-1.18→1.21), 409 doublon nom (RG-1.16) + 409 restauration (RG-1.23), gating par onglet `accounting.manage`/`archive` + mode strict 403 (RG-1.28), archivage exclusif + `archivedAt` (RG-1.22), RG-1.01 / RG-1.03 (mutex + type catégorie) / RG-1.12 / RG-1.13 / RG-1.04. - **ClientReadGroupContextBuilder** : ajout conditionnel du groupe `client:read:accounting` selon `commercial.clients.accounting.view`. - **CategoryReferenceDenormalizer** : résout les IRI catégorie vers `Category` (dénormalisation impossible sur l'interface sinon). - **Contrats Shared** : `CategoryInterface::getCategoryTypeCode()`, `BusinessRoleAwareInterface` + `BusinessRoles::COMMERCIALE`. ## Coordination stack - Permissions `commercial.clients.*` **référencées** ici, déclarées en **ERP-59** (tests RBAC en **ERP-60**). - Rôle métier `commerciale` seedé par **ERP-74** (RG-1.04 dormante d'ici là). - Config globale pagination (itemsPerPage client / max 50) portée par **ERP-72**. - Référentiels comptables (PaymentType/Bank/...) exposés en **ERP-56** → RG-1.12/1.13 testées en unitaire ici (pas d'IRI référentiel disponible avant ERP-56). ## Tests 31 tests Commercial (intégration admin sur les RG métier + unitaires sur le gating / RG-1.04 / RG-1.12 / RG-1.13 / context builder). Suite complète verte (343 tests). Règle n°1 respectée (aucun import inter-modules dans Commercial). --------- Co-authored-by: tristan <tristan@yuno.malio.fr> Co-authored-by: Matthieu <contact@malio.fr> Co-authored-by: Matthieu <mtholot19@gmail.com> Reviewed-on: #31 Co-authored-by: THOLOT DECHENE Matthieu <matthieu@yuno.malio.fr> Co-committed-by: THOLOT DECHENE Matthieu <matthieu@yuno.malio.fr>
63 lines
2.3 KiB
PHP
63 lines
2.3 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace App\Tests\Module\Commercial\Unit;
|
|
|
|
use ApiPlatform\Metadata\IriConverterInterface;
|
|
use App\Module\Commercial\Infrastructure\ApiPlatform\Serializer\CategoryReferenceDenormalizer;
|
|
use App\Shared\Domain\Contract\CategoryInterface;
|
|
use PHPUnit\Framework\TestCase;
|
|
use stdClass;
|
|
use Symfony\Component\Serializer\Exception\UnexpectedValueException;
|
|
|
|
/**
|
|
* Tests unitaires du CategoryReferenceDenormalizer : resolution d'un IRI vers
|
|
* une Category concrete, et rejet explicite d'un IRI pointant sur une autre
|
|
* ressource (au lieu d'un null silencieux qui perdrait la reference).
|
|
*
|
|
* @internal
|
|
*/
|
|
final class CategoryReferenceDenormalizerTest extends TestCase
|
|
{
|
|
public function testResolvesCategoryIri(): void
|
|
{
|
|
$category = $this->createStub(CategoryInterface::class);
|
|
$iriConverter = $this->createMock(IriConverterInterface::class);
|
|
$iriConverter->method('getResourceFromIri')->willReturn($category);
|
|
|
|
$denormalizer = new CategoryReferenceDenormalizer($iriConverter);
|
|
|
|
self::assertSame(
|
|
$category,
|
|
$denormalizer->denormalize('/api/categories/1', CategoryInterface::class),
|
|
);
|
|
}
|
|
|
|
public function testRejectsIriOfWrongType(): void
|
|
{
|
|
// Bug review ERP-55 : un IRI syntaxiquement valide mais pointant sur une
|
|
// autre ressource (ex: /api/clients/5) doit lever une exception au lieu
|
|
// d'etre silencieusement ignore.
|
|
$iriConverter = $this->createMock(IriConverterInterface::class);
|
|
$iriConverter->method('getResourceFromIri')->willReturn(new stdClass());
|
|
|
|
$denormalizer = new CategoryReferenceDenormalizer($iriConverter);
|
|
|
|
$this->expectException(UnexpectedValueException::class);
|
|
$denormalizer->denormalize('/api/clients/5', CategoryInterface::class);
|
|
}
|
|
|
|
public function testReturnsNullForEmptyData(): void
|
|
{
|
|
// Valeur vide deleguee par l'ArrayDenormalizer : aucun appel a
|
|
// l'IriConverter, retour null.
|
|
$iriConverter = $this->createMock(IriConverterInterface::class);
|
|
$iriConverter->expects(self::never())->method('getResourceFromIri');
|
|
|
|
$denormalizer = new CategoryReferenceDenormalizer($iriConverter);
|
|
|
|
self::assertNull($denormalizer->denormalize('', CategoryInterface::class));
|
|
}
|
|
}
|