Fix

sonata-project · Sep 28, 2023 · 8223120 · 8223120
1 parent d628d96
commit 8223120
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 19 deletions.
diff --git a/composer.json b/composer.json
@@ -64,7 +64,7 @@
         "phpunit/phpunit": "^9.5",
         "psalm/plugin-phpunit": "^0.18",
         "psalm/plugin-symfony": "^5.0",
-        "rector/rector": "^0.17",
+        "rector/rector": "^0.18",
         "sonata-project/admin-bundle": "^4.20",
         "sonata-project/block-bundle": "^4.11 || ^5.0",
         "sonata-project/doctrine-orm-admin-bundle": "^4.0",

diff --git a/tests/Action/LoginActionTest.php b/tests/Action/LoginActionTest.php
@@ -134,7 +134,7 @@ public function testAlreadyAuthenticated(): void
     }
 
     /**
-     * @dataProvider unauthenticatedProvider
+     * @dataProvider provideUnauthenticatedCases
      */
     public function testUnauthenticated(string $lastUsername, ?AuthenticationException $errorMessage = null): void
     {
@@ -208,14 +208,11 @@ public function testUnauthenticated(string $lastUsername, ?AuthenticationExcepti
      *
      * @phpstan-return iterable<array{string, AuthenticationException|null}>
      */
-    public function unauthenticatedProvider(): iterable
+    public function provideUnauthenticatedCases(): iterable
     {
         $error = new AuthenticationException('An error');
-
-        return [
-            ['', null],
-            ['FooUser', $error],
-        ];
+        yield ['', null];
+        yield ['FooUser', $error];
     }
 
     private function getAction(): LoginAction

diff --git a/tests/Functional/Admin/UserAdminTest.php b/tests/Functional/Admin/UserAdminTest.php
@@ -54,7 +54,7 @@ public static function provideCrudUrlsCases(): iterable
     }
 
     /**
-     * @dataProvider provideFormUrlsCases
+     * @dataProvider provideFormsUrlsCases
      *
      * @param array<string, mixed> $parameters
      * @param array<string, mixed> $fieldValues
@@ -77,7 +77,7 @@ public function testFormsUrls(string $url, array $parameters, string $button, ar
      *
      * @phpstan-return iterable<array{0: string, 1: array<string, mixed>, 2: string, 3?: array<string, mixed>}>
      */
-    public static function provideFormUrlsCases(): iterable
+    public static function provideFormsUrlsCases(): iterable
     {
         yield 'Create User' => ['/admin/tests/app/user/create', [
             'uniqid' => 'user',

diff --git a/tests/Mailer/MailerTest.php b/tests/Mailer/MailerTest.php
@@ -68,7 +68,7 @@ public function testSendConfirmationEmailMessage(): void
     }
 
     /**
-     * @dataProvider emailTemplateData
+     * @dataProvider provideSendResettingEmailMessageCases
      */
     public function testSendResettingEmailMessage(string $template, string $subject, string $body): void
     {
@@ -111,15 +111,13 @@ public function testSendResettingEmailMessage(string $template, string $subject,
      *
      * @phpstan-return iterable<array{string, string, string}>
      */
-    public function emailTemplateData(): iterable
+    public function provideSendResettingEmailMessageCases(): iterable
     {
-        return [
-            'CR' => ["Subject\rFirst line\rSecond line", 'Subject', "First line\rSecond line"],
-            'LF' => ["Subject\nFirst line\nSecond line", 'Subject', "First line\nSecond line"],
-            'CRLF' => ["Subject\r\nFirst line\r\nSecond line", 'Subject', "First line\r\nSecond line"],
-            'LFLF' => ["Subject\n\nFirst line\n\nSecond line", 'Subject', "\nFirst line\n\nSecond line"],
-            'CRCR' => ["Subject\r\rFirst line\r\rSecond line", 'Subject', "\rFirst line\r\rSecond line"],
-        ];
+        yield 'CR' => ["Subject\rFirst line\rSecond line", 'Subject', "First line\rSecond line"];
+        yield 'LF' => ["Subject\nFirst line\nSecond line", 'Subject', "First line\nSecond line"];
+        yield 'CRLF' => ["Subject\r\nFirst line\r\nSecond line", 'Subject', "First line\r\nSecond line"];
+        yield 'LFLF' => ["Subject\n\nFirst line\n\nSecond line", 'Subject', "\nFirst line\n\nSecond line"];
+        yield 'CRCR' => ["Subject\r\rFirst line\r\rSecond line", 'Subject', "\rFirst line\r\rSecond line"];
     }
 
     private function getMailer(): Mailer