2018-04-06 18:44:20 +01:00
|
|
|
<?php
|
|
|
|
|
|
|
|
/*
|
|
|
|
* This file is part of the NelmioApiDocBundle package.
|
|
|
|
*
|
|
|
|
* (c) Nelmio
|
|
|
|
*
|
|
|
|
* For the full copyright and license information, please view the LICENSE
|
|
|
|
* file that was distributed with this source code.
|
|
|
|
*/
|
|
|
|
|
|
|
|
namespace Nelmio\ApiDocBundle\Tests\ModelDescriber\Annotations;
|
|
|
|
|
|
|
|
use Doctrine\Common\Annotations\AnnotationReader;
|
|
|
|
use Nelmio\ApiDocBundle\ModelDescriber\Annotations\SymfonyConstraintAnnotationReader;
|
2020-05-28 13:19:11 +02:00
|
|
|
use OpenApi\Annotations as OA;
|
2018-04-06 18:44:20 +01:00
|
|
|
use PHPUnit\Framework\TestCase;
|
|
|
|
use Symfony\Component\Validator\Constraints as Assert;
|
|
|
|
|
|
|
|
class SymfonyConstraintAnnotationReaderTest extends TestCase
|
|
|
|
{
|
|
|
|
public function testUpdatePropertyFix1283()
|
|
|
|
{
|
|
|
|
$entity = new class() {
|
|
|
|
/**
|
|
|
|
* @Assert\NotBlank()
|
|
|
|
* @Assert\Length(min = 1)
|
|
|
|
*/
|
|
|
|
private $property1;
|
|
|
|
/**
|
|
|
|
* @Assert\NotBlank()
|
|
|
|
*/
|
|
|
|
private $property2;
|
|
|
|
};
|
2018-09-11 13:57:22 +03:00
|
|
|
|
2020-05-28 13:19:11 +02:00
|
|
|
$schema = new OA\Schema([]);
|
|
|
|
$schema->merge([new OA\Property(['property' => 'property1'])]);
|
|
|
|
$schema->merge([new OA\Property(['property' => 'property2'])]);
|
2018-04-06 18:44:20 +01:00
|
|
|
|
|
|
|
$symfonyConstraintAnnotationReader = new SymfonyConstraintAnnotationReader(new AnnotationReader());
|
|
|
|
$symfonyConstraintAnnotationReader->setSchema($schema);
|
2018-09-11 13:57:22 +03:00
|
|
|
|
2020-05-28 13:19:11 +02:00
|
|
|
$symfonyConstraintAnnotationReader->updateProperty(new \ReflectionProperty($entity, 'property1'), $schema->properties[0]);
|
|
|
|
$symfonyConstraintAnnotationReader->updateProperty(new \ReflectionProperty($entity, 'property2'), $schema->properties[1]);
|
2018-04-06 18:44:20 +01:00
|
|
|
|
|
|
|
// expect required to be numeric array with sequential keys (not [0 => ..., 2 => ...])
|
2020-05-28 13:19:11 +02:00
|
|
|
$this->assertEquals($schema->required, ['property1', 'property2']);
|
2018-04-06 18:44:20 +01:00
|
|
|
}
|
2020-03-14 15:36:59 +03:00
|
|
|
|
2020-08-31 23:22:42 +03:00
|
|
|
public function testOptionalProperty()
|
|
|
|
{
|
2020-09-28 22:45:24 +03:00
|
|
|
if (!\property_exists(Assert\NotBlank::class, 'allowNull')) {
|
2020-09-20 20:38:26 +02:00
|
|
|
$this->markTestSkipped('NotBlank::allowNull was added in symfony/validator 4.3.');
|
|
|
|
}
|
|
|
|
|
2020-08-31 23:22:42 +03:00
|
|
|
$entity = new class() {
|
|
|
|
/**
|
|
|
|
* @Assert\NotBlank(allowNull = true)
|
|
|
|
* @Assert\Length(min = 1)
|
|
|
|
*/
|
|
|
|
private $property1;
|
|
|
|
/**
|
|
|
|
* @Assert\NotBlank()
|
|
|
|
*/
|
|
|
|
private $property2;
|
|
|
|
};
|
|
|
|
|
2020-09-09 08:55:41 +02:00
|
|
|
$schema = new OA\Schema([]);
|
|
|
|
$schema->merge([new OA\Property(['property' => 'property1'])]);
|
|
|
|
$schema->merge([new OA\Property(['property' => 'property2'])]);
|
2020-08-31 23:22:42 +03:00
|
|
|
|
|
|
|
$symfonyConstraintAnnotationReader = new SymfonyConstraintAnnotationReader(new AnnotationReader());
|
|
|
|
$symfonyConstraintAnnotationReader->setSchema($schema);
|
|
|
|
|
2020-09-09 08:55:41 +02:00
|
|
|
$symfonyConstraintAnnotationReader->updateProperty(new \ReflectionProperty($entity, 'property1'), $schema->properties[0]);
|
|
|
|
$symfonyConstraintAnnotationReader->updateProperty(new \ReflectionProperty($entity, 'property2'), $schema->properties[1]);
|
2020-08-31 23:22:42 +03:00
|
|
|
|
|
|
|
// expect required to be numeric array with sequential keys (not [0 => ..., 2 => ...])
|
2020-09-09 08:55:41 +02:00
|
|
|
$this->assertEquals($schema->required, ['property2']);
|
2018-04-06 18:44:20 +01:00
|
|
|
}
|
2020-03-14 15:36:59 +03:00
|
|
|
|
|
|
|
public function testAssertChoiceResultsInNumericArray()
|
|
|
|
{
|
|
|
|
define('TEST_ASSERT_CHOICE_STATUSES', [
|
|
|
|
1 => 'active',
|
|
|
|
2 => 'blocked',
|
|
|
|
]);
|
|
|
|
|
|
|
|
$entity = new class() {
|
|
|
|
/**
|
|
|
|
* @Assert\Length(min = 1)
|
|
|
|
* @Assert\Choice(choices=TEST_ASSERT_CHOICE_STATUSES)
|
|
|
|
*/
|
|
|
|
private $property1;
|
|
|
|
};
|
|
|
|
|
2020-05-28 13:19:11 +02:00
|
|
|
$schema = new OA\Schema([]);
|
|
|
|
$schema->merge([new OA\Property(['property' => 'property1'])]);
|
2020-03-14 15:36:59 +03:00
|
|
|
|
|
|
|
$symfonyConstraintAnnotationReader = new SymfonyConstraintAnnotationReader(new AnnotationReader());
|
|
|
|
$symfonyConstraintAnnotationReader->setSchema($schema);
|
|
|
|
|
2020-05-28 13:19:11 +02:00
|
|
|
$symfonyConstraintAnnotationReader->updateProperty(new \ReflectionProperty($entity, 'property1'), $schema->properties[0]);
|
2020-03-14 15:36:59 +03:00
|
|
|
|
|
|
|
// expect enum to be numeric array with sequential keys (not [1 => "active", 2 => "active"])
|
2020-05-28 13:19:11 +02:00
|
|
|
$this->assertEquals($schema->properties[0]->enum, ['active', 'blocked']);
|
2020-03-14 15:36:59 +03:00
|
|
|
}
|
2021-02-10 10:33:55 -06:00
|
|
|
|
Apply `enum` from Choice Constraints to Items When Choice is Multiple (#1784)
* Apply `enum` from Choice Constraints to Items When Choice is Multiple
Otherwise JSON schema like this is generated:
```
"property": {
"type": "array",
"enum": ["one", "two", "three"],
"items": {
"type": "string"
}
}
```
With this change, however, this schema is generated:
```
"property": {
"type": "array",
"items": {
"type": "string",
"enum": ["one", "two", "three"]
}
}
```
A possible downside here is that the symfony constraint stuff happens
before types are figured out from PHPDoc. So it's _possible_ to end up
with something that won't validated. Take something like this:
```
/**
* @Assert\Choice(multiple=true, choices={"..."})
* @var string
*/
```
This would generate:
```
"property": {
"type": "string",
"items": {
"enum": ["..."]
}
}
```
* Fix CS
* cs
* more cs
* fix tests
Co-authored-by: Guilhem Niot <guilhem@gniot.fr>
2021-02-19 02:41:32 -06:00
|
|
|
public function testMultipleChoiceConstraintsApplyEnumToItems()
|
|
|
|
{
|
|
|
|
$entity = new class() {
|
|
|
|
/**
|
|
|
|
* @Assert\Choice(choices={"one", "two"}, multiple=true)
|
|
|
|
*/
|
|
|
|
private $property1;
|
|
|
|
};
|
|
|
|
|
|
|
|
$schema = new OA\Schema([]);
|
|
|
|
$schema->merge([new OA\Property(['property' => 'property1'])]);
|
|
|
|
|
|
|
|
$symfonyConstraintAnnotationReader = new SymfonyConstraintAnnotationReader(new AnnotationReader());
|
|
|
|
$symfonyConstraintAnnotationReader->setSchema($schema);
|
|
|
|
|
|
|
|
$symfonyConstraintAnnotationReader->updateProperty(new \ReflectionProperty($entity, 'property1'), $schema->properties[0]);
|
|
|
|
|
|
|
|
$this->assertInstanceOf(OA\Items::class, $schema->properties[0]->items);
|
|
|
|
$this->assertEquals($schema->properties[0]->items->enum, ['one', 'two']);
|
|
|
|
}
|
|
|
|
|
2021-02-10 10:33:55 -06:00
|
|
|
/**
|
|
|
|
* @group https://github.com/nelmio/NelmioApiDocBundle/issues/1780
|
|
|
|
*/
|
|
|
|
public function testLengthConstraintDoesNotSetMaxLengthIfMaxIsNotSet()
|
|
|
|
{
|
|
|
|
$entity = new class() {
|
|
|
|
/**
|
|
|
|
* @Assert\Length(min = 1)
|
|
|
|
*/
|
|
|
|
private $property1;
|
|
|
|
};
|
|
|
|
|
|
|
|
$schema = new OA\Schema([]);
|
|
|
|
$schema->merge([new OA\Property(['property' => 'property1'])]);
|
|
|
|
|
|
|
|
$symfonyConstraintAnnotationReader = new SymfonyConstraintAnnotationReader(new AnnotationReader());
|
|
|
|
$symfonyConstraintAnnotationReader->setSchema($schema);
|
|
|
|
|
|
|
|
$symfonyConstraintAnnotationReader->updateProperty(new \ReflectionProperty($entity, 'property1'), $schema->properties[0]);
|
|
|
|
|
|
|
|
$this->assertSame(OA\UNDEFINED, $schema->properties[0]->maxLength);
|
|
|
|
$this->assertSame(1, $schema->properties[0]->minLength);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* @group https://github.com/nelmio/NelmioApiDocBundle/issues/1780
|
|
|
|
*/
|
|
|
|
public function testLengthConstraintDoesNotSetMinLengthIfMinIsNotSet()
|
|
|
|
{
|
|
|
|
$entity = new class() {
|
|
|
|
/**
|
|
|
|
* @Assert\Length(max = 100)
|
|
|
|
*/
|
|
|
|
private $property1;
|
|
|
|
};
|
|
|
|
|
|
|
|
$schema = new OA\Schema([]);
|
|
|
|
$schema->merge([new OA\Property(['property' => 'property1'])]);
|
|
|
|
|
|
|
|
$symfonyConstraintAnnotationReader = new SymfonyConstraintAnnotationReader(new AnnotationReader());
|
|
|
|
$symfonyConstraintAnnotationReader->setSchema($schema);
|
|
|
|
|
|
|
|
$symfonyConstraintAnnotationReader->updateProperty(new \ReflectionProperty($entity, 'property1'), $schema->properties[0]);
|
|
|
|
|
|
|
|
$this->assertSame(OA\UNDEFINED, $schema->properties[0]->minLength);
|
|
|
|
$this->assertSame(100, $schema->properties[0]->maxLength);
|
|
|
|
}
|
2018-04-06 18:44:20 +01:00
|
|
|
}
|