Reconfigure PHP CS Fixer and run again

RandyCupic · RandyCupic · commit ac6d16bf575c · 2025-10-15T13:35:24.000+02:00
diff --git a/.php-cs-fixer.php b/.php-cs-fixer.php
@@ -37,12 +37,14 @@
             'import_functions' => true,
         ],
         'heredoc_indentation' => ['indentation' => 'same_as_start'],
-        'mb_str_functions' => true,
+        //'mb_str_functions' => true,
         'native_constant_invocation' => true,
         'nullable_type_declaration_for_default_null_value' => true,
         'static_lambda' => true,
         'ternary_to_null_coalescing' => true,
         'use_arrow_functions' => true,
+        'php_unit_data_provider_name' => false,
+        'php_unit_data_provider_method_order' => false,
     ])
     ->setFinder(
         PhpCsFixer\Finder::create()
diff --git a/tests/bundle/DependencyInjection/ConfigurationTest.php b/tests/bundle/DependencyInjection/ConfigurationTest.php
@@ -158,7 +158,7 @@ public function testInvalidNamedObjectsConfiguration(array $configuration): void
         $this->assertConfigurationIsInvalid($configuration);
     }
 
-    public static function invalidNamedObjectsProvider(): array
+    public static function invalidNamedObjectsProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/API/Search/QueryTest.php b/tests/lib/API/Search/QueryTest.php
@@ -199,7 +199,7 @@ public function testSettingNextCursor(): void
         );
     }
 
-    public static function constructorPropsProvider(): array
+    public static function constructorPropsProvider(): iterable
     {
         return [
             [
@@ -264,7 +264,7 @@ public static function constructorPropsProvider(): array
         ];
     }
 
-    public static function remoteIdsProvider(): array
+    public static function remoteIdsProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/API/Search/ResultTest.php b/tests/lib/API/Search/ResultTest.php
@@ -37,7 +37,7 @@ public function testConstructor(
         );
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [0, null, []],
diff --git a/tests/lib/API/Upload/ResourceStructTest.php b/tests/lib/API/Upload/ResourceStructTest.php
@@ -142,7 +142,7 @@ public function testCreate(
         );
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/API/Values/FolderTest.php b/tests/lib/API/Values/FolderTest.php
@@ -91,7 +91,7 @@ public function testCreateFromPath(string $name, ?Folder $parent, bool $isRoot,
         );
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/Converter/ResourceTypeTest.php b/tests/lib/Core/Provider/Cloudinary/Converter/ResourceTypeTest.php
@@ -54,7 +54,7 @@ public function testGetDocumentFormats(): void
         );
     }
 
-    public static function fromCloudinaryDataProvider(): array
+    public static function fromCloudinaryDataProvider(): iterable
     {
         return [
             ['image', null, RemoteResource::TYPE_IMAGE],
@@ -76,7 +76,7 @@ public static function fromCloudinaryDataProvider(): array
         ];
     }
 
-    public static function toCloudinaryTypeProvider(): array
+    public static function toCloudinaryTypeProvider(): iterable
     {
         return [
             [RemoteResource::TYPE_IMAGE, 'image'],
diff --git a/tests/lib/Core/Provider/Cloudinary/Converter/VisibilityTypeTest.php b/tests/lib/Core/Provider/Cloudinary/Converter/VisibilityTypeTest.php
@@ -56,31 +56,31 @@ public function testToCloudinaryAccessControl(string $visibility, array $expecte
         );
     }
 
-    public static function fromCloudinaryTypeProvider(): array
+    public static function fromCloudinaryTypeProvider(): iterable
     {
         return [
             ['upload', RemoteResource::VISIBILITY_PUBLIC],
             ['authenticated', RemoteResource::VISIBILITY_PROTECTED],
         ];
     }
 
-    public static function toCloudinaryTypeProvider(): array
+    public static function toCloudinaryTypeProvider(): iterable
     {
         return [
             [RemoteResource::VISIBILITY_PUBLIC, 'upload'],
             [RemoteResource::VISIBILITY_PROTECTED, 'authenticated'],
         ];
     }
 
-    public static function toCloudinaryAccessModeProvider(): array
+    public static function toCloudinaryAccessModeProvider(): iterable
     {
         return [
             [RemoteResource::VISIBILITY_PUBLIC, 'public'],
             [RemoteResource::VISIBILITY_PROTECTED, 'authenticated'],
         ];
     }
 
-    public static function toCloudinaryAccessControlProvider(): array
+    public static function toCloudinaryAccessControlProvider(): iterable
     {
         return [
             [RemoteResource::VISIBILITY_PUBLIC, [['access_type' => 'anonymous']]],
diff --git a/tests/lib/Core/Provider/Cloudinary/Factory/RemoteResourceTest.php b/tests/lib/Core/Provider/Cloudinary/Factory/RemoteResourceTest.php
@@ -93,7 +93,7 @@ public function testCreateMissingUrls(): void
         $this->dynamicFolderModeRemoteResourceFactory->create(['public_id' => 'test']);
     }
 
-    public static function createDataProvider(): array
+    public static function createDataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/Factory/SearchResultTest.php b/tests/lib/Core/Provider/Cloudinary/Factory/SearchResultTest.php
@@ -47,7 +47,7 @@ public function testCreate(array $data, SearchResult $expectedResult): void
         );
     }
 
