Fix

sonata-project · Oct 3, 2023 · 51bf0d1 · 51bf0d1
1 parent ab604bf
commit 51bf0d1
Show file tree

Hide file tree

Showing 6 changed files with 126 additions and 146 deletions.
diff --git a/composer.json b/composer.json
@@ -43,7 +43,7 @@
         "phpunit/phpunit": "^9.5",
         "psalm/plugin-phpunit": "^0.18",
         "psalm/plugin-symfony": "^5.0",
-        "rector/rector": "^0.17",
+        "rector/rector": "^0.18",
         "symfony/phpunit-bridge": "^6.2",
         "symfony/security-core": "^5.4 || ^6.2",
         "vimeo/psalm": "^5.0"

diff --git a/tests/Helper/NumberFormatterTest.php b/tests/Helper/NumberFormatterTest.php
@@ -95,7 +95,7 @@ public function testExceptionOnInvalidParams(): void
     }
 
     /**
-     * @dataProvider provideConstantValues
+     * @dataProvider provideParseConstantValueCases
      */
     public function testParseConstantValue(string $constantName, int $expectedConstant, bool $exceptionExpected): void
     {
@@ -112,21 +112,19 @@ public function testParseConstantValue(string $constantName, int $expectedConsta
     }
 
     /**
-     * @return array<array{string, int, bool}>
+     * @return iterable<array{string, int, bool}>
      */
-    public function provideConstantValues(): array
+    public function provideParseConstantValueCases(): iterable
     {
-        return [
-            ['positive_prefix', \NumberFormatter::POSITIVE_PREFIX, false],
-            ['non_existent_constant', \NumberFormatter::NEGATIVE_PREFIX, true],
-        ];
+        yield ['positive_prefix', \NumberFormatter::POSITIVE_PREFIX, false];
+        yield ['non_existent_constant', \NumberFormatter::NEGATIVE_PREFIX, true];
     }
 
     /**
      * @param array<string, string> $attributes
      * @param array<int, string>    $expectedAttributes
      *
-     * @dataProvider provideAttributeValues
+     * @dataProvider provideParseAttributesCases
      */
     public function testParseAttributes(array $attributes, array $expectedAttributes, bool $exceptionExpected): void
     {
@@ -143,29 +141,27 @@ public function testParseAttributes(array $attributes, array $expectedAttributes
     }
 
     /**
-     * @return array<array{array<string, string>, array<int, string>, bool}>
+     * @return iterable<array{array<string, string>, array<int, string>, bool}>
      */
-    public function provideAttributeValues(): array
+    public function provideParseAttributesCases(): iterable
     {
-        return [
+        yield [
+            [
+                'positive_prefix' => 'POSITIVE',
+                'negative_prefix' => 'NEGATIVE',
+            ],
             [
-                [
-                    'positive_prefix' => 'POSITIVE',
-                    'negative_prefix' => 'NEGATIVE',
-                ],
-                [
-                    \NumberFormatter::POSITIVE_PREFIX => 'POSITIVE',
-                    \NumberFormatter::NEGATIVE_PREFIX => 'NEGATIVE',
-                ],
-                false,
+                \NumberFormatter::POSITIVE_PREFIX => 'POSITIVE',
+                \NumberFormatter::NEGATIVE_PREFIX => 'NEGATIVE',
             ],
+            false,
+        ];
+        yield [
             [
-                [
-                    'non_existent_constant' => 'NON_EXISTENT_VALUE',
-                ],
-                [],
-                true,
+                'non_existent_constant' => 'NON_EXISTENT_VALUE',
             ],
+            [],
+            true,
         ];
     }
 

diff --git a/tests/SonataIntlBundleTest.php b/tests/SonataIntlBundleTest.php
@@ -19,21 +19,19 @@
 class SonataIntlBundleTest extends TestCase
 {
     /**
-     * @return array<array{string, string, bool, bool}>
+     * @return iterable<array{string, string, bool, bool}>
      */
-    public function getVersions(): array
+    public function provideSymfonyVersionCases(): iterable
     {
-        return [
-            ['2.0.1', '2.0.1', true, true],
-            ['2.0.2', '2.0.1', true, true],
-            ['2.1.1-DEV', '2.1.1', false, true],
-            ['2.1.0-RC1', '2.1.0', false, true],
-            ['2.1.0-RC1', '2.1.1', false, false],
-        ];
+        yield ['2.0.1', '2.0.1', true, true];
+        yield ['2.0.2', '2.0.1', true, true];
+        yield ['2.1.1-DEV', '2.1.1', false, true];
+        yield ['2.1.0-RC1', '2.1.0', false, true];
+        yield ['2.1.0-RC1', '2.1.1', false, false];
     }
 
     /**
-     * @dataProvider getVersions
+     * @dataProvider provideSymfonyVersionCases
      */
     public function testSymfonyVersion(string $currentVersion, string $minVersion, bool $versionExpected, bool $versionBundle): void
     {

diff --git a/tests/Timezone/ChainTimezoneDetectorTest.php b/tests/Timezone/ChainTimezoneDetectorTest.php
@@ -23,23 +23,21 @@
 final class ChainTimezoneDetectorTest extends TestCase
 {
     /**
-     * @return array<array{array<string|null>, string}>
+     * @return iterable<array{array<string|null>, string}>
      */
-    public static function timezoneProvider(): array
+    public static function provideDetectsTimezoneForUserCases(): iterable
     {
-        return [
-            [['Europe/Paris', 'Europe/London'], 'Europe/Paris'],
-            [['Europe/Paris', null], 'Europe/Paris'],
-            [[null, 'Europe/Paris'], 'Europe/Paris'],
-            [[null, null], 'America/Denver'],
-            [['Invalid/Timezone', null], 'America/Denver'],
-        ];
+        yield [['Europe/Paris', 'Europe/London'], 'Europe/Paris'];
+        yield [['Europe/Paris', null], 'Europe/Paris'];
+        yield [[null, 'Europe/Paris'], 'Europe/Paris'];
+        yield [[null, null], 'America/Denver'];
+        yield [['Invalid/Timezone', null], 'America/Denver'];
     }
 
     /**
      * @param array<string|null> $detectorsTimezones
      *
-     * @dataProvider timezoneProvider
+     * @dataProvider provideDetectsTimezoneForUserCases
      */
     public function testDetectsTimezoneForUser(array $detectorsTimezones, string $expectedTimezone): void
     {

diff --git a/tests/Timezone/UserBasedTimezoneDetectorTest.php b/tests/Timezone/UserBasedTimezoneDetectorTest.php
@@ -29,16 +29,14 @@ final class UserBasedTimezoneDetectorTest extends TestCase
     /**
      * @return iterable<array{string|null}>
      */
-    public static function timezoneProvider(): iterable
+    public static function provideUserTimezoneDetectionCases(): iterable
     {
-        return [
-            ['Europe/Paris'],
-            [null],
-        ];
+        yield ['Europe/Paris'];
+        yield [null];
     }
 
     /**
-     * @dataProvider timezoneProvider
+     * @dataProvider provideUserTimezoneDetectionCases
      */
     public function testUserTimezoneDetection(?string $timezone): void
     {