[CodeQuality] Skip Switch_ always returned on ExplicitReturnNullRector (

#5800)

* [CodeQuality] Skip Switch_ always returned on ExplicitReturnNullRector

* [ci-review] Rector Rectify

---------

Co-authored-by: GitHub Action <actions@github.com>

rules-tests/CodeQuality/Rector/ClassMethod/ExplicitReturnNullRector/Fixture/skip_switch_always_return.php.inc

@@ -0,0 +1,17 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\ClassMethod\ExplicitReturnNullRector\Fixture;
+
+final class SkipSwitchAlwaysReturn
+{
+    public function run(int $number)
+    {
+        switch ($number) {
+            case 1:
+                return 1;
+            case 2:
+            default:
+                return 2;
+        }
+    }
+}

rules/TypeDeclaration/TypeInferer/SilentVoidResolver.php

@@ -14,6 +14,7 @@
 use PhpParser\Node\FunctionLike;
 use PhpParser\Node\Stmt;
 use PhpParser\Node\Stmt\Break_;
+use PhpParser\Node\Stmt\Case_;
 use PhpParser\Node\Stmt\ClassMethod;
 use PhpParser\Node\Stmt\Continue_;
 use PhpParser\Node\Stmt\Do_;
@@ -166,8 +167,10 @@ private function isSwitchWithAlwaysReturnOrExit(Switch_ $switch): bool
 
         $casesWithReturnOrExitCount = $this->resolveReturnOrExitCount($switch);
 
+        $cases = array_filter($switch->cases, static fn(Case_ $case): bool => $case->stmts !== []);
+
         // has same amount of first return or exit nodes as switches
-        return count($switch->cases) === $casesWithReturnOrExitCount;
+        return count($cases) === $casesWithReturnOrExitCount;
     }
 
     private function isTryCatchAlwaysReturnOrExit(TryCatch $tryCatch): bool