tidy up

Author: TomasVotruba

rules-tests/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHash/Fixture/fixture_named_arguments.php.inc

@@ -2,7 +2,7 @@
 
 namespace Rector\Tests\DowngradePhp81\Rector\FuncCall\DowngradeHashAlgorithmXxHash\Fixture;
 
-class Fixture
+final class FixtureNamedArguments
 {
     public function run()
     {
@@ -16,7 +16,7 @@ class Fixture
 
 namespace Rector\Tests\DowngradePhp81\Rector\FuncCall\DowngradeHashAlgorithmXxHash\Fixture;
 
-class Fixture
+final class FixtureNamedArguments
 {
     public function run()
     {

rules-tests/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHash/Fixture/fixture_named_arguments_false_order.php.inc

@@ -2,11 +2,15 @@
 
 namespace Rector\Tests\DowngradePhp81\Rector\FuncCall\DowngradeHashAlgorithmXxHash\Fixture;
 
-class Fixture
+final class FixtureXxh32
 {
     public function run()
     {
-        return hash('xxh32', 'some-data-to-hash');
+        $value = hash('xxh32', 'some-data-to-hash');
+
+        $value = hash('xxh64', 'some-data-to-hash');
+
+        $value = hash('xxh128', 'some-data-to-hash');
     }
 }
 
@@ -16,11 +20,15 @@ class Fixture
 
 namespace Rector\Tests\DowngradePhp81\Rector\FuncCall\DowngradeHashAlgorithmXxHash\Fixture;
 
-class Fixture
+final class FixtureXxh32
 {
     public function run()
     {
-        return hash('md5', 'some-data-to-hash');
+        $value = hash('md5', 'some-data-to-hash');
+
+        $value = hash('md5', 'some-data-to-hash');
+
+        $value = hash('md5', 'some-data-to-hash');
     }
 }
 

rules-tests/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHash/Fixture/fixture_xxh128.php.inc

@@ -6,8 +6,12 @@ final class SkipCheckVersionCompareTernary
 {
     public function run_v()
     {
-        return version_compare(PHP_VERSION, '8.1','>=')
+        $value = version_compare(PHP_VERSION, '8.1','>=')
             ? hash( 'xxh128', $value )
             : hash( 'md4', $value );
+
+        $value = version_compare(PHP_VERSION, '8.1','<')
+            ? hash( 'md4', $value )
+            : hash( 'xxh128', $value );
     }
 }

rules-tests/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHash/Fixture/fixture_xxh32.php.inc

@@ -68,16 +68,18 @@ function test(array|Traversable $data) {
 
     /**
      * @param Ternary|FuncCall $node
-     * @return null|FuncCall|NodeVisitor::DONT_TRAVERSE_CHILDREN
+     * @return null|FuncCall
      */
-    public function refactor(Node $node): null|FuncCall|int
+    public function refactor(Node $node): null|FuncCall
     {
         if ($node instanceof Ternary) {
             $hasIsArrayCheck = (bool) $this->betterNodeFinder->findFirst(
                 $node,
                 fn (Node $subNode): bool => $subNode instanceof FuncCall && $this->isName($subNode, 'is_array')
             );
 
+            // get tgype...
+
             if ($hasIsArrayCheck) {
                 return NodeVisitor::DONT_TRAVERSE_CHILDREN;
             }

rules-tests/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHash/Fixture/fixture_xxh64.php.inc

@@ -84,8 +84,7 @@ public function getRuleDefinition(): RuleDefinition
      */
     public function getNodeTypes(): array
     {
-        $stmtsAware = NodeGroup::STMTS_AWARE;
-        return [...$stmtsAware, Switch_::class, Return_::class, Expression::class, Echo_::class];
+        return [...NodeGroup::STMTS_AWARE, Switch_::class, Return_::class, Expression::class, Echo_::class];
     }
 
     /**