Fix incorrect behavior for empty return types, related to #5

goaop · Feb 21, 2016 · eb05bb9 · eb05bb9
1 parent 872c383
commit eb05bb9
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()
+                    );
                 }
             }
         }