Test: Add test for PhpDoc property fixer/sniff

Author: OndraM

tests/Integration/CodingStandardTest.php

@@ -36,6 +36,7 @@ public function provideFilesToFix(): array
                 __DIR__ . '/Fixtures/NewPhpFeatures.wrong.php.inc',
                 __DIR__ . '/Fixtures/NewPhpFeatures.correct.php.inc',
             ],
+            'PhpDoc' => [__DIR__ . '/Fixtures/PhpDoc.wrong.php.inc', __DIR__ . '/Fixtures/PhpDoc.correct.php.inc'],
         ];
     }
 

tests/Integration/Fixtures/Basic.correct.php.inc

@@ -127,19 +127,6 @@ class Basic extends AbstractBasic implements InterfaceFromThisNamespace // Fully
         ];
     }
 
-    /**
-     * Very well documented method.
-     * It tests PhpdocAlignFixer, NoSuperfluousPhpdocTagsFixer and possibly other Phpdoc rules.
-     * @param int|float $second Second parameter does have a comment, unlike the first one.
-     * @param string|null $third Third parameter is optional and has a default value.
-     * @return mixed There is also information about return type.
-     * @throws \Exception
-     */
-    public function veryWellDocumented(string $first, int|float $second, ?string $third = '3rd'): mixed
-    {
-        return $first . $third;
-    }
-
     public function emptyFunction1(): void {} // SingleLineEmptyBodyFixer
 
     public function emptyFunction2(

tests/Integration/Fixtures/Basic.wrong.php.inc

@@ -120,20 +120,6 @@ but should be heredoc instead';
         ];
     }
 
-    /**
-     * Very well documented method.
-     * It tests PhpdocAlignFixer, NoSuperfluousPhpdocTagsFixer and possibly other Phpdoc rules.
-     * @param   string      $first
-     * @throws  \Exception
-     * @param   int|float   $second Second parameter does have a comment, unlike the first one.
-     * @param   string|null $third Third parameter is optional and has a default value.
-     * @return  mixed There is also information about return type.
-     */
-    public function veryWellDocumented(string $first, int|float $second, ?string $third = '3rd'): mixed
-    {
-        return $first . $third;
-    }
-
     public function emptyFunction1(): void {
     } // SingleLineEmptyBodyFixer
 

tests/Integration/Fixtures/PhpDoc.correct.php.inc

@@ -0,0 +1,25 @@
+<?php declare(strict_types=1);
+
+namespace Lmc\CodingStandard\Integration\Fixtures;
+
+class PhpDoc
+{
+    // PhpdocToPropertyTypeFixer, PropertyTypeHintSniff
+    private int $integerValue = 3;
+    private $undefinedTypeValue = 3;
+    private mixed $undeclaredMixedType;
+    private \DateTimeImmutable $nonScalarType;
+
+    /**
+     * Very well documented method.
+     * It tests PhpdocAlignFixer, NoSuperfluousPhpdocTagsFixer and possibly other Phpdoc rules.
+     * @param int|float $second Second parameter does have a comment, unlike the first one.
+     * @param string|null $third Third parameter is optional and has a default value.
+     * @return mixed There is also information about return type.
+     * @throws \Exception
+     */
+    public function veryWellDocumented(string $first, int|float $second, ?string $third = '3rd'): mixed
+    {
+        return $first . $third;
+    }
+}

tests/Integration/Fixtures/PhpDoc.wrong.php.inc

@@ -0,0 +1,29 @@
+<?php declare(strict_types=1);
+
+namespace Lmc\CodingStandard\Integration\Fixtures;
+
+class PhpDoc
+{
+    // PhpdocToPropertyTypeFixer, PropertyTypeHintSniff
+    /** @var int */
+    private $integerValue = 3;
+    private $undefinedTypeValue = 3;
+    /** @var mixed */
+    private $undeclaredMixedType;
+    /** @var \DateTimeImmutable */
+    private $nonScalarType;
+
+    /**
+     * Very well documented method.
+     * It tests PhpdocAlignFixer, NoSuperfluousPhpdocTagsFixer and possibly other Phpdoc rules.
+     * @param   string      $first
+     * @throws  \Exception
+     * @param   int|float   $second Second parameter does have a comment, unlike the first one.
+     * @param   string|null $third Third parameter is optional and has a default value.
+     * @return  mixed There is also information about return type.
+     */
+    public function veryWellDocumented(string $first, int|float $second, ?string $third = '3rd'): mixed
+    {
+        return $first . $third;
+    }
+}