Add workaround for ReflectionMethod::isConstructor() issue described in

#4139 (comment)
sebastianbergmann · Mar 28, 2020 · 477798f · 477798f
1 parent fb668be
commit 477798f
Showing 1 changed file with 17 additions and 1 deletion.
diff --git a/src/Framework/MockObject/Generator.php b/src/Framework/MockObject/Generator.php
@@ -974,7 +974,7 @@ private function generateMockClassDeclaration(array $mockClassName, bool $isInte
 
     private function canMockMethod(\ReflectionMethod $method): bool
     {
-        return !($method->isConstructor() || $method->isFinal() || $method->isPrivate() || $this->isMethodNameBlacklisted($method->getName()));
+        return !($this->isConstructor($method) || $method->isFinal() || $method->isPrivate() || $this->isMethodNameBlacklisted($method->getName()));
     }
 
     private function isMethodNameBlacklisted(string $name): bool
@@ -992,4 +992,20 @@ private function getTemplate(string $template): \Text_Template
 
         return self::$templates[$filename];
     }
+
+    /**
+     * @see https://github.com/sebastianbergmann/phpunit/issues/4139#issuecomment-605409765
+     */
+    private function isConstructor(\ReflectionMethod $method): bool
+    {
+        $methodName = \strtolower($method->getName());
+
+        if ($methodName === '__construct') {
+            return true;
+        }
+
+        $className  = \strtolower($method->getDeclaringClass()->getName());
+
+        return $methodName === $className;
+    }
 }