Merge pull request #1136: apply Rector on tests files

spiral · Sep 5, 2024 · 141fa8f · 141fa8f
1 parent a5e0766
commit 141fa8f
Show file tree

Hide file tree

Showing 6 changed files with 2 additions and 26 deletions.
diff --git a/.gitignore b/.gitignore
@@ -17,3 +17,4 @@ Thumbs.db
 clover.xml
 .env
 builds
+tests/runtime/
diff --git a/tests/ClassNode/ConflictResolver/ConflictResolverTest.php b/tests/ClassNode/ConflictResolver/ConflictResolverTest.php
@@ -115,10 +115,7 @@ public function testDuplicateProperty(): void
     }
 
     /**
-     * @param string $filename
-     * @param array  $dependencies
      *
-     * @return ClassNode
      * @throws \Throwable
      */
     private function getDefinition(string $filename, array $dependencies): ClassNode
@@ -127,7 +124,6 @@ private function getDefinition(string $filename, array $dependencies): ClassNode
     }
 
     /**
-     * @return NodeExtractor
      * @throws \Throwable
      */
     private function getExtractor(): NodeExtractor

diff --git a/tests/InjectorTest.php b/tests/InjectorTest.php
@@ -273,10 +273,6 @@ public function testParentConstructorParamsTypeDefinition(): void
     }
 
     /**
-     * @param string $filename
-     * @param array  $dependencies
-     *
-     * @return ClassNode
      * @throws \ReflectionException
      * @throws ClassNotDeclaredException
      */

diff --git a/tests/TraitTest.php b/tests/TraitTest.php
@@ -57,9 +57,7 @@ public function testOK(): void
         $c->bindSingleton(TestClass::class, $t);
         $p->bindProperty('testClass', TestClass::class);
 
-        $r = ContainerScope::runScope($c, static function () use ($t) {
-            return $t->getTest();
-        });
+        $r = ContainerScope::runScope($c, static fn() => $t->getTest());
 
         $this->assertSame($t, $r);
     }

diff --git a/tests/Traverse/ConstructorParamsVisitor.php b/tests/Traverse/ConstructorParamsVisitor.php
@@ -43,9 +43,6 @@ public function leaveNode(Node $node): void
         }
     }
 
-    /**
-     * @return array
-     */
     public function getParams(): array
     {
         return $this->params;

diff --git a/tests/Traverse/Extractor.php b/tests/Traverse/Extractor.php
@@ -19,19 +19,11 @@ public function __construct(Parser $parser = null)
         $this->parser = $parser ?? (new ParserFactory())->create(ParserFactory::ONLY_PHP7);
     }
 
-    /**
-     * @param string $filename
-     * @return array
-     */
     public function extractFromFilename(string $filename): array
     {
         return $this->extractFromString(file_get_contents($filename));
     }
 
-    /**
-     * @param string $code
-     * @return array
-     */
     public function extractFromString(string $code): array
     {
         $params = new ConstructorParamsVisitor();
@@ -40,10 +32,6 @@ public function extractFromString(string $code): array
         return $params->getParams();
     }
 
-    /**
-     * @param string      $code
-     * @param NodeVisitor ...$visitors
-     */
     private function traverse(string $code, NodeVisitor ...$visitors): void
     {
         $tr = new NodeTraverser();