ExplicitReturnNullRector should skip never return (#5802)

* ExplicitReturnNullRector should skip never return

* cs

* cs

rules-tests/CodeQuality/Rector/ClassMethod/ExplicitReturnNullRector/Fixture/skip-never-return.php.inc

@@ -0,0 +1,21 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\ClassMethod\ExplicitReturnNullRector\Fixture;
+
+final class SkipNeverReturn
+{
+    public function run(bool $param)
+    {
+        if (rand(0,1)) {
+            return 5;
+        }
+
+        $this->neverReturns();
+    }
+
+    private function neverReturns($value): never
+    {
+        exit();
+    }
+}
+?>

rules/CodeQuality/Rector/ClassMethod/ExplicitReturnNullRector.php

@@ -21,6 +21,7 @@
 use Rector\BetterPhpDocParser\PhpDocManipulator\PhpDocTypeChanger;
 use Rector\NodeTypeResolver\PHPStan\Type\TypeFactory;
 use Rector\Rector\AbstractRector;
+use Rector\TypeDeclaration\NodeAnalyzer\NeverFuncCallAnalyzer;
 use Rector\TypeDeclaration\TypeInferer\ReturnTypeInferer;
 use Rector\TypeDeclaration\TypeInferer\SilentVoidResolver;
 use Rector\ValueObject\MethodName;
@@ -37,6 +38,7 @@ public function __construct(
         private readonly PhpDocInfoFactory $phpDocInfoFactory,
         private readonly TypeFactory $typeFactory,
         private readonly PhpDocTypeChanger $phpDocTypeChanger,
+        private readonly NeverFuncCallAnalyzer $neverFuncCallAnalyzer,
         private readonly ReturnTypeInferer $returnTypeInferer
     ) {
     }
@@ -129,7 +131,9 @@ public function refactor(Node $node): ?Node
             return null;
         });
 
-        if (! $this->silentVoidResolver->hasSilentVoid($node)) {
+        if (! $this->silentVoidResolver->hasSilentVoid($node)
+            || $this->neverFuncCallAnalyzer->hasNeverFuncCall($node)
+        ) {
             if ($hasChanged) {
                 $this->transformDocUnionVoidToUnionNull($node);
                 return $node;