Fix

sonata-project · Oct 3, 2023 · 4cd7d1a · 4cd7d1a
1 parent 1ca2225
commit 4cd7d1a
Show file tree

Hide file tree

Showing 14 changed files with 189 additions and 213 deletions.
diff --git a/composer.json b/composer.json
@@ -55,7 +55,7 @@
         "phpunit/phpunit": "^9.6",
         "psalm/plugin-phpunit": "^0.18",
         "psalm/plugin-symfony": "^5.0",
-        "rector/rector": "^0.17",
+        "rector/rector": "^0.18",
         "sonata-project/block-bundle": "^4.2",
         "symfony/browser-kit": "^5.4 || ^6.2",
         "symfony/css-selector": "^5.4 || ^6.2",

diff --git a/tests/Builder/DatagridBuilderTest.php b/tests/Builder/DatagridBuilderTest.php
@@ -80,7 +80,7 @@ protected function setUp(): void
     /**
      * @phpstan-param class-string $pager
      *
-     * @dataProvider getBaseDatagridData
+     * @dataProvider provideGetBaseDatagridCases
      */
     public function testGetBaseDatagrid(string $pagerType, string $pager): void
     {
@@ -102,17 +102,15 @@ public function testGetBaseDatagrid(string $pagerType, string $pager): void
     /**
      * @phpstan-return iterable<array-key, array{string, class-string}>
      */
-    public function getBaseDatagridData(): iterable
+    public function provideGetBaseDatagridCases(): iterable
     {
-        return [
-            'simple' => [
-                Pager::TYPE_SIMPLE,
-                SimplePager::class,
-            ],
-            'default' => [
-                Pager::TYPE_DEFAULT,
-                Pager::class,
-            ],
+        yield 'simple' => [
+            Pager::TYPE_SIMPLE,
+            SimplePager::class,
+        ];
+        yield 'default' => [
+            Pager::TYPE_DEFAULT,
+            Pager::class,
         ];
     }
 

diff --git a/tests/DependencyInjection/Compiler/AddGuesserCompilerPassTest.php b/tests/DependencyInjection/Compiler/AddGuesserCompilerPassTest.php
@@ -21,7 +21,7 @@
 
 final class AddGuesserCompilerPassTest extends AbstractCompilerPassTestCase
 {
-    /** @dataProvider getBuilders */
+    /** @dataProvider provideAddsGuessersCases */
     public function testAddsGuessers(string $builderServiceId, string $guesserTag): void
     {
         $builderService = new Definition(null, [[]]);
@@ -43,23 +43,21 @@ public function testAddsGuessers(string $builderServiceId, string $guesserTag):
     }
 
     /**
-     * @phpstan-return array<array{string, string}>
+     * @phpstan-return iterable<array{string, string}>
      */
-    public function getBuilders(): array
+    public function provideAddsGuessersCases(): iterable
     {
-        return [
-            'list_builder' => [
-                'sonata.admin.guesser.doctrine_mongodb_list_chain',
-                'sonata.admin.guesser.doctrine_mongodb_list',
-            ],
-            'datagrid_builder' => [
-                'sonata.admin.guesser.doctrine_mongodb_datagrid_chain',
-                'sonata.admin.guesser.doctrine_mongodb_datagrid',
-            ],
-            'show_builder' => [
-                'sonata.admin.guesser.doctrine_mongodb_show_chain',
-                'sonata.admin.guesser.doctrine_mongodb_show',
-            ],
+        yield 'list_builder' => [
+            'sonata.admin.guesser.doctrine_mongodb_list_chain',
+            'sonata.admin.guesser.doctrine_mongodb_list',
+        ];
+        yield 'datagrid_builder' => [
+            'sonata.admin.guesser.doctrine_mongodb_datagrid_chain',
+            'sonata.admin.guesser.doctrine_mongodb_datagrid',
+        ];
+        yield 'show_builder' => [
+            'sonata.admin.guesser.doctrine_mongodb_show_chain',
+            'sonata.admin.guesser.doctrine_mongodb_show',
         ];
     }
 

diff --git a/tests/FieldDescription/FilterTypeGuesserTest.php b/tests/FieldDescription/FilterTypeGuesserTest.php
@@ -97,7 +97,7 @@ public function testGuessTypeWithAssociation(): void
     }
 
     /**
-     * @dataProvider noAssociationData
+     * @dataProvider provideGuessTypeNoAssociationCases
      */
     public function testGuessTypeNoAssociation(string $type, string $resultType, int $confidence, ?string $fieldType = null): void
     {
@@ -131,54 +131,52 @@ public function testGuessTypeNoAssociation(string $type, string $resultType, int
      *
      * @phpstan-return iterable<array{0: string, 1: string, 2: int, 3?: string}>
      */
-    public function noAssociationData(): iterable
+    public function provideGuessTypeNoAssociationCases(): iterable
     {
-        return [
-            // TODO: Remove it when dropping support of doctrine/mongodb-odm < 3.0
-            Type::BOOLEAN => [
-                'boolean',
-                BooleanFilter::class,
-                Guess::HIGH_CONFIDENCE,
-                BooleanType::class,
-            ],
-            Type::TIMESTAMP => [
-                'timestamp',
-                DateTimeFilter::class,
-                Guess::HIGH_CONFIDENCE,
-            ],
-            Type::DATE => [
-                'date',
-                DateFilter::class,
-                Guess::HIGH_CONFIDENCE,
-            ],
-            Type::FLOAT => [
-                'float',
-                NumberFilter::class,
-                Guess::MEDIUM_CONFIDENCE,
-                NumberType::class,
-            ],
-            Type::INT => [
-                'int',
-                NumberFilter::class,
-                Guess::MEDIUM_CONFIDENCE,
-                NumberType::class,
-            ],
-            Type::STRING => [
-                'string',
-                StringFilter::class,
-                Guess::MEDIUM_CONFIDENCE,
-                TextType::class,
-            ],
-            Type::ID => [
-                Type::ID,
-                IdFilter::class,
-                Guess::MEDIUM_CONFIDENCE,
-            ],
-            'somefake' => [
-                'somefake',
-                StringFilter::class,
-                Guess::LOW_CONFIDENCE,
-            ],
+        // TODO: Remove it when dropping support of doctrine/mongodb-odm < 3.0
+        yield Type::BOOLEAN => [
+            'boolean',
+            BooleanFilter::class,
+            Guess::HIGH_CONFIDENCE,
+            BooleanType::class,
+        ];
+        yield Type::TIMESTAMP => [
+            'timestamp',
+            DateTimeFilter::class,
+            Guess::HIGH_CONFIDENCE,
+        ];
+        yield Type::DATE => [
+            'date',
+            DateFilter::class,
+            Guess::HIGH_CONFIDENCE,
+        ];
+        yield Type::FLOAT => [
+            'float',
+            NumberFilter::class,
+            Guess::MEDIUM_CONFIDENCE,
+            NumberType::class,
+        ];
+        yield Type::INT => [
+            'int',
+            NumberFilter::class,
+            Guess::MEDIUM_CONFIDENCE,
+            NumberType::class,
+        ];
+        yield Type::STRING => [
+            'string',
+            StringFilter::class,
+            Guess::MEDIUM_CONFIDENCE,
+            TextType::class,
+        ];
+        yield Type::ID => [
+            Type::ID,
+            IdFilter::class,
+            Guess::MEDIUM_CONFIDENCE,
+        ];
+        yield 'somefake' => [
+            'somefake',
+            StringFilter::class,
+            Guess::LOW_CONFIDENCE,
         ];
     }
 }
diff --git a/tests/FieldDescription/TypeGuesserTest.php b/tests/FieldDescription/TypeGuesserTest.php
@@ -35,7 +35,7 @@ protected function setUp(): void
     }
 
     /**
-     * @dataProvider associationData
+     * @dataProvider provideGuessTypeWithAssociationCases
      */
     public function testGuessTypeWithAssociation(string $mappingType, string $type): void
     {
@@ -59,24 +59,22 @@ public function testGuessTypeWithAssociation(string $mappingType, string $type):
     }
 
     /**
-     * @phpstan-return array<array{string, string}>
+     * @phpstan-return iterable<array{string, string}>
      */
-    public function associationData(): array
+    public function provideGuessTypeWithAssociationCases(): iterable
     {
-        return [
-            'many-to-one' => [
-                ClassMetadata::ONE,
-                FieldDescriptionInterface::TYPE_MANY_TO_ONE,
-            ],
-            'one-to-many' => [
-                ClassMetadata::MANY,
-                FieldDescriptionInterface::TYPE_MANY_TO_MANY,
-            ],
+        yield 'many-to-one' => [
+            ClassMetadata::ONE,
+            FieldDescriptionInterface::TYPE_MANY_TO_ONE,
+        ];
+        yield 'one-to-many' => [
+            ClassMetadata::MANY,
+            FieldDescriptionInterface::TYPE_MANY_TO_MANY,
         ];
     }
 
     /**
-     * @dataProvider noAssociationData
+     * @dataProvider provideGuessTypeNoAssociationCases
      */
     public function testGuessTypeNoAssociation(string $type, string $resultType, int $confidence): void
     {
@@ -100,61 +98,59 @@ public function testGuessTypeNoAssociation(string $type, string $resultType, int
     }
 
     /**
-     * @phpstan-return array<array{string, string, int}>
+     * @phpstan-return iterable<array{string, string, int}>
      */
-    public function noAssociationData(): array
+    public function provideGuessTypeNoAssociationCases(): iterable
     {
-        return [
-            'collection' => [
-                Type::COLLECTION,
-                FieldDescriptionInterface::TYPE_ARRAY,
-                Guess::HIGH_CONFIDENCE,
-            ],
-            'hash' => [
-                Type::HASH,
-                FieldDescriptionInterface::TYPE_ARRAY,
-                Guess::HIGH_CONFIDENCE,
-            ],
-            'bool' => [
-                Type::BOOL,
-                FieldDescriptionInterface::TYPE_BOOLEAN,
-                Guess::HIGH_CONFIDENCE,
-            ],
-            'timestamp' => [
-                Type::TIMESTAMP,
-                FieldDescriptionInterface::TYPE_DATETIME,
-                Guess::HIGH_CONFIDENCE,
-            ],
-            'date' => [
-                Type::DATE,
-                FieldDescriptionInterface::TYPE_DATE,
-                Guess::HIGH_CONFIDENCE,
-            ],
-            'date_immutable' => [
-                Type::DATE_IMMUTABLE,
-                FieldDescriptionInterface::TYPE_DATE,
-                Guess::HIGH_CONFIDENCE,
-            ],
-            'float' => [
-                Type::FLOAT,
-                FieldDescriptionInterface::TYPE_FLOAT,
-                Guess::MEDIUM_CONFIDENCE,
-            ],
-            'integer' => [
-                Type::INT,
-                FieldDescriptionInterface::TYPE_INTEGER,
-                Guess::MEDIUM_CONFIDENCE,
-            ],
-            'string' => [
-                Type::STRING,
-                FieldDescriptionInterface::TYPE_STRING,
-                Guess::MEDIUM_CONFIDENCE,
-            ],
-            'somefake' => [
-                'somefake',
-                FieldDescriptionInterface::TYPE_STRING,
-                Guess::LOW_CONFIDENCE,
-            ],
+        yield 'collection' => [
+            Type::COLLECTION,
+            FieldDescriptionInterface::TYPE_ARRAY,
+            Guess::HIGH_CONFIDENCE,
+        ];
+        yield 'hash' => [
+            Type::HASH,
+            FieldDescriptionInterface::TYPE_ARRAY,
+            Guess::HIGH_CONFIDENCE,
+        ];
+        yield 'bool' => [
+            Type::BOOL,
+            FieldDescriptionInterface::TYPE_BOOLEAN,
+            Guess::HIGH_CONFIDENCE,
+        ];
+        yield 'timestamp' => [
+            Type::TIMESTAMP,
+            FieldDescriptionInterface::TYPE_DATETIME,
+            Guess::HIGH_CONFIDENCE,
+        ];
+        yield 'date' => [
+            Type::DATE,
+            FieldDescriptionInterface::TYPE_DATE,
+            Guess::HIGH_CONFIDENCE,
+        ];
+        yield 'date_immutable' => [
+            Type::DATE_IMMUTABLE,
+            FieldDescriptionInterface::TYPE_DATE,
+            Guess::HIGH_CONFIDENCE,
+        ];
+        yield 'float' => [
+            Type::FLOAT,
+            FieldDescriptionInterface::TYPE_FLOAT,
+            Guess::MEDIUM_CONFIDENCE,
+        ];
+        yield 'integer' => [
+            Type::INT,
+            FieldDescriptionInterface::TYPE_INTEGER,
+            Guess::MEDIUM_CONFIDENCE,
+        ];
+        yield 'string' => [
+            Type::STRING,
+            FieldDescriptionInterface::TYPE_STRING,
+            Guess::MEDIUM_CONFIDENCE,
+        ];
+        yield 'somefake' => [
+            'somefake',
+            FieldDescriptionInterface::TYPE_STRING,
+            Guess::LOW_CONFIDENCE,
         ];
     }
 }
diff --git a/tests/Filter/BooleanFilterTest.php b/tests/Filter/BooleanFilterTest.php
@@ -37,7 +37,7 @@ public function testFilterEmpty(): void
     }
 
     /**
-     * @dataProvider getScalarValues
+     * @dataProvider provideFilterScalarCases
      */
     public function testFilterScalar(bool $equalsReturnValue, int $value): void
     {
@@ -57,14 +57,12 @@ public function testFilterScalar(bool $equalsReturnValue, int $value): void
     }
 
     /**
-     * @phpstan-return array<array{bool, int}>
+     * @phpstan-return iterable<array{bool, int}>
      */
-    public function getScalarValues(): array
+    public function provideFilterScalarCases(): iterable
     {
-        return [
-            [false, BooleanType::TYPE_NO],
-            [true, BooleanType::TYPE_YES],
-        ];
+        yield [false, BooleanType::TYPE_NO];
+        yield [true, BooleanType::TYPE_YES];
     }
 
     public function testFilterArray(): void