skip existing Iterator return in AddReturnDocblockDataProviderRector

Author: TomasVotruba

rules-tests/TypeDeclarationDocblocks/Rector/Class_/AddReturnDocblockDataProviderRector/Fixture/skip_already_set_type.php.inc

@@ -0,0 +1,37 @@
+<?php
+
+namespace Rector\Tests\TypeDeclarationDocblocks\Rector\Class_\AddReturnDocblockDataProviderRector\Fixture;
+
+use PHPUnit\Framework\Attributes\DataProvider;
+use PHPUnit\Framework\TestCase;
+
+final class SkipAlreadySetType extends TestCase
+{
+    #[DataProvider('provideData')]
+    public function testSomething()
+    {
+    }
+
+    /**
+     * @return \Generator<array<int, array{0: string, 1: string}>>
+     */
+    public static function provideData(): \Generator
+    {
+        yield ['data1', 'data2'];
+        yield ['item4', 'item5'];
+    }
+
+    #[DataProvider('provideDataNext')]
+    public function testSomethingElse()
+    {
+    }
+
+    /**
+     * @return \Iterator<array<int, array{0: string, 1: string}>>
+     */
+    public static function provideDataNext(): \Iterator
+    {
+        yield ['data1', 'data2'];
+        yield ['item4', 'item5'];
+    }
+}

tests/BetterPhpDocParser/PhpDocInfo/PhpDocInfoPrinter/MultilineTest.php

@@ -23,6 +23,9 @@ public function test(string $docFilePath): void
         $this->assertSame($docComment, $printedPhpDocInfo);
     }
 
+    /**
+     * @return Iterator<array<int, string>>
+     */
     public static function provideData(): Iterator
     {
         yield [__DIR__ . '/Source/Multiline/multiline2.txt'];

tests/Console/Formatter/ColorConsoleDiffFormatterTest.php

@@ -28,6 +28,9 @@ public function test(string $content, string $expectedFormattedFileContent): voi
         $this->assertStringEqualsFile($expectedFormattedFileContent, $formattedContent);
     }
 
+    /**
+     * @return Iterator<(array<int, string>|array<int, string>)>
+     */
     public static function provideData(): Iterator
     {
         yield ['...', __DIR__ . '/Source/expected/expected.txt'];

tests/DependencyInjection/ConfigurableRectorImportConfigCallsMergeTest.php

@@ -30,6 +30,9 @@ public function testMainConfigValues(string $configFile, array $expectedConfigur
         $this->assertSame($expectedConfiguration, $renamedClassesDataCollector->getOldToNewClasses());
     }
 
+    /**
+     * @return Iterator<(array<int, array<string, string>>|array<int, string>)>
+     */
     public static function provideData(): Iterator
     {
         yield [

tests/NodeTypeResolver/PerNodeTypeResolver/ClassAndInterfaceTypeResolver/ClassTypeResolverTest.php

@@ -32,6 +32,9 @@ public function test(string $file, int $nodePosition, ObjectType $expectedObject
         $this->assertSame($expectedObjectType->getClassName(), $resolvedType->getClassName());
     }
 
+    /**
+     * @return Iterator<array<int, (int|ObjectType|string)>>
+     */
     public static function dataProvider(): Iterator
     {
         yield [