Use a single line to describe return type

Signed-off-by: Henrique Moody <[email protected]>
Respect · Feb 4, 2024 · b1555fb · b1555fb
1 parent c079ecb
commit b1555fb
Show file tree

Hide file tree

Showing 140 changed files with 552 additions and 1,096 deletions.
diff --git a/tests/unit/Rules/AlnumTest.php b/tests/unit/Rules/AlnumTest.php
@@ -19,9 +19,7 @@
 #[CoversClass(Alnum::class)]
 final class AlnumTest extends RuleTestCase
 {
-    /**
-     * @return array<array{Alnum, mixed}>
-     */
+    /** @return iterable<array{Alnum, mixed}> */
     public static function providerForValidInput(): iterable
     {
         return [
@@ -42,9 +40,7 @@ public static function providerForValidInput(): iterable
         ];
     }
 
-    /**
-     * @return array<array{Alnum, mixed}>
-     */
+    /** @return iterable<array{Alnum, mixed}> */
     public static function providerForInvalidInput(): iterable
     {
         return [

diff --git a/tests/unit/Rules/AlphaTest.php b/tests/unit/Rules/AlphaTest.php
@@ -19,10 +19,8 @@
 #[CoversClass(Alpha::class)]
 final class AlphaTest extends RuleTestCase
 {
-    /**
-     * @return array<string, array{Alpha, mixed}>
-     */
-    public static function providerForValidInput(): array
+    /** @return iterable<string, array{Alpha, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         return [
             'alphabetic' => [new Alpha(), 'alganet'],
@@ -33,10 +31,8 @@ public static function providerForValidInput(): array
         ];
     }
 
-    /**
-     * @return array<string, array{Alpha, mixed}>
-     */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<string, array{Alpha, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         return [
             'empty string' => [new Alpha(), ''],

diff --git a/tests/unit/Rules/AlwaysInvalidTest.php b/tests/unit/Rules/AlwaysInvalidTest.php
@@ -31,7 +31,7 @@ public function itShouldAlwaysBeInvalid(mixed $input): void
     /**
      * @return mixed[][]
      */
-    public static function providerForInvalidInput(): array
+    public static function providerForInvalidInput(): iterable
     {
         return [
             [0],

diff --git a/tests/unit/Rules/AlwaysValidTest.php b/tests/unit/Rules/AlwaysValidTest.php
@@ -31,7 +31,7 @@ public function itAlwaysBeValid(mixed $input): void
     /**
      * @return mixed[][]
      */
-    public static function providerForValidInput(): array
+    public static function providerForValidInput(): iterable
     {
         return [
             [0],

diff --git a/tests/unit/Rules/ArrayTypeTest.php b/tests/unit/Rules/ArrayTypeTest.php
@@ -19,10 +19,8 @@
 #[CoversClass(ArrayType::class)]
 final class ArrayTypeTest extends RuleTestCase
 {
-    /**
-     * @return array<array{ArrayType, mixed}>
-     */
-    public static function providerForValidInput(): array
+    /** @return iterable<array{ArrayType, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         $rule = new ArrayType();
 
@@ -32,10 +30,8 @@ public static function providerForValidInput(): array
         ];
     }
 
-    /**
-     * @return array<array{ArrayType, mixed}>
-     */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<array{ArrayType, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         $rule = new ArrayType();
 

diff --git a/tests/unit/Rules/ArrayValTest.php b/tests/unit/Rules/ArrayValTest.php
@@ -20,10 +20,8 @@
 #[CoversClass(ArrayVal::class)]
 final class ArrayValTest extends RuleTestCase
 {
-    /**
-     * @return array<array{ArrayVal, mixed}>
-     */
-    public static function providerForValidInput(): array
+    /** @return iterable<array{ArrayVal, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         $rule = new ArrayVal();
 
@@ -35,10 +33,8 @@ public static function providerForValidInput(): array
         ];
     }
 
-    /**
-     * @return array<array{ArrayVal, mixed}>
-     */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<array{ArrayVal, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         $rule = new ArrayVal();
 

diff --git a/tests/unit/Rules/Base64Test.php b/tests/unit/Rules/Base64Test.php
@@ -19,10 +19,8 @@
 #[CoversClass(Base64::class)]
 final class Base64Test extends RuleTestCase
 {
-    /**
-     * @return array<array{Base64, mixed}>
-     */
-    public static function providerForValidInput(): array
+    /** @return iterable<array{Base64, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         $rule = new Base64();
 
@@ -51,10 +49,8 @@ public static function providerForValidInput(): array
         ];
     }
 
-    /**
-     * @return array<array{Base64, mixed}>
-     */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<array{Base64, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         $rule = new Base64();
 

diff --git a/tests/unit/Rules/BaseTest.php b/tests/unit/Rules/BaseTest.php
@@ -17,10 +17,8 @@
 #[CoversClass(Base::class)]
 final class BaseTest extends RuleTestCase
 {
-    /**
-     * @return array<array{Base, mixed}>
-     */
-    public static function providerForValidInput(): array
+    /** @return iterable<array{Base, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         return [
             [new Base(2), '011010001'],
@@ -35,10 +33,8 @@ public static function providerForValidInput(): array
         ];
     }
 
-    /**
-     * @return array<array{Base, mixed}>
-     */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<array{Base, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         return [
             [new Base(2), ''],

diff --git a/tests/unit/Rules/BetweenTest.php b/tests/unit/Rules/BetweenTest.php
@@ -38,10 +38,8 @@ public function minimumValueShouldNotBeEqualsToMaximumValue(): void
         new Between(5, 5);
     }
 
-    /**
-     * @return array<array{Between, mixed}>
-     */
-    public static function providerForValidInput(): array
+    /** @return iterable<array{Between, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         return [
             [new Between(0, 1), 1],
@@ -56,10 +54,8 @@ public static function providerForValidInput(): array
         ];
     }
 
-    /**
-     * @return array<array{Between, mixed}>
-     */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<array{Between, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         return [
             [new Between(10, 20), ''],

diff --git a/tests/unit/Rules/BoolTypeTest.php b/tests/unit/Rules/BoolTypeTest.php
@@ -18,10 +18,8 @@
 #[CoversClass(BoolType::class)]
 final class BoolTypeTest extends RuleTestCase
 {
-    /**
-     * @return array<array{BoolType, mixed}>
-     */
-    public static function providerForValidInput(): array
+    /** @return iterable<array{BoolType, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         $rule = new BoolType();
 
@@ -31,10 +29,8 @@ public static function providerForValidInput(): array
         ];
     }
 
-    /**
-     * @return array<array{BoolType, mixed}>
-     */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<array{BoolType, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         $rule = new BoolType();
 

diff --git a/tests/unit/Rules/BoolValTest.php b/tests/unit/Rules/BoolValTest.php
@@ -17,10 +17,8 @@
 #[CoversClass(BoolVal::class)]
 final class BoolValTest extends RuleTestCase
 {
-    /**
-     * @return array<array{BoolVal, mixed}>
-     */
-    public static function providerForValidInput(): array
+    /** @return iterable<array{BoolVal, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         $rule = new BoolVal();
 
@@ -37,10 +35,8 @@ public static function providerForValidInput(): array
         ];
     }
 
-    /**
-     * @return array<array{BoolVal, mixed}>
-     */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<array{BoolVal, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         $rule = new BoolVal();
 

diff --git a/tests/unit/Rules/BsnTest.php b/tests/unit/Rules/BsnTest.php
@@ -17,10 +17,8 @@
 #[CoversClass(Bsn::class)]
 final class BsnTest extends RuleTestCase
 {
-    /**
-     * @return array<array{Bsn, mixed}>
-     */
-    public static function providerForValidInput(): array
+    /** @return iterable<array{Bsn, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         $rule = new Bsn();
 
@@ -38,10 +36,8 @@ public static function providerForValidInput(): array
         ];
     }
 
-    /**
-     * @return array<array{Bsn, mixed}>
-     */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<array{Bsn, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         $rule = new Bsn();
 

diff --git a/tests/unit/Rules/CallTest.php b/tests/unit/Rules/CallTest.php
@@ -90,16 +90,16 @@ public function itShouldRestorePreviousPhpErrorHandler(): void
         trigger_error('Forcing PHP to trigger an error');
     }
 
-    /** @return array<string, array{Call, mixed}> */
-    public static function providerForValidInput(): array
+    /** @return iterable<string, array{Call, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         return [
             'valid rule and valid callable' => [new Call('trim', Stub::pass(1)), ' input '],
         ];
     }
 
-    /** @return array<string, array{Call, mixed}> */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<string, array{Call, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         return [
             'PHP error' => [new Call('trim', Stub::pass(1)), []],

diff --git a/tests/unit/Rules/CallableTypeTest.php b/tests/unit/Rules/CallableTypeTest.php
@@ -21,10 +21,8 @@
 #[CoversClass(CallableType::class)]
 final class CallableTypeTest extends RuleTestCase
 {
-    /**
-     * @return array<array{CallableType, mixed}>
-     */
-    public static function providerForValidInput(): array
+    /** @return iterable<array{CallableType, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         $rule = new CallableType();
 
@@ -36,10 +34,8 @@ public static function providerForValidInput(): array
         ];
     }
 
-    /**
-     * @return array<array{CallableType, mixed}>
-     */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<array{CallableType, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         $rule = new CallableType();
 

diff --git a/tests/unit/Rules/CallbackTest.php b/tests/unit/Rules/CallbackTest.php
@@ -19,10 +19,8 @@
 #[CoversClass(Callback::class)]
 final class CallbackTest extends RuleTestCase
 {
-    /**
-     * @return array<array{Callback, mixed}>
-     */
-    public static function providerForValidInput(): array
+    /** @return iterable<array{Callback, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         return [
             [new Callback('is_a', 'stdClass'), new stdClass()],
@@ -37,10 +35,8 @@ public static function providerForValidInput(): array
         ];
     }
 
-    /**
-     * @return array<array{Callback, mixed}>
-     */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<array{Callback, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         return [
             [

diff --git a/tests/unit/Rules/CharsetTest.php b/tests/unit/Rules/CharsetTest.php
@@ -30,10 +30,8 @@ public function itShouldThrowsExceptionWhenCharsetIsNotValid(): void
         new Charset('UTF-8', 'UTF-9');
     }
 
-    /**
-     * @return array<array{Charset, mixed}>
-     */
-    public static function providerForValidInput(): array
+    /** @return iterable<array{Charset, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         return [
             [new Charset('UTF-8'), ''],
@@ -47,10 +45,8 @@ public static function providerForValidInput(): array
         ];
     }
 
-    /**
-     * @return array<array{Charset, mixed}>
-     */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<array{Charset, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         $rule = new Charset('ASCII');
 

diff --git a/tests/unit/Rules/CnhTest.php b/tests/unit/Rules/CnhTest.php
@@ -18,10 +18,8 @@
 #[CoversClass(Cnh::class)]
 final class CnhTest extends RuleTestCase
 {
-    /**
-     * @return array<array{Cnh, mixed}>
-     */
-    public static function providerForValidInput(): array
+    /** @return iterable<array{Cnh, mixed}> */
+    public static function providerForValidInput(): iterable
     {
         $rule = new Cnh();
 
@@ -53,10 +51,8 @@ public static function providerForValidInput(): array
         ];
     }
 
-    /**
-     * @return array<array{Cnh, mixed}>
-     */
-    public static function providerForInvalidInput(): array
+    /** @return iterable<array{Cnh, mixed}> */
+    public static function providerForInvalidInput(): iterable
     {
         $rule = new Cnh();