[CodeQuality] Add AddTraitGetterReturnTypeBasedOnSetterRequiredRector (#671)

* [CodeQuality] Add AddTraitGetterReturnTypeBasedOnSetterRequiredRector

* [CodeQuality] Add AddTraitGetterReturnTypeBasedOnSetterRequiredRector

* Fix phpstan

* Add Required attribute support

* cs fix

Author: samsonasik

rules-tests/CodeQuality/Rector/Trait_/AddTraitGetterReturnTypeBasedOnSetterRequiredRector/AddTraitGetterReturnTypeBasedOnSetterRequiredRectorTest.php

@@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Symfony\Tests\CodeQuality\Rector\Trait_\AddTraitGetterReturnTypeBasedOnSetterRequiredRector;
+
+use Iterator;
+use PHPUnit\Framework\Attributes\DataProvider;
+use Rector\Testing\PHPUnit\AbstractRectorTestCase;
+
+final class AddTraitGetterReturnTypeBasedOnSetterRequiredRectorTest extends AbstractRectorTestCase
+{
+    #[DataProvider('provideData')]
+    public function test(string $filePath): void
+    {
+        $this->doTestFile($filePath);
+    }
+
+    public static function provideData(): Iterator
+    {
+        return self::yieldFilesFromDirectory(__DIR__ . '/Fixture');
+    }
+
+    public function provideConfigFilePath(): string
+    {
+        return __DIR__ . '/config/configured_rule.php';
+    }
+}

rules-tests/CodeQuality/Rector/Trait_/AddTraitGetterReturnTypeBasedOnSetterRequiredRector/Fixture/flip_set_first.php.inc

@@ -0,0 +1,51 @@
+<?php
+
+namespace Rector\Symfony\Tests\CodeQuality\Rector\Trait_\AddTraitGetterReturnTypeBasedOnSetterRequiredRector\Fixture;
+
+use stdClass;
+
+trait FlipSetFirst
+{
+    private $service;
+
+    /**
+     * @required
+     */
+    public function setService(stdClass $stdClass)
+    {
+        $this->stdClass = $stdClass;
+    }
+
+    public function getService()
+    {
+        return $this->service;
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Symfony\Tests\CodeQuality\Rector\Trait_\AddTraitGetterReturnTypeBasedOnSetterRequiredRector\Fixture;
+
+use stdClass;
+
+trait FlipSetFirst
+{
+    private $service;
+
+    /**
+     * @required
+     */
+    public function setService(stdClass $stdClass)
+    {
+        $this->stdClass = $stdClass;
+    }
+
+    public function getService(): stdClass
+    {
+        return $this->service;
+    }
+}
+
+?>

rules-tests/CodeQuality/Rector/Trait_/AddTraitGetterReturnTypeBasedOnSetterRequiredRector/Fixture/skip_already_has_return_type.php.inc

@@ -0,0 +1,25 @@
+<?php
+
+namespace Rector\Symfony\Tests\CodeQuality\Rector\Trait_\AddTraitGetterReturnTypeBasedOnSetterRequiredRector\Fixture;
+
+use stdClass;
+
+trait SkipAlreadyHasReturnType
+{
+    private $service;
+
+    public function getService(): stdClass
+    {
+        return $this->service;
+    }
+
+    /**
+     * @required
+     */
+    public function setService(stdClass $stdClass)
+    {
+        $this->stdClass = $stdClass;
+    }
+}
+
+?>

rules-tests/CodeQuality/Rector/Trait_/AddTraitGetterReturnTypeBasedOnSetterRequiredRector/Fixture/some_trait.php.inc

@@ -0,0 +1,51 @@
+<?php
+
+namespace Rector\Symfony\Tests\CodeQuality\Rector\Trait_\AddTraitGetterReturnTypeBasedOnSetterRequiredRector\Fixture;
+
+use stdClass;
+
+trait SomeTrait
+{
+    private $service;
+
+    public function getService()
+    {
+        return $this->service;
+    }
+
+    /**
+     * @required
+     */
+    public function setService(stdClass $stdClass)
+    {
+        $this->stdClass = $stdClass;
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Symfony\Tests\CodeQuality\Rector\Trait_\AddTraitGetterReturnTypeBasedOnSetterRequiredRector\Fixture;
+
+use stdClass;
+
+trait SomeTrait
+{
+    private $service;
+
+    public function getService(): stdClass
+    {
+        return $this->service;
+    }
+
+    /**
+     * @required
+     */
+    public function setService(stdClass $stdClass)
+    {
+        $this->stdClass = $stdClass;
+    }
+}
+
+?>

rules-tests/CodeQuality/Rector/Trait_/AddTraitGetterReturnTypeBasedOnSetterRequiredRector/Fixture/with_required_attribute.php.inc

@@ -0,0 +1,49 @@
+<?php
+
+namespace Rector\Symfony\Tests\CodeQuality\Rector\Trait_\AddTraitGetterReturnTypeBasedOnSetterRequiredRector\Fixture;
+
+use stdClass;
+use Symfony\Contracts\Service\Attribute\Required;
+
+trait WithRequiredAttribute
+{
+    private $service;
+
+    public function getService()
+    {
+        return $this->service;
+    }
+
+    #[Required]
+    public function setService(stdClass $stdClass)
+    {
+        $this->stdClass = $stdClass;
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Symfony\Tests\CodeQuality\Rector\Trait_\AddTraitGetterReturnTypeBasedOnSetterRequiredRector\Fixture;
+
+use stdClass;
+use Symfony\Contracts\Service\Attribute\Required;
+
+trait WithRequiredAttribute
+{
+    private $service;
+
+    public function getService(): stdClass
+    {
+        return $this->service;
+    }
+
+    #[Required]
+    public function setService(stdClass $stdClass)
+    {
+        $this->stdClass = $stdClass;
+    }
+}
+
+?>

rules-tests/CodeQuality/Rector/Trait_/AddTraitGetterReturnTypeBasedOnSetterRequiredRector/config/configured_rule.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\Symfony\CodeQuality\Rector\Trait_\AddTraitGetterReturnTypeBasedOnSetterRequiredRector;
+
+return static function (RectorConfig $rectorConfig): void {
+    $rectorConfig->rule(AddTraitGetterReturnTypeBasedOnSetterRequiredRector::class);
+};