Merge pull request #379 from proophsoftware/fix/interface_method_body…

…_ast_is_null Return empty ast for interface method body
Roave · Nov 1, 2017 · 9583638 · 9583638
2 parents bbeaf81 + 578bcbe
commit 9583638
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 1 deletion.
diff --git a/src/Reflection/ReflectionFunctionAbstract.php b/src/Reflection/ReflectionFunctionAbstract.php
@@ -498,7 +498,7 @@ public function __clone()
      */
     public function getBodyAst() : array
     {
-        return $this->node->stmts;
+        return $this->node->stmts ?: [];
     }
 
     /**

diff --git a/test/unit/Fixture/InterfaceWithMethod.php b/test/unit/Fixture/InterfaceWithMethod.php
@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Roave\BetterReflectionTest\Fixture;
+
+interface InterfaceWithMethod
+{
+    public function someMethod();
+}
diff --git a/test/unit/Reflection/ReflectionMethodTest.php b/test/unit/Reflection/ReflectionMethodTest.php
@@ -33,6 +33,7 @@
 use Roave\BetterReflectionTest\Fixture\ClassWithNonStaticMethod;
 use Roave\BetterReflectionTest\Fixture\ClassWithStaticMethod;
 use Roave\BetterReflectionTest\Fixture\ExampleClass;
+use Roave\BetterReflectionTest\Fixture\InterfaceWithMethod;
 use Roave\BetterReflectionTest\Fixture\Methods;
 use Roave\BetterReflectionTest\Fixture\Php4StyleConstructInNamespace;
 use Roave\BetterReflectionTest\Fixture\UpperCaseConstructDestruct;
@@ -615,4 +616,12 @@ public function testInvokeOfObjectMethod() : void
         self::assertSame(103, $methodReflection->invoke($object, 1, 2));
         self::assertSame(107, $methodReflection->invoke($object, 3, 4));
     }
+
+    public function testInterfaceMethodBodyAst() : void
+    {
+        $classInfo  = $this->reflector->reflect(InterfaceWithMethod::class);
+        $methodInfo = $classInfo->getMethod('someMethod');
+
+        self::assertSame([], $methodInfo->getBodyAst());
+    }
 }