-    public static function createDataProvider(): array
+    public static function createDataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/Resolver/AuthTokenTest.php b/tests/lib/Core/Provider/Cloudinary/Resolver/AuthTokenTest.php
@@ -37,7 +37,7 @@ public function testHasEncryptionKey(): void
         self::assertTrue($this->resolver->hasEncryptionKey());
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/Resolver/SearchExpressionTest.php b/tests/lib/Core/Provider/Cloudinary/Resolver/SearchExpressionTest.php
@@ -49,7 +49,7 @@ public function testResolve(Query $query, string $expression, string $folderMode
         );
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/Resolver/UploadOptionsTest.php b/tests/lib/Core/Provider/Cloudinary/Resolver/UploadOptionsTest.php
@@ -51,7 +51,7 @@ public function testResolve(
         self::assertSame($options, $resolvedOptions);
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/TransformationHandler/CropTest.php b/tests/lib/Core/Provider/Cloudinary/TransformationHandler/CropTest.php
@@ -37,7 +37,7 @@ public function testWithException(array $config): void
         $this->crop->process($config);
     }
 
-    public static function validDataProvider(): array
+    public static function validDataProvider(): iterable
     {
         return [
             [
@@ -73,7 +73,7 @@ public static function validDataProvider(): array
         ];
     }
 
-    public static function invalidDataProvider(): array
+    public static function invalidDataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/TransformationHandler/FillTest.php b/tests/lib/Core/Provider/Cloudinary/TransformationHandler/FillTest.php
@@ -36,7 +36,7 @@ public function test(array $config, array $result): void
         );
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/TransformationHandler/FitTest.php b/tests/lib/Core/Provider/Cloudinary/TransformationHandler/FitTest.php
@@ -36,7 +36,7 @@ public function test(array $config, array $result): void
         );
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/TransformationHandler/FlagsTest.php b/tests/lib/Core/Provider/Cloudinary/TransformationHandler/FlagsTest.php
@@ -28,7 +28,7 @@ public function test(array $input, array $output): void
         );
     }
 
-    public function validDataProvider(): array
+    public static function validDataProvider(): iterable
     {
         return [
             [['rasterize', 'test'], ['flags' => ['rasterize']]],
diff --git a/tests/lib/Core/Provider/Cloudinary/TransformationHandler/LfillTest.php b/tests/lib/Core/Provider/Cloudinary/TransformationHandler/LfillTest.php
@@ -36,7 +36,7 @@ public function test(array $config, array $result): void
         );
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/TransformationHandler/LimitTest.php b/tests/lib/Core/Provider/Cloudinary/TransformationHandler/LimitTest.php
@@ -36,7 +36,7 @@ public function test(array $config, array $result): void
         );
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/TransformationHandler/LpadTest.php b/tests/lib/Core/Provider/Cloudinary/TransformationHandler/LpadTest.php
@@ -36,7 +36,7 @@ public function test(array $config, array $result): void
         );
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/TransformationHandler/MfitTest.php b/tests/lib/Core/Provider/Cloudinary/TransformationHandler/MfitTest.php
@@ -36,7 +36,7 @@ public function test(array $config, array $result): void
         );
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/TransformationHandler/MpadTest.php b/tests/lib/Core/Provider/Cloudinary/TransformationHandler/MpadTest.php
@@ -36,7 +36,7 @@ public function test(array $config, array $result): void
         );
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/TransformationHandler/PadTest.php b/tests/lib/Core/Provider/Cloudinary/TransformationHandler/PadTest.php
@@ -36,7 +36,7 @@ public function test(array $config, array $result): void
         );
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/TransformationHandler/ScaleTest.php b/tests/lib/Core/Provider/Cloudinary/TransformationHandler/ScaleTest.php
@@ -36,7 +36,7 @@ public function test(array $config, array $result): void
         );
     }
 
