Add nullable support to ScalarArgumentToExpectedParamTypeRector

rules-tests/CodeQuality/Rector/MethodCall/ScalarArgumentToExpectedParamTypeRector/Fixture/pass_to_nullable_string.php.inc

@@ -0,0 +1,35 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\MethodCall\ScalarArgumentToExpectedParamTypeRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+use Rector\PHPUnit\Tests\CodeQuality\Rector\MethodCall\ScalarArgumentToExpectedParamTypeRector\Source\NullableSetter;
+
+final class PassToNullableString extends TestCase
+{
+    public function test()
+    {
+        $nullableSetter = new NullableSetter();
+        $nullableSetter->setMaybe(123456);
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\MethodCall\ScalarArgumentToExpectedParamTypeRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+use Rector\PHPUnit\Tests\CodeQuality\Rector\MethodCall\ScalarArgumentToExpectedParamTypeRector\Source\NullableSetter;
+
+final class PassToNullableString extends TestCase
+{
+    public function test()
+    {
+        $nullableSetter = new NullableSetter();
+        $nullableSetter->setMaybe('123456');
+    }
+}
+
+?>

rules-tests/CodeQuality/Rector/MethodCall/ScalarArgumentToExpectedParamTypeRector/Source/NullableSetter.php

@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\MethodCall\ScalarArgumentToExpectedParamTypeRector\Source;
+
+final class NullableSetter
+{
+    public function setMaybe(?string $status)
+    {
+    }
+}

rules/CodeQuality/Rector/MethodCall/ScalarArgumentToExpectedParamTypeRector.php

@@ -13,6 +13,7 @@
 use PHPStan\Type\IntegerType;
 use PHPStan\Type\StringType;
 use PHPStan\Type\Type;
+use PHPStan\Type\TypeCombinator;
 use Rector\PHPUnit\CodeQuality\Reflection\MethodParametersAndReturnTypesResolver;
 use Rector\PHPUnit\NodeAnalyzer\TestsNodeAnalyzer;
 use Rector\Rector\AbstractRector;
@@ -93,24 +94,11 @@ public function getNodeTypes(): array
      */
     public function refactor(Node $node): ?Node
     {
-        if (! $this->testsNodeAnalyzer->isInTestClass($node)) {
-            return null;
-        }
-
-        if ($node->isFirstClassCallable()) {
-            return null;
-        }
-
-        if ($node->getArgs() === []) {
+        if ($this->shouldSkipCall($node)) {
             return null;
         }
 
         $hasChanged = false;
-
-        if (! $this->hasStringOrNumberArguments($node)) {
-            return null;
-        }
-
         $callParameterTypes = $this->methodParametersAndReturnTypesResolver->resolveCallParameterTypes($node);
 
         foreach ($node->getArgs() as $key => $arg) {
@@ -123,6 +111,9 @@ public function refactor(Node $node): ?Node
                 continue;
             }
 
+            // remove null
+            $knownParameterType = TypeCombinator::removeNull($knownParameterType);
+
             if ($knownParameterType instanceof StringType && $arg->value instanceof Int_) {
                 $arg->value = new String_((string) $arg->value->value);
                 $hasChanged = true;
@@ -141,6 +132,23 @@ public function refactor(Node $node): ?Node
         return $node;
     }
 
+    private function shouldSkipCall(StaticCall|MethodCall $call): bool
+    {
+        if (! $this->testsNodeAnalyzer->isInTestClass($call)) {
+            return true;
+        }
+
+        if ($call->isFirstClassCallable()) {
+            return true;
+        }
+
+        if ($call->getArgs() === []) {
+            return true;
+        }
+
+        return ! $this->hasStringOrNumberArguments($call);
+    }
+
     private function hasStringOrNumberArguments(StaticCall|MethodCall $call): bool
     {
         foreach ($call->getArgs() as $arg) {