Separate chain calls (#36)

yiisoft · May 22, 2022 · 50b2e66 · 50b2e66
1 parent e858d96
commit 50b2e66
Show file tree

Hide file tree

Showing 5 changed files with 30 additions and 8 deletions.
diff --git a/src/ParameterDefinition.php b/src/ParameterDefinition.php
@@ -78,7 +78,9 @@ public function resolve(ContainerInterface $container)
                 // If type name is "self", it means that called class and
                 // $parameter->getDeclaringClass() returned instance of `ReflectionClass`.
                 /** @psalm-suppress PossiblyNullReference */
-                $typeName = $this->parameter->getDeclaringClass()->getName();
+                $typeName = $this->parameter
+                    ->getDeclaringClass()
+                    ->getName();
             }
 
             try {
@@ -185,7 +187,9 @@ private function resolveUnionType(ContainerInterface $container)
                     // If type name is "self", it means that called class and
                     // $parameter->getDeclaringClass() returned instance of `ReflectionClass`.
                     /** @psalm-suppress PossiblyNullReference */
-                    $typeName = $this->parameter->getDeclaringClass()->getName();
+                    $typeName = $this->parameter
+                        ->getDeclaringClass()
+                        ->getName();
                 }
 
                 try {
@@ -265,7 +269,10 @@ private function getCallable(): string
         if ($class !== null) {
             $callable[] = $class->getName();
         }
-        $callable[] = $this->parameter->getDeclaringFunction()->getName() . '()';
+        $callable[] = $this->parameter
+                ->getDeclaringFunction()
+                ->getName() .
+            '()';
 
         return implode('::', $callable);
     }

diff --git a/tests/Php8/Helpers/DefinitionExtractorTest.php b/tests/Php8/Helpers/DefinitionExtractorTest.php
@@ -57,7 +57,10 @@ public function testFromClassWithUnionSelfDependency(): void
     {
         $definition = DefinitionExtractor::fromClassName(UnionSelfDependency::class)['a'];
 
-        $actualType = implode('|', $definition->getReflection()->getType()->getTypes());
+        $actualType = implode('|', $definition
+            ->getReflection()
+            ->getType()
+            ->getTypes());
         $this->assertInstanceOf(ParameterDefinition::class, $definition);
         $this->assertSame('self|' . ColorInterface::class, $actualType);
     }

diff --git a/tests/Php8/ParameterDefinitionTest.php b/tests/Php8/ParameterDefinitionTest.php
@@ -44,7 +44,9 @@ public function testResolveUnionType(): void
     public function testNotInstantiable(): void
     {
         $definition = new ParameterDefinition(
-            (new ReflectionClass(UnionCar::class))->getConstructor()->getParameters()[0]
+            (new ReflectionClass(UnionCar::class))
+                ->getConstructor()
+                ->getParameters()[0]
         );
         $container = new SimpleContainer();
 

diff --git a/tests/Unit/Helpers/DefinitionExtractorTest.php b/tests/Unit/Helpers/DefinitionExtractorTest.php
@@ -145,6 +145,12 @@ public function testFromClassWithSelfDependency(): void
         $definition = DefinitionExtractor::fromClassName(SelfDependency::class)['a'];
 
         $this->assertInstanceOf(ParameterDefinition::class, $definition);
-        $this->assertSame('self', $definition->getReflection()->getType()->getName());
+        $this->assertSame(
+            'self',
+            $definition
+                ->getReflection()
+                ->getType()
+                ->getName(),
+        );
     }
 }
diff --git a/tests/Unit/ParameterDefinitionTest.php b/tests/Unit/ParameterDefinitionTest.php
@@ -120,7 +120,9 @@ public function testResolveWithIncorrectTypeInContainer(): void
     public function testNullableParameterNotInstantiable(): void
     {
         $definition = new ParameterDefinition(
-            (new ReflectionClass(NullableConcreteDependency::class))->getConstructor()->getParameters()[0]
+            (new ReflectionClass(NullableConcreteDependency::class))
+                ->getConstructor()
+                ->getParameters()[0]
         );
         $container = new SimpleContainer();
 
@@ -203,7 +205,9 @@ public function testResolveSelf(): void
     public function testNotInstantiable(): void
     {
         $definition = new ParameterDefinition(
-            (new ReflectionClass(Car::class))->getConstructor()->getParameters()[0]
+            (new ReflectionClass(Car::class))
+                ->getConstructor()
+                ->getParameters()[0]
         );
         $container = new SimpleContainer();