fix(back,front) : adresse client — au moins une categorie obligatoire
Spec-front § Adresse : la categorie est obligatoire sur une adresse, mais n'etait enforced ni au back ni au front. - Back : ClientAddress::$categories porte desormais Assert\Count(min: 1) (POST/PATCH sans categorie -> 422). Test testAddressRequiresAtLeastOneCategory ; deux tests existants qui creaient une adresse sans categorie recoivent une categorie SECTEUR. - Front : canValidateAddresses (creation + modification) exige >= 1 categorie par adresse -> bouton Enregistrer desactive tant qu'aucune categorie n'est choisie (meme gating que les sites).
This commit is contained in:
@@ -167,8 +167,9 @@ final class ClientAddressTest extends AbstractCommercialApiTestCase
|
||||
public function testNonBillingAddressAcceptsEmptyBillingEmail(): void
|
||||
{
|
||||
$this->skipIfSitesModuleDisabled();
|
||||
$client = $this->createAdminClient();
|
||||
$seed = $this->seedClient('Non Billing Empty Email');
|
||||
$client = $this->createAdminClient();
|
||||
$seed = $this->seedClient('Non Billing Empty Email');
|
||||
$category = $this->createCategory('SECTEUR');
|
||||
|
||||
$client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
|
||||
'headers' => ['Content-Type' => self::LD],
|
||||
@@ -179,6 +180,7 @@ final class ClientAddressTest extends AbstractCommercialApiTestCase
|
||||
'city' => 'Châtellerault',
|
||||
'street' => '1 rue du Test',
|
||||
'sites' => [$this->firstSiteIri()],
|
||||
'categories' => ['/api/categories/'.$category->getId()],
|
||||
],
|
||||
]);
|
||||
|
||||
@@ -286,6 +288,29 @@ final class ClientAddressTest extends AbstractCommercialApiTestCase
|
||||
self::assertResponseStatusCodeSame(201);
|
||||
}
|
||||
|
||||
/**
|
||||
* Spec-front § Adresse : au moins une categorie est obligatoire sur une
|
||||
* adresse. POST sans categorie (mais avec site) -> 422.
|
||||
*/
|
||||
public function testAddressRequiresAtLeastOneCategory(): void
|
||||
{
|
||||
$this->skipIfSitesModuleDisabled();
|
||||
$client = $this->createAdminClient();
|
||||
$seed = $this->seedClient('Address No Cat');
|
||||
|
||||
$client->request('POST', '/api/clients/'.$seed->getId().'/addresses', [
|
||||
'headers' => ['Content-Type' => self::LD],
|
||||
'json' => [
|
||||
'postalCode' => '86100',
|
||||
'city' => 'Châtellerault',
|
||||
'street' => '1 rue du Test',
|
||||
'sites' => [$this->firstSiteIri()],
|
||||
],
|
||||
]);
|
||||
|
||||
self::assertResponseStatusCodeSame(422);
|
||||
}
|
||||
|
||||
/**
|
||||
* Retourne l'IRI du premier site seede (fixtures Sites).
|
||||
*/
|
||||
|
||||
Reference in New Issue
Block a user