Merge pull request #7 from goaop/fix/empty-return-type

Fix incorrect behavior for empty return types
goaop · Feb 21, 2016 · d8f51e0 · d8f51e0
2 parents 872c383 + eb05bb9
commit d8f51e0
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/src/Traits/ReflectionFunctionLikeTrait.php b/src/Traits/ReflectionFunctionLikeTrait.php
@@ -186,6 +186,8 @@ public function getReturnType()
             $returnType = $returnType->toString();
         } elseif (is_string($returnType)) {
             $isBuiltin = true;
+        } else {
+            return null;
         }
 
         return new ReflectionType($returnType, false, $isBuiltin);

diff --git a/tests/ReflectionFunctionTest.php b/tests/ReflectionFunctionTest.php
@@ -126,6 +126,11 @@ public function testGetReturnTypeMethod()
                     $this->assertSame($originalReturnType->allowsNull(), $parsedReturnType->allowsNull());
                     $this->assertSame($originalReturnType->isBuiltin(), $parsedReturnType->isBuiltin());
                     $this->assertSame($originalReturnType->__toString(), $parsedReturnType->__toString());
+                } else {
+                    $this->assertSame(
+                        $originalRefFunction->getReturnType(),
+                        $refFunction->getReturnType()
+                    );
                 }
             }
         }