remove switch

Author: TomasVotruba

phpstan.neon

@@ -43,8 +43,6 @@ parameters:
         # see https://github.com/rectorphp/rector-src/actions/runs/11798721617/job/32865546672?pr=6422#step:5:110
         - '#Doing instanceof PHPStan\\Type\\.+ is error\-prone and deprecated#'
 
-        #- identifier: instanceof.alwaysTrue
-
         # false positive
         -
             identifier: assign.propertyType

rules-tests/CodeQuality/Rector/MethodCall/AssertEqualsToSameRector/Fixture/skip_if_an_object_is_in_any_level_of_an_array.php

@@ -6,6 +6,7 @@
 
 use PhpParser\Node\AttributeGroup;
 use Rector\PhpAttribute\NodeFactory\PhpAttributeGroupFactory;
+use Rector\PHPUnit\AnnotationsToAttributes\ValueObject\RequiresAttributeAndValue;
 use Rector\PHPUnit\Enum\PHPUnitAttribute;
 
 final readonly class RequiresAttributeFactory
@@ -22,58 +23,74 @@ public function create(string $annotationValue): ?AttributeGroup
         $type = array_shift($annotationValues);
         $attributeValue = array_shift($annotationValues);
 
-        switch ($type) {
-            case 'PHP':
-                $attributeClass = PHPUnitAttribute::REQUIRES_PHP;
+        $requiresAttributeAndValue = $this->matchRequiresAttributeAndValue($type, $attributeValue);
+        if (! $requiresAttributeAndValue instanceof RequiresAttributeAndValue) {
+            return null;
+        }
 
-                // only version is used, we need to prefix with >=
-                if (is_string($attributeValue) && is_numeric($attributeValue[0])) {
-                    $attributeValue = '>= ' . $attributeValue;
-                }
+        return $this->phpAttributeGroupFactory->createFromClassWithItems(
+            $requiresAttributeAndValue->getAttributeClass(),
+            $requiresAttributeAndValue->getValue()
+        );
+    }
 
-                $attributeValue = [$attributeValue];
-                break;
-            case 'PHPUnit':
-                $attributeClass = PHPUnitAttribute::REQUIRES_PHPUNIT;
+    private function matchRequiresAttributeAndValue(string $type, mixed $attributeValue): ?RequiresAttributeAndValue
+    {
+        if ($type === 'PHP') {
+            // only version is used, we need to prefix with >=
+            if (is_string($attributeValue) && is_numeric($attributeValue[0])) {
+                $attributeValue = '>= ' . $attributeValue;
+            }
+
+            return new RequiresAttributeAndValue(PHPUnitAttribute::REQUIRES_PHP, [$attributeValue]);
+        }
 
-                // only version is used, we need to prefix with >=
-                if (is_string($attributeValue) && is_numeric($attributeValue[0])) {
-                    $attributeValue = '>= ' . $attributeValue;
-                }
+        if ($type === 'PHPUnit') {
+            // only version is used, we need to prefix with >=
+            if (is_string($attributeValue) && is_numeric($attributeValue[0])) {
+                $attributeValue = '>= ' . $attributeValue;
+            }
 
+            return new RequiresAttributeAndValue(PHPUnitAttribute::REQUIRES_PHPUNIT, [$attributeValue]);
+        }
+
+        if ($type === 'OS') {
+            return new RequiresAttributeAndValue(PHPUnitAttribute::REQUIRES_OS, [$attributeValue]);
+        }
+
+        if ($type === 'OSFAMILY') {
+            return new RequiresAttributeAndValue(PHPUnitAttribute::REQUIRES_OS_FAMILY, [$attributeValue]);
+        }
+
+        if ($type === 'function') {
+            if (str_contains((string) $attributeValue, '::')) {
+                $attributeClass = PHPUnitAttribute::REQUIRES_METHOD;
+                $attributeValue = explode('::', (string) $attributeValue);
+                $attributeValue[0] .= '::class';
+            } else {
+                $attributeClass = PHPUnitAttribute::REQUIRES_FUNCTION;
                 $attributeValue = [$attributeValue];
-                break;
-            case 'OS':
-                $attributeClass = PHPUnitAttribute::REQUIRES_OS;
-                $attributeValue = [$attributeValue];
-                break;
-            case 'OSFAMILY':
-                $attributeClass = PHPUnitAttribute::REQUIRES_OS_FAMILY;
-                $attributeValue = [$attributeValue];
-                break;
-            case 'function':
-                if (str_contains((string) $attributeValue, '::')) {
-                    $attributeClass = PHPUnitAttribute::REQUIRES_METHOD;
-                    $attributeValue = explode('::', (string) $attributeValue);
-                    $attributeValue[0] .= '::class';
-                } else {
-                    $attributeClass = PHPUnitAttribute::REQUIRES_FUNCTION;
-                    $attributeValue = [$attributeValue];
-                }
-
-                break;
-            case 'extension':
-                $attributeClass = PHPUnitAttribute::REQUIRES_PHP_EXTENSION;
-                $attributeValue = explode(' ', (string) $attributeValue, 2);
-                break;
-            case 'setting':
-                $attributeClass = PHPUnitAttribute::REQUIRES_SETTING;
-                $attributeValue = explode(' ', (string) $attributeValue, 2);
-                break;
-            default:
-                return null;
+            }
+
+            return new RequiresAttributeAndValue($attributeClass, $attributeValue);
+        }
+
+        if ($type === 'extension') {
+            return new RequiresAttributeAndValue(PHPUnitAttribute::REQUIRES_PHP_EXTENSION, explode(
+                ' ',
+                (string) $attributeValue,
+                2
+            ));
+        }
+
+        if ($type === 'setting') {
+            return new RequiresAttributeAndValue(PHPUnitAttribute::REQUIRES_SETTING, explode(
+                ' ',
+                (string) $attributeValue,
+                2
+            ));
         }
 
-        return $this->phpAttributeGroupFactory->createFromClassWithItems($attributeClass, [...$attributeValue]);
+        return null;
     }
 }

rules/AnnotationsToAttributes/NodeFactory/RequiresAttributeFactory.php

@@ -0,0 +1,36 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\PHPUnit\AnnotationsToAttributes\ValueObject;
+
+use Rector\PHPUnit\Enum\PHPUnitAttribute;
+
+final readonly class RequiresAttributeAndValue
+{
+    /**
+     * @param PHPUnitAttribute::* $attributeClass
+     * @param string[] $value
+     */
+    public function __construct(
+        private string $attributeClass,
+        private array $value,
+    ) {
+    }
+
+    /**
+     * @return PHPUnitAttribute::*
+     */
+    public function getAttributeClass(): string
+    {
+        return $this->attributeClass;
+    }
+
+    /**
+     * @return string[]
+     */
+    public function getValue(): array
+    {
+        return $this->value;
+    }
+}