123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344 |
- <?php declare(strict_types=1);
- namespace PhpParser;
- use PhpParser\Node\Expr;
- use PhpParser\Node\Scalar\String_;
- class NodeTraverserTest extends \PHPUnit\Framework\TestCase
- {
- public function testNonModifying() {
- $str1Node = new String_('Foo');
- $str2Node = new String_('Bar');
- $echoNode = new Node\Stmt\Echo_([$str1Node, $str2Node]);
- $stmts = [$echoNode];
- $visitor = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor->expects($this->at(0))->method('beforeTraverse')->with($stmts);
- $visitor->expects($this->at(1))->method('enterNode')->with($echoNode);
- $visitor->expects($this->at(2))->method('enterNode')->with($str1Node);
- $visitor->expects($this->at(3))->method('leaveNode')->with($str1Node);
- $visitor->expects($this->at(4))->method('enterNode')->with($str2Node);
- $visitor->expects($this->at(5))->method('leaveNode')->with($str2Node);
- $visitor->expects($this->at(6))->method('leaveNode')->with($echoNode);
- $visitor->expects($this->at(7))->method('afterTraverse')->with($stmts);
- $traverser = new NodeTraverser;
- $traverser->addVisitor($visitor);
- $this->assertEquals($stmts, $traverser->traverse($stmts));
- }
- public function testModifying() {
- $str1Node = new String_('Foo');
- $str2Node = new String_('Bar');
- $printNode = new Expr\Print_($str1Node);
- // first visitor changes the node, second verifies the change
- $visitor1 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor2 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- // replace empty statements with string1 node
- $visitor1->expects($this->at(0))->method('beforeTraverse')->with([])
- ->willReturn([$str1Node]);
- $visitor2->expects($this->at(0))->method('beforeTraverse')->with([$str1Node]);
- // replace string1 node with print node
- $visitor1->expects($this->at(1))->method('enterNode')->with($str1Node)
- ->willReturn($printNode);
- $visitor2->expects($this->at(1))->method('enterNode')->with($printNode);
- // replace string1 node with string2 node
- $visitor1->expects($this->at(2))->method('enterNode')->with($str1Node)
- ->willReturn($str2Node);
- $visitor2->expects($this->at(2))->method('enterNode')->with($str2Node);
- // replace string2 node with string1 node again
- $visitor1->expects($this->at(3))->method('leaveNode')->with($str2Node)
- ->willReturn($str1Node);
- $visitor2->expects($this->at(3))->method('leaveNode')->with($str1Node);
- // replace print node with string1 node again
- $visitor1->expects($this->at(4))->method('leaveNode')->with($printNode)
- ->willReturn($str1Node);
- $visitor2->expects($this->at(4))->method('leaveNode')->with($str1Node);
- // replace string1 node with empty statements again
- $visitor1->expects($this->at(5))->method('afterTraverse')->with([$str1Node])
- ->willReturn([]);
- $visitor2->expects($this->at(5))->method('afterTraverse')->with([]);
- $traverser = new NodeTraverser;
- $traverser->addVisitor($visitor1);
- $traverser->addVisitor($visitor2);
- // as all operations are reversed we end where we start
- $this->assertEquals([], $traverser->traverse([]));
- }
- public function testRemove() {
- $str1Node = new String_('Foo');
- $str2Node = new String_('Bar');
- $visitor = $this->getMockBuilder(NodeVisitor::class)->getMock();
- // remove the string1 node, leave the string2 node
- $visitor->expects($this->at(2))->method('leaveNode')->with($str1Node)
- ->willReturn(NodeTraverser::REMOVE_NODE);
- $traverser = new NodeTraverser;
- $traverser->addVisitor($visitor);
- $this->assertEquals([$str2Node], $traverser->traverse([$str1Node, $str2Node]));
- }
- public function testMerge() {
- $strStart = new String_('Start');
- $strMiddle = new String_('End');
- $strEnd = new String_('Middle');
- $strR1 = new String_('Replacement 1');
- $strR2 = new String_('Replacement 2');
- $visitor = $this->getMockBuilder(NodeVisitor::class)->getMock();
- // replace strMiddle with strR1 and strR2 by merge
- $visitor->expects($this->at(4))->method('leaveNode')->with($strMiddle)
- ->willReturn([$strR1, $strR2]);
- $traverser = new NodeTraverser;
- $traverser->addVisitor($visitor);
- $this->assertEquals(
- [$strStart, $strR1, $strR2, $strEnd],
- $traverser->traverse([$strStart, $strMiddle, $strEnd])
- );
- }
- public function testInvalidDeepArray() {
- $this->expectException(\LogicException::class);
- $this->expectExceptionMessage('Invalid node structure: Contains nested arrays');
- $strNode = new String_('Foo');
- $stmts = [[[$strNode]]];
- $traverser = new NodeTraverser;
- $this->assertEquals($stmts, $traverser->traverse($stmts));
- }
- public function testDontTraverseChildren() {
- $strNode = new String_('str');
- $printNode = new Expr\Print_($strNode);
- $varNode = new Expr\Variable('foo');
- $mulNode = new Expr\BinaryOp\Mul($varNode, $varNode);
- $negNode = new Expr\UnaryMinus($mulNode);
- $stmts = [$printNode, $negNode];
- $visitor1 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor2 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor1->expects($this->at(1))->method('enterNode')->with($printNode)
- ->willReturn(NodeTraverser::DONT_TRAVERSE_CHILDREN);
- $visitor2->expects($this->at(1))->method('enterNode')->with($printNode);
- $visitor1->expects($this->at(2))->method('leaveNode')->with($printNode);
- $visitor2->expects($this->at(2))->method('leaveNode')->with($printNode);
- $visitor1->expects($this->at(3))->method('enterNode')->with($negNode);
- $visitor2->expects($this->at(3))->method('enterNode')->with($negNode);
- $visitor1->expects($this->at(4))->method('enterNode')->with($mulNode);
- $visitor2->expects($this->at(4))->method('enterNode')->with($mulNode)
- ->willReturn(NodeTraverser::DONT_TRAVERSE_CHILDREN);
- $visitor1->expects($this->at(5))->method('leaveNode')->with($mulNode);
- $visitor2->expects($this->at(5))->method('leaveNode')->with($mulNode);
- $visitor1->expects($this->at(6))->method('leaveNode')->with($negNode);
- $visitor2->expects($this->at(6))->method('leaveNode')->with($negNode);
- $traverser = new NodeTraverser;
- $traverser->addVisitor($visitor1);
- $traverser->addVisitor($visitor2);
- $this->assertEquals($stmts, $traverser->traverse($stmts));
- }
- public function testDontTraverseCurrentAndChildren() {
- // print 'str'; -($foo * $foo);
- $strNode = new String_('str');
- $printNode = new Expr\Print_($strNode);
- $varNode = new Expr\Variable('foo');
- $mulNode = new Expr\BinaryOp\Mul($varNode, $varNode);
- $divNode = new Expr\BinaryOp\Div($varNode, $varNode);
- $negNode = new Expr\UnaryMinus($mulNode);
- $stmts = [$printNode, $negNode];
- $visitor1 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor2 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor1->expects($this->at(1))->method('enterNode')->with($printNode)
- ->willReturn(NodeTraverser::DONT_TRAVERSE_CURRENT_AND_CHILDREN);
- $visitor1->expects($this->at(2))->method('leaveNode')->with($printNode);
- $visitor1->expects($this->at(3))->method('enterNode')->with($negNode);
- $visitor2->expects($this->at(1))->method('enterNode')->with($negNode);
- $visitor1->expects($this->at(4))->method('enterNode')->with($mulNode)
- ->willReturn(NodeTraverser::DONT_TRAVERSE_CURRENT_AND_CHILDREN);
- $visitor1->expects($this->at(5))->method('leaveNode')->with($mulNode)->willReturn($divNode);
- $visitor1->expects($this->at(6))->method('leaveNode')->with($negNode);
- $visitor2->expects($this->at(2))->method('leaveNode')->with($negNode);
- $traverser = new NodeTraverser;
- $traverser->addVisitor($visitor1);
- $traverser->addVisitor($visitor2);
- $resultStmts = $traverser->traverse($stmts);
- $this->assertInstanceOf(Expr\BinaryOp\Div::class, $resultStmts[1]->expr);
- }
- public function testStopTraversal() {
- $varNode1 = new Expr\Variable('a');
- $varNode2 = new Expr\Variable('b');
- $varNode3 = new Expr\Variable('c');
- $mulNode = new Expr\BinaryOp\Mul($varNode1, $varNode2);
- $printNode = new Expr\Print_($varNode3);
- $stmts = [$mulNode, $printNode];
- // From enterNode() with array parent
- $visitor = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor->expects($this->at(1))->method('enterNode')->with($mulNode)
- ->willReturn(NodeTraverser::STOP_TRAVERSAL);
- $visitor->expects($this->at(2))->method('afterTraverse');
- $traverser = new NodeTraverser;
- $traverser->addVisitor($visitor);
- $this->assertEquals($stmts, $traverser->traverse($stmts));
- // From enterNode with Node parent
- $visitor = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor->expects($this->at(2))->method('enterNode')->with($varNode1)
- ->willReturn(NodeTraverser::STOP_TRAVERSAL);
- $visitor->expects($this->at(3))->method('afterTraverse');
- $traverser = new NodeTraverser;
- $traverser->addVisitor($visitor);
- $this->assertEquals($stmts, $traverser->traverse($stmts));
- // From leaveNode with Node parent
- $visitor = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor->expects($this->at(3))->method('leaveNode')->with($varNode1)
- ->willReturn(NodeTraverser::STOP_TRAVERSAL);
- $visitor->expects($this->at(4))->method('afterTraverse');
- $traverser = new NodeTraverser;
- $traverser->addVisitor($visitor);
- $this->assertEquals($stmts, $traverser->traverse($stmts));
- // From leaveNode with array parent
- $visitor = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor->expects($this->at(6))->method('leaveNode')->with($mulNode)
- ->willReturn(NodeTraverser::STOP_TRAVERSAL);
- $visitor->expects($this->at(7))->method('afterTraverse');
- $traverser = new NodeTraverser;
- $traverser->addVisitor($visitor);
- $this->assertEquals($stmts, $traverser->traverse($stmts));
- // Check that pending array modifications are still carried out
- $visitor = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor->expects($this->at(6))->method('leaveNode')->with($mulNode)
- ->willReturn(NodeTraverser::REMOVE_NODE);
- $visitor->expects($this->at(7))->method('enterNode')->with($printNode)
- ->willReturn(NodeTraverser::STOP_TRAVERSAL);
- $visitor->expects($this->at(8))->method('afterTraverse');
- $traverser = new NodeTraverser;
- $traverser->addVisitor($visitor);
- $this->assertEquals([$printNode], $traverser->traverse($stmts));
- }
- public function testRemovingVisitor() {
- $visitor1 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor2 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor3 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $traverser = new NodeTraverser;
- $traverser->addVisitor($visitor1);
- $traverser->addVisitor($visitor2);
- $traverser->addVisitor($visitor3);
- $preExpected = [$visitor1, $visitor2, $visitor3];
- $this->assertAttributeSame($preExpected, 'visitors', $traverser, 'The appropriate visitors have not been added');
- $traverser->removeVisitor($visitor2);
- $postExpected = [0 => $visitor1, 2 => $visitor3];
- $this->assertAttributeSame($postExpected, 'visitors', $traverser, 'The appropriate visitors are not present after removal');
- }
- public function testNoCloneNodes() {
- $stmts = [new Node\Stmt\Echo_([new String_('Foo'), new String_('Bar')])];
- $traverser = new NodeTraverser;
- $this->assertSame($stmts, $traverser->traverse($stmts));
- }
- /**
- * @dataProvider provideTestInvalidReturn
- */
- public function testInvalidReturn($visitor, $message) {
- $this->expectException(\LogicException::class);
- $this->expectExceptionMessage($message);
- $stmts = [new Node\Stmt\Expression(new Node\Scalar\LNumber(42))];
- $traverser = new NodeTraverser();
- $traverser->addVisitor($visitor);
- $traverser->traverse($stmts);
- }
- public function provideTestInvalidReturn() {
- $visitor1 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor1->expects($this->at(1))->method('enterNode')
- ->willReturn('foobar');
- $visitor2 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor2->expects($this->at(2))->method('enterNode')
- ->willReturn('foobar');
- $visitor3 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor3->expects($this->at(3))->method('leaveNode')
- ->willReturn('foobar');
- $visitor4 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor4->expects($this->at(4))->method('leaveNode')
- ->willReturn('foobar');
- $visitor5 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor5->expects($this->at(3))->method('leaveNode')
- ->willReturn([new Node\Scalar\DNumber(42.0)]);
- $visitor6 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor6->expects($this->at(4))->method('leaveNode')
- ->willReturn(false);
- $visitor7 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor7->expects($this->at(1))->method('enterNode')
- ->willReturn(new Node\Scalar\LNumber(42));
- $visitor8 = $this->getMockBuilder(NodeVisitor::class)->getMock();
- $visitor8->expects($this->at(2))->method('enterNode')
- ->willReturn(new Node\Stmt\Return_());
- return [
- [$visitor1, 'enterNode() returned invalid value of type string'],
- [$visitor2, 'enterNode() returned invalid value of type string'],
- [$visitor3, 'leaveNode() returned invalid value of type string'],
- [$visitor4, 'leaveNode() returned invalid value of type string'],
- [$visitor5, 'leaveNode() may only return an array if the parent structure is an array'],
- [$visitor6, 'bool(false) return from leaveNode() no longer supported. Return NodeTraverser::REMOVE_NODE instead'],
- [$visitor7, 'Trying to replace statement (Stmt_Expression) with expression (Scalar_LNumber). Are you missing a Stmt_Expression wrapper?'],
- [$visitor8, 'Trying to replace expression (Scalar_LNumber) with statement (Stmt_Return)'],
- ];
- }
- }
|