make addVisitor() immutable

Author: TomasVotruba

phpstan.neon

@@ -385,15 +385,8 @@ parameters:
             path: src/PhpParser/NodeTraverser/AbstractImmutableNodeTraverser.php
         -
             identifier: offsetAccess.nonArray
-<<<<<<< HEAD
             path: src/PhpParser/NodeTraverser/AbstractImmutableNodeTraverser.php
-        -
-            message: '#Property PhpParser\\Node\\Stmt\\ClassMethod\:\:\$stmts \(array<PhpParser\\Node\\Stmt>\|null\) does not accept array<PhpParser\\Node>#'
-            path: scripts/create-immutable-node-visitor.php
-=======
-            path: src/PhpParser/NodeTraverser/RectorNodeTraverser.php
 
->>>>>>> e44afd95ee (remove not needed script)
         -
             message: '#Property Rector\\PhpParser\\NodeTraverser\\AbstractImmutableNodeTraverser\:\:\$visitors \(list<PhpParser\\NodeVisitor>\) does not accept array<int\|string, PhpParser\\NodeVisitor>#'
             path: src/PhpParser/NodeTraverser/AbstractImmutableNodeTraverser.php

src/PhpParser/NodeTraverser/AbstractImmutableNodeTraverser.php

@@ -34,14 +34,9 @@ public function __construct(NodeVisitor ...$visitors)
         $this->visitors = $visitors;
     }
 
-    /**
-     * Adds a visitor.
-     *
-     * @param NodeVisitor $visitor Visitor to add
-     */
     public function addVisitor(NodeVisitor $visitor): void
     {
-        $this->visitors[] = $visitor;
+        throw new ShouldNotHappenException('The immutable node traverser does not support adding visitors.');
     }
 
     public function removeVisitor(NodeVisitor $visitor): void

src/PhpParser/NodeTraverser/RectorNodeTraverser.php

@@ -63,6 +63,8 @@ public function refreshPhpRectors(array $rectors): void
         $this->visitorsPerNodeClass = [];
 
         $this->areNodeVisitorsPrepared = false;
+
+        $this->prepareNodeVisitors();
     }
 
     /**

tests/PhpParser/NodeTraverser/RectorNodeTraverserTest.php

@@ -48,7 +48,7 @@ public function testGetVisitorsForNodeWhenNoVisitorsAvailable(): void
     public function testGetVisitorsForNodeWhenNoVisitorsMatch(): void
     {
         $class = new Class_('test');
-        $this->rectorNodeTraverser->addVisitor($this->ruleUsingFunctionRector);
+        $this->rectorNodeTraverser->refreshPhpRectors([$this->ruleUsingFunctionRector]);
 
         $visitors = $this->rectorNodeTraverser->getVisitorsForNode($class);
 
@@ -58,8 +58,10 @@ public function testGetVisitorsForNodeWhenNoVisitorsMatch(): void
     public function testGetVisitorsForNodeWhenSomeVisitorsMatch(): void
     {
         $class = new Class_('test');
-        $this->rectorNodeTraverser->addVisitor($this->ruleUsingFunctionRector);
-        $this->rectorNodeTraverser->addVisitor($this->ruleUsingClassRector);
+        $this->rectorNodeTraverser->refreshPhpRectors([
+            $this->ruleUsingFunctionRector,
+            $this->ruleUsingClassRector,
+        ]);
 
         $visitors = $this->rectorNodeTraverser->getVisitorsForNode($class);
 
@@ -69,8 +71,10 @@ public function testGetVisitorsForNodeWhenSomeVisitorsMatch(): void
     public function testGetVisitorsForNodeWhenAllVisitorsMatch(): void
     {
         $class = new Class_('test');
-        $this->rectorNodeTraverser->addVisitor($this->ruleUsingClassRector);
-        $this->rectorNodeTraverser->addVisitor($this->ruleUsingClassLikeRector);
+        $this->rectorNodeTraverser->refreshPhpRectors([
+            $this->ruleUsingClassRector,
+            $this->ruleUsingClassLikeRector,
+        ]);
 
         $visitors = $this->rectorNodeTraverser->getVisitorsForNode($class);