symfony
diff --git a/‎Tests/ExpressionLanguageTest.php
Copy file name to clipboardExpand all lines: Tests/ExpressionLanguageTest.php
+7Lines changed: 7 additions & 0 deletions b/‎Tests/ExpressionLanguageTest.php
Copy file name to clipboardExpand all lines: Tests/ExpressionLanguageTest.php
+7Lines changed: 7 additions & 0 deletions
diff --git a/‎Tests/Node/BinaryNodeTest.php
Copy file name to clipboardExpand all lines: Tests/Node/BinaryNodeTest.php
+20Lines changed: 20 additions & 0 deletions b/‎Tests/Node/BinaryNodeTest.php
Copy file name to clipboardExpand all lines: Tests/Node/BinaryNodeTest.php
+20Lines changed: 20 additions & 0 deletions
diff --git a/‎Tests/Node/NodeTest.php
Copy file name to clipboardExpand all lines: Tests/Node/NodeTest.php
+30Lines changed: 30 additions & 0 deletions b/‎Tests/Node/NodeTest.php
Copy file name to clipboardExpand all lines: Tests/Node/NodeTest.php
+30Lines changed: 30 additions & 0 deletions
diff --git a/‎Tests/ParserTest.php
Copy file name to clipboardExpand all lines: Tests/ParserTest.php
+11Lines changed: 11 additions & 0 deletions b/‎Tests/ParserTest.php
Copy file name to clipboardExpand all lines: Tests/ParserTest.php
+11Lines changed: 11 additions & 0 deletions
@@ -287,4 +287,11 @@ function (ExpressionLanguage $el) {
             ],
         ];
     }
+
+    public function testParseAlreadyParsedExpressionReturnsSameObject()
+    {
+        $el = new ExpressionLanguage();
+        $parsed = $el->parse('1 + 1', []);
+        $this->assertSame($parsed, $el->parse($parsed, []));
+    }
 }
@@ -207,4 +207,24 @@ public function testCompileMatchesWithInvalidRegexpAsExpression()
         $node->compile($compiler);
         eval('$regexp = "this is not a regexp"; '.$compiler->getSource().';');
     }
+
+    public function testDivisionByZero()
+    {
+        $node = new BinaryNode('/', new ConstantNode(1), new ConstantNode(0));
+
+        $this->expectException(\DivisionByZeroError::class);
+        $this->expectExceptionMessage('Division by zero.');
+
+        $node->evaluate([], []);
+    }
+
+    public function testModuloByZero()
+    {
+        $node = new BinaryNode('%', new ConstantNode(1), new ConstantNode(0));
+
+        $this->expectException(\DivisionByZeroError::class);
+        $this->expectExceptionMessage('Modulo by zero.');
+
+        $node->evaluate([], []);
+    }
 }
@@ -12,6 +12,7 @@
 namespace Symfony\Component\ExpressionLanguage\Tests\Node;
 
 use PHPUnit\Framework\TestCase;
+use Symfony\Component\ExpressionLanguage\Compiler;
 use Symfony\Component\ExpressionLanguage\Node\ConstantNode;
 use Symfony\Component\ExpressionLanguage\Node\Node;
 
@@ -38,4 +39,33 @@ public function testSerialization()
 
         $this->assertEquals($node, $unserializedNode);
     }
+
+    public function testCompileActuallyCompilesAllNodes()
+    {
+        $nodes = [];
+        foreach (range(1, 10) as $ignored) {
+            $node = $this->createMock(Node::class);
+            $node->expects($this->once())->method('compile');
+
+            $nodes[] = $node;
+        }
+
+        $node = new Node($nodes);
+        $node->compile($this->createMock(Compiler::class));
+    }
+
+    public function testEvaluateActuallyEvaluatesAllNodes()
+    {
+        $nodes = [];
+        foreach (range(1, 3) as $i) {
+            $node = $this->createMock(Node::class);
+            $node->expects($this->once())->method('evaluate')
+                ->willReturn($i);
+
+            $nodes[] = $node;
+        }
+
+        $node = new Node($nodes);
+        $this->assertSame([1, 2, 3], $node->evaluate([], []));
+    }
 }
@@ -37,6 +37,17 @@ public function testParseWithZeroInNames()
         $parser->parse($lexer->tokenize('foo'), [0]);
     }
 
+    public function testParsePrimaryExpressionWithUnknownFunctionThrows()
+    {
+        $parser = new Parser([]);
+        $stream = (new Lexer())->tokenize('foo()');
+
+        $this->expectException(SyntaxError::class);
+        $this->expectExceptionMessage('The function "foo" does not exist around position 1 for expression `foo()`.');
+
+        $parser->parse($stream);
+    }
+
     /**
      * @dataProvider getParseData
      */
Original file line number	Diff line number	Diff line change
`@@ -287,4 +287,11 @@ function (ExpressionLanguage $el) {`
`287`	`287`	`],`
`288`	`288`	`];`
`289`	`289`	`}`
	`290`	`+`
	`291`	`+ public function testParseAlreadyParsedExpressionReturnsSameObject()`
	`292`	`+ {`
	`293`	`+ $el = new ExpressionLanguage();`
	`294`	`+ $parsed = $el->parse('1 + 1', []);`
	`295`	`+ $this->assertSame($parsed, $el->parse($parsed, []));`
	`296`	`+ }`
`290`	`297`	`}`