-    public static function dataProvider(): array
+    public static function dataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Core/Provider/Cloudinary/TransformationHandler/WatermarkTextTest.php b/tests/lib/Core/Provider/Cloudinary/TransformationHandler/WatermarkTextTest.php
@@ -38,7 +38,7 @@ public function testInvalid(array $config): void
         $this->watermarkText->process($config);
     }
 
-    public static function validDataProvider(): array
+    public static function validDataProvider(): iterable
     {
         return [
             [
@@ -126,7 +126,7 @@ public static function validDataProvider(): array
         ];
     }
 
-    public static function invalidDataProvider(): array
+    public static function invalidDataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Form/DataTransformer/RemoteMediaFolderTransformerTest.php b/tests/lib/Form/DataTransformer/RemoteMediaFolderTransformerTest.php
@@ -45,7 +45,7 @@ public function testReverseTransform(mixed $value, ?Folder $expectedData): void
         self::assertNull($this->dataTransformer->reverseTransform($value));
     }
 
-    public static function transformDataProvider(): array
+    public static function transformDataProvider(): iterable
     {
         return [
             [
@@ -75,7 +75,7 @@ public static function transformDataProvider(): array
         ];
     }
 
-    public static function reverseTransformDataProvider(): array
+    public static function reverseTransformDataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Form/DataTransformer/RemoteMediaTransformerTest.php b/tests/lib/Form/DataTransformer/RemoteMediaTransformerTest.php
@@ -393,7 +393,7 @@ public function testReverseTransformWithExistingLocationNewResource(): void
         );
     }
 
-    public static function transformDataProvider(): array
+    public static function transformDataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Form/Type/RemoteMediaFolderTypeTest.php b/tests/lib/Form/Type/RemoteMediaFolderTypeTest.php
@@ -88,7 +88,7 @@ public function testSubmitValidData(
         }
     }
 
