Updated PHPUnit and PHPStan dependencies

.travis.yml

@@ -76,7 +76,7 @@ jobs:
       php: 7.1
       env: DEPENDENCIES=""
       before_script:
-        - travis_retry composer require --dev --prefer-dist --prefer-stable phpstan/phpstan:^0.8.3
+        - travis_retry composer require --dev --prefer-dist --prefer-stable phpstan/phpstan:^0.9.2
       script: vendor/bin/phpstan analyse -l 5 -c phpstan.neon src
 
     - stage: Run benchmarks

composer.json

@@ -33,7 +33,7 @@
 
     ],
     "require-dev": {
-        "phpunit/phpunit": "^6.3.0"
+        "phpunit/phpunit": "^7.0.0"
     },
     "autoload": {
         "psr-4": {

src/NodeCompiler/CompileNodeToValue.php

@@ -130,27 +130,27 @@ private function compileConstFetch(Node\Expr\ConstFetch $constNode)
      */
     private function compileClassConstFetch(Node\Expr\ClassConstFetch $node, CompilerContext $context)
     {
-        /** @var Node\Name $node->class */
+        /** @var string $nodeName */
+        $nodeName  = $node->name;
         $className = $node->class->toString();
 
-        if ($node->name === 'class') {
+        if ($nodeName === 'class') {
             return $className;
         }
 
         /** @var ReflectionClass|null $classInfo */
         $classInfo = null;
 
         if ($className === 'self' || $className === 'static') {
-            $classInfo = $this->getConstantDeclaringClass($node->name, $context->getSelf());
+            $classInfo = $this->getConstantDeclaringClass($nodeName, $context->getSelf());
         }
 
         if ($classInfo === null) {
             /** @var ReflectionClass $classInfo */
             $classInfo = $context->getReflector()->reflect($className);
         }
 
-        /** @var string $node->name */
-        $reflectionConstant = $classInfo->getReflectionConstant($node->name);
+        $reflectionConstant = $classInfo->getReflectionConstant($nodeName);
 
         return $this->__invoke(
             $reflectionConstant->getAst()->consts[$reflectionConstant->getPositionInAst()]->value,

src/Reflection/ReflectionClass.php

@@ -259,9 +259,12 @@ function (ClassMethod $methodNode) : ReflectionMethod {
             ...array_map(
                 function (ReflectionClass $trait) : array {
                     return array_map(function (ReflectionMethod $method) use ($trait) : ReflectionMethod {
+                        /** @var ClassMethod $methodAst */
+                        $methodAst = $method->getAst();
+
                         return ReflectionMethod::createFromNode(
                             $this->reflector,
-                            $method->getAst(),
+                            $methodAst,
                             $this->declaringNamespace,
                             $trait,
                             $this
@@ -958,6 +961,7 @@ public function getTraitAliases() : array
             $adaptations = $traitUsage->adaptations;
 
             foreach ($adaptations as $adaptation) {
+                /** @var Node\Name|null $usedTrait */
                 $usedTrait = $adaptation->trait;
                 if ($usedTrait === null) {
                     $usedTrait = $traitNames[0];

src/Reflection/ReflectionFunctionAbstract.php

@@ -318,7 +318,9 @@ private function nodeIsOrContainsYield(Node $node) : bool
             return true;
         }
 
-        foreach ($node as $nodeProperty) {
+        foreach ($node->getSubNodeNames() as $nodeName) {
+            $nodeProperty = $node->$nodeName;
+
             if ($nodeProperty instanceof Node && $this->nodeIsOrContainsYield($nodeProperty)) {
                 return true;
             }
@@ -379,7 +381,7 @@ public function getEndColumn() : int
      */
     public function returnsReference() : bool
     {
-        return (bool) $this->node->byRef;
+        return $this->node->byRef;
     }
 
     /**
@@ -503,7 +505,10 @@ public function setBodyFromClosure(Closure $newBody) : void
             new Identifier(self::CLOSURE_NAME, new IdentifierType(IdentifierType::IDENTIFIER_FUNCTION))
         );
 
-        $this->node->stmts = $closureReflection->getNode()->stmts;
+        /** @var Node\Stmt\Function_ $functionNode */
+        $functionNode = $closureReflection->getNode();
+
+        $this->node->stmts = $functionNode->stmts;
     }
 
     /**

src/Reflection/ReflectionParameter.php

@@ -131,7 +131,7 @@ public static function createFromClosure(Closure $closure, string $parameterName
      *  - ['foo']
      *  - [function () {}]
      *
-     * @param string[]|string|\Closure $spec
+     * @param object[]|string[]|string|\Closure $spec
      * @throws \Exception
      * @throws \InvalidArgumentException
      */
@@ -196,8 +196,9 @@ private function parseDefaultValueNode() : void
             $className = $defaultValueNode->class->toString();
 
             if ($className === 'self' || $className === 'static') {
-                /** @var string $defaultValueNode->name */
-                $className = $this->findParentClassDeclaringConstant($defaultValueNode->name);
+                /** @var string $constantName */
+                $constantName = $defaultValueNode->name;
+                $className    = $this->findParentClassDeclaringConstant($constantName);
             }
 
             $this->isDefaultValueConstant = true;

src/Reflector/ClassReflector.php

@@ -24,13 +24,14 @@ public function __construct(SourceLocator $sourceLocator)
     /**
      * Create a ReflectionClass for the specified $className.
      *
-     * @return \Roave\BetterReflection\Reflection\ReflectionClass|Reflection
+     * @return \Roave\BetterReflection\Reflection\ReflectionClass
      * @throws \Roave\BetterReflection\Reflector\Exception\IdentifierNotFound
      */
     public function reflect(string $className) : Reflection
     {
         $identifier = new Identifier($className, new IdentifierType(IdentifierType::IDENTIFIER_CLASS));
 
+        /** @var \Roave\BetterReflection\Reflection\ReflectionClass|null $classInfo */
         $classInfo = $this->sourceLocator->locateIdentifier($this, $identifier);
 
         if ($classInfo === null) {

src/Reflector/FunctionReflector.php

@@ -30,13 +30,14 @@ public function __construct(SourceLocator $sourceLocator, ClassReflector $classR
     /**
      * Create a ReflectionFunction for the specified $functionName.
      *
-     * @return \Roave\BetterReflection\Reflection\Reflection|\Roave\BetterReflection\Reflection\ReflectionFunction
+     * @return \Roave\BetterReflection\Reflection\ReflectionFunction
      * @throws \Roave\BetterReflection\Reflector\Exception\IdentifierNotFound
      */
     public function reflect(string $functionName) : Reflection
     {
         $identifier = new Identifier($functionName, new IdentifierType(IdentifierType::IDENTIFIER_FUNCTION));
 
+        /** @var \Roave\BetterReflection\Reflection\ReflectionFunction|null $functionInfo */
         $functionInfo = $this->sourceLocator->locateIdentifier($this->classReflector, $identifier);
 
         if ($functionInfo === null) {

test/unit/TestListener.php

@@ -6,17 +6,20 @@
 
 use ClassWithExplicitGlobalNamespace;
 use ClassWithNoNamespace;
-use PHPUnit\Framework\BaseTestListener;
 use PHPUnit\Framework\Test;
 use PHPUnit\Framework\TestCase;
+use PHPUnit\Framework\TestListener as BaseTestListener;
+use PHPUnit\Framework\TestListenerDefaultImplementation;
 use PHPUnit\Framework\TestSuite;
 use Roave\BetterReflectionTest\Fixture\ExampleClass;
 use Roave\BetterReflectionTest\Fixture\Methods;
 use Roave\BetterReflectionTest\FixtureOther\AnotherClass;
 use function class_exists;
 
-class TestListener extends BaseTestListener
+class TestListener implements BaseTestListener
 {
+    use TestListenerDefaultImplementation;
+
     /**
      * @var \PHPUnit\Framework\TestSuite|null
      */
@@ -50,11 +53,10 @@ class_exists($className, false),
     /**
      * Ensure the fixture classes have not actually been loaded (where applicable)
      *
-     * @param float $time
      * @throws \PHPUnit\Framework\AssertionFailedError
      * @phpcsSuppress SlevomatCodingStandard.TypeHints.TypeHintDeclaration.MissingParameterTypeHint
      */
-    public function endTest(Test $test, $time) : void
+    public function endTest(Test $test, float $time) : void
     {
         // Only test PHPUnit tests (i.e. no .phpt tests or anything else unexpected)
         if (! ($test instanceof TestCase)) {