[CodeQuality] Skip test method via #[Test] Attribute on DataProviderArrayItemsNewLinedRector (#664)

* [CodeQuality] Skip test method via #[Test] Attribute

* fix

Author: samsonasik

rules-tests/CodeQuality/Rector/ClassMethod/DataProviderArrayItemsNewLinedRector/Fixture/skip_test_method_via_test_attribute.php.inc

@@ -0,0 +1,32 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\ClassMethod\DataProviderArrayItemsNewlinedRector\Fixture;
+
+use PHPUnit\Framework\Attributes\Test;
+use PHPUnit\Framework\TestCase;
+
+final class SkipTestMethod extends TestCase
+{
+    #[Test]
+    public function run(): void
+    {
+        $importer = self::createMock(Importer::class);
+        $importer->expects($invokedCount)
+            ->method('processFiles')
+            ->willReturnCallback(function (array $parameters) use ($invokedCount): array {
+                switch ($invokedCount->numberOfInvocations()) {
+                    case 1:
+                        self::assertSame(range(1, 250), $parameters);
+
+                        return [233, [], []];
+                    case 2:
+                        self::assertSame(range(251, 500), $parameters);
+
+                        return [15, [], []];
+
+                    default:
+                        throw new \OutOfBoundsException('Did not expect this to run thrice.');
+                }
+            });
+    }
+}

rules/CodeQuality/Rector/ClassMethod/DataProviderArrayItemsNewLinedRector.php

@@ -9,7 +9,9 @@
 use PhpParser\Node\Stmt\ClassMethod;
 use PhpParser\Node\Stmt\Return_;
 use Rector\NodeTypeResolver\Node\AttributeKey;
+use Rector\Php80\NodeAnalyzer\PhpAttributeAnalyzer;
 use Rector\PhpParser\Node\BetterNodeFinder;
+use Rector\PHPUnit\Enum\PHPUnitAttribute;
 use Rector\PHPUnit\NodeAnalyzer\TestsNodeAnalyzer;
 use Rector\Rector\AbstractRector;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
@@ -22,7 +24,8 @@ final class DataProviderArrayItemsNewLinedRector extends AbstractRector
 {
     public function __construct(
         private readonly TestsNodeAnalyzer $testsNodeAnalyzer,
-        private readonly BetterNodeFinder $betterNodeFinder
+        private readonly BetterNodeFinder $betterNodeFinder,
+        private readonly PhpAttributeAnalyzer $phpAttributeAnalyzer
     ) {
     }
 
@@ -96,7 +99,8 @@ public function refactor(Node $node): ?Node
         }
 
         // skip test methods
-        if (str_starts_with($node->name->toString(), 'test')) {
+        if (str_starts_with($node->name->toString(), 'test')
+            || $this->phpAttributeAnalyzer->hasPhpAttribute($node, PHPUnitAttribute::TEST)) {
             return null;
         }