add fixture (#856)

Author: TomasVotruba

rules-tests/Configs/Rector/Closure/RemoveConstructorAutowireServiceRector/Fixture/named_class.php.inc

@@ -0,0 +1,42 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Symfony\Tests\Configs\Rector\Closure\RemoveConstructorAutowireServiceRector\Fixture;
+
+use Rector\Symfony\Tests\Configs\Rector\Closure\RemoveConstructorAutowireServiceRector\Source\AnotherClassWithoutConstructor;
+use Rector\Symfony\Tests\Configs\Rector\Closure\RemoveConstructorAutowireServiceRector\Source\PassedAsDependency;
+use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
+use function Symfony\Component\DependencyInjection\Loader\Configurator\service;
+
+return static function (ContainerConfigurator $containerConfigurator): void {
+    $services = $containerConfigurator->services();
+
+    $services->defaults()->autowire();
+
+    $services->set('named-class', AnotherClassWithoutConstructor::class)
+        ->arg('$passedAsDependency', service(PassedAsDependency::class));
+};
+
+?>
+-----
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Symfony\Tests\Configs\Rector\Closure\RemoveConstructorAutowireServiceRector\Fixture;
+
+use Rector\Symfony\Tests\Configs\Rector\Closure\RemoveConstructorAutowireServiceRector\Source\AnotherClassWithoutConstructor;
+use Rector\Symfony\Tests\Configs\Rector\Closure\RemoveConstructorAutowireServiceRector\Source\PassedAsDependency;
+use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
+use function Symfony\Component\DependencyInjection\Loader\Configurator\service;
+
+return static function (ContainerConfigurator $containerConfigurator): void {
+    $services = $containerConfigurator->services();
+
+    $services->defaults()->autowire();
+
+    $services->set('named-class', AnotherClassWithoutConstructor::class);
+};
+
+?>

rules/Configs/Rector/Closure/RemoveConstructorAutowireServiceRector.php

@@ -182,13 +182,19 @@ private function matchSetServicesClass(MethodCall $methodCall): ?string
             $methodCall = $methodCall->var;
         }
 
-        /** @var MethodCall $methodCall */
-        $firstArg = $methodCall->getArgs()[0];
-        if (! $firstArg->value instanceof ClassConstFetch) {
+        if (! $methodCall instanceof MethodCall) {
             return null;
         }
 
-        return $this->valueResolver->getValue($firstArg->value);
+        foreach ($methodCall->getArgs() as $arg) {
+            if (! $arg->value instanceof ClassConstFetch) {
+                continue;
+            }
+
+            return $this->valueResolver->getValue($arg->value);
+        }
+
+        return null;
     }
 
     private function isParameterTypeMatchingPassedArgExprClass(