-    public static function submitDataProvider(): array
+    public static function submitDataProvider(): iterable
     {
         return [
             [
diff --git a/tests/lib/Form/Type/RemoteMediaTypeTest.php b/tests/lib/Form/Type/RemoteMediaTypeTest.php
@@ -115,7 +115,7 @@ public function testSubmitValidData(
         }
     }
 
-    public static function submitDataProvider(): array
+    public static function submitDataProvider(): iterable
     {
         return [
             [

Original file line number	Diff line number	Diff line change
`@@ -158,7 +158,7 @@ public function testInvalidNamedObjectsConfiguration(array $configuration): void`
`158`	`158`	`$this->assertConfigurationIsInvalid($configuration);`
`159`	`159`	`}`
`160`	`160`
`161`		`- public static function invalidNamedObjectsProvider(): array`
	`161`	`+ public static function invalidNamedObjectsProvider(): iterable`
`162`	`162`	`{`
`163`	`163`	`return [`
`164`	`164`	`[`
Original file line number	Diff line number	Diff line change
`@@ -199,7 +199,7 @@ public function testSettingNextCursor(): void`
`199`	`199`	`);`
`200`	`200`	`}`
`201`	`201`
`202`		`- public static function constructorPropsProvider(): array`
	`202`	`+ public static function constructorPropsProvider(): iterable`
`203`	`203`	`{`
`204`	`204`	`return [`
`205`	`205`	`[`
`@@ -264,7 +264,7 @@ public static function constructorPropsProvider(): array`
`264`	`264`	`];`
`265`	`265`	`}`
`266`	`266`
`267`		`- public static function remoteIdsProvider(): array`
	`267`	`+ public static function remoteIdsProvider(): iterable`
`268`	`268`	`{`
`269`	`269`	`return [`
`270`	`270`	`[`
Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ public function testConstructor(`
`37`	`37`	`);`
`38`	`38`	`}`
`39`	`39`
`40`		`- public static function dataProvider(): array`
	`40`	`+ public static function dataProvider(): iterable`
`41`	`41`	`{`
`42`	`42`	`return [`
`43`	`43`	`[0, null, []],`
Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@ public function testCreate(`
`142`	`142`	`);`
`143`	`143`	`}`
`144`	`144`
`145`		`- public static function dataProvider(): array`
	`145`	`+ public static function dataProvider(): iterable`
`146`	`146`	`{`
`147`	`147`	`return [`
`148`	`148`	`[`
Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,7 @@ public function testCreateFromPath(string $name, ?Folder $parent, bool $isRoot,`
`91`	`91`	`);`
`92`	`92`	`}`
`93`	`93`
`94`		`- public static function dataProvider(): array`
	`94`	`+ public static function dataProvider(): iterable`
`95`	`95`	`{`
`96`	`96`	`return [`
`97`	`97`	`[`
Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ public function testGetDocumentFormats(): void`
`54`	`54`	`);`
`55`	`55`	`}`
`56`	`56`
`57`		`- public static function fromCloudinaryDataProvider(): array`
	`57`	`+ public static function fromCloudinaryDataProvider(): iterable`
`58`	`58`	`{`
`59`	`59`	`return [`
`60`	`60`	`['image', null, RemoteResource::TYPE_IMAGE],`
`@@ -76,7 +76,7 @@ public static function fromCloudinaryDataProvider(): array`
`76`	`76`	`];`
`77`	`77`	`}`
`78`	`78`
`79`		`- public static function toCloudinaryTypeProvider(): array`
	`79`	`+ public static function toCloudinaryTypeProvider(): iterable`
`80`	`80`	`{`
`81`	`81`	`return [`
`82`	`82`	`[RemoteResource::TYPE_IMAGE, 'image'],`
Original file line number	Diff line number	Diff line change
`@@ -56,31 +56,31 @@ public function testToCloudinaryAccessControl(string $visibility, array $expecte`
`56`	`56`	`);`
`57`	`57`	`}`
`58`	`58`
`59`		`- public static function fromCloudinaryTypeProvider(): array`
	`59`	`+ public static function fromCloudinaryTypeProvider(): iterable`
`60`	`60`	`{`
`61`	`61`	`return [`
`62`	`62`	`['upload', RemoteResource::VISIBILITY_PUBLIC],`
`63`	`63`	`['authenticated', RemoteResource::VISIBILITY_PROTECTED],`
`64`	`64`	`];`
`65`	`65`	`}`
`66`	`66`
`67`		`- public static function toCloudinaryTypeProvider(): array`
	`67`	`+ public static function toCloudinaryTypeProvider(): iterable`
`68`	`68`	`{`
`69`	`69`	`return [`
`70`	`70`	`[RemoteResource::VISIBILITY_PUBLIC, 'upload'],`
`71`	`71`	`[RemoteResource::VISIBILITY_PROTECTED, 'authenticated'],`
`72`	`72`	`];`
`73`	`73`	`}`
`74`	`74`
`75`		`- public static function toCloudinaryAccessModeProvider(): array`
	`75`	`+ public static function toCloudinaryAccessModeProvider(): iterable`
`76`	`76`	`{`
`77`	`77`	`return [`
`78`	`78`	`[RemoteResource::VISIBILITY_PUBLIC, 'public'],`
`79`	`79`	`[RemoteResource::VISIBILITY_PROTECTED, 'authenticated'],`
`80`	`80`	`];`
`81`	`81`	`}`
`82`	`82`
`83`		`- public static function toCloudinaryAccessControlProvider(): array`
	`83`	`+ public static function toCloudinaryAccessControlProvider(): iterable`
`84`	`84`	`{`
`85`	`85`	`return [`
`86`	`86`	`[RemoteResource::VISIBILITY_PUBLIC, [['access_type' => 'anonymous']]],`
Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@ public function testCreateMissingUrls(): void`
`93`	`93`	`$this->dynamicFolderModeRemoteResourceFactory->create(['public_id' => 'test']);`
`94`	`94`	`}`
`95`	`95`
`96`		`- public static function createDataProvider(): array`
	`96`	`+ public static function createDataProvider(): iterable`
`97`	`97`	`{`
`98`	`98`	`return [`
`99`	`99`	`[`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ public function testCreate(array $data, SearchResult $expectedResult): void`
`47`	`47`	`);`
`48`	`48`	`}`
`49`	`49`
`50`		`- public static function createDataProvider(): array`
	`50`	`+ public static function createDataProvider(): iterable`
`51`	`51`	`{`
`52`	`52`	`return [`
`53`	`53`	`[`