Do not require return type for renamed trait constructor [close #23] (#…

…24)

solved in phpstan in phpstan/phpstan#9571
phpstan/phpstan-src@f6ca9c8

src/Collectors/ReturnTypeDeclarationCollector.php

@@ -30,6 +30,13 @@ public function processNode(Node $node, Scope $scope): ?array
             return null;
         }
 
+		if ($scope->isInTrait()) {
+			$originalMethodName = $node->getAttribute('originalTraitMethodName');
+			if ($originalMethodName === '__construct') {
+				return null;
+			}
+		}
+
         $missingTypeLines = [];
 
         if (! $node->returnType instanceof Node) {

tests/Rules/ReturnTypeCoverageRule/Fixture/SkipTraitConstructor.php

@@ -0,0 +1,24 @@
+<?php
+
+declare(strict_types=1);
+
+namespace TomasVotruba\TypeCoverage\Tests\Rules\ReturnTypeCoverageRule\Fixture;
+
+final class SkipTraitConstructor
+{
+    use TraitWithConstructor {
+		TraitWithConstructor::__construct as traitConstruct;
+    }
+
+    public function __construct()
+    {
+        $this->traitConstruct();
+    }
+}
+
+trait TraitWithConstructor
+{
+    public function __construct()
+    {
+    }
+}

tests/Rules/ReturnTypeCoverageRule/ReturnTypeCoverageRuleTest.php

@@ -28,6 +28,7 @@ public static function provideData(): Iterator
     {
         yield [[__DIR__ . '/Fixture/SkipKnownReturnType.php', __DIR__ . '/Fixture/SkipAgainKnownReturnType.php'], []];
         yield [[__DIR__ . '/Fixture/SkipConstructor.php'], []];
+        yield [[__DIR__ . '/Fixture/SkipTraitConstructor.php'], []];
 
         $errorMessage = sprintf(ReturnTypeCoverageRule::ERROR_MESSAGE, 2, 0, 0, 80);
         yield [[__DIR__ . '/Fixture/UnknownReturnType.php'], [[$errorMessage, 9], [$errorMessage, 13]]];