Fix static tests for php8 compatibility

Author: karyna-t

dev/tests/static/framework/Magento/TestFramework/Integrity/Library/PhpParser/StaticCalls.php

@@ -82,6 +82,13 @@ protected function getClassByStaticCall($staticCall)
     {
         $step = 1;
         $staticClassParts = [];
+
+        $token = $this->tokens->getTokenCodeByKey($staticCall - $step);
+        if ($token === T_NAME_FULLY_QUALIFIED || $token === T_NAME_QUALIFIED) {
+            return $this->tokens->getTokenValueByKey($staticCall - $step);
+        }
+
+        // PHP 7 compatibility
         while ($this->tokens->getTokenCodeByKey(
             $staticCall - $step
         ) == T_STRING || $this->tokens->getTokenCodeByKey(
@@ -90,6 +97,7 @@ protected function getClassByStaticCall($staticCall)
             $staticClassParts[] = $this->tokens->getTokenValueByKey($staticCall - $step);
             $step++;
         }
+
         return implode(array_reverse($staticClassParts));
     }
 

dev/tests/static/framework/Magento/TestFramework/Integrity/Library/PhpParser/Throws.php

@@ -57,14 +57,19 @@ public function getDependencies(Uses $uses)
             $class = '';
             if ($this->tokens->getTokenCodeByKey($throw + 2) == T_NEW) {
                 $step = 4;
-                while ($this->tokens->getTokenCodeByKey(
-                    $throw + $step
-                ) == T_STRING || $this->tokens->getTokenCodeByKey(
-                    $throw + $step
-                ) == T_NS_SEPARATOR) {
-                    $class .= trim($this->tokens->getTokenValueByKey($throw + $step));
-                    $step++;
+
+                $token = $this->tokens->getTokenCodeByKey($throw + $step);
+                if ($token === T_NAME_FULLY_QUALIFIED || $token === T_NAME_QUALIFIED) {
+                    $class = $this->tokens->getTokenValueByKey($throw + $step);
+                } else {
+                    // PHP 7 compatibility
+                    while ($this->tokens->getTokenCodeByKey($throw + $step) === T_STRING
+                        || $this->tokens->getTokenCodeByKey($throw + $step) === T_NS_SEPARATOR) {
+                        $class .= trim($this->tokens->getTokenValueByKey($throw + $step));
+                        $step++;
+                    }
                 }
+
                 if ($uses->hasUses()) {
                     $class = $uses->getClassNameWithNamespace($class);
                 }

dev/tests/static/testsuite/Magento/Test/Integrity/Di/CompilerTest.php

@@ -127,6 +127,7 @@ private function getPluginBlacklist(): array
             );
             $blacklistItems = [];
             foreach (glob($blacklistFiles) as $fileName) {
+                // phpcs:ignore Magento2.Performance.ForeachArrayMerge
                 $blacklistItems = array_merge(
                     $blacklistItems,
                     file($fileName, FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES)
@@ -243,6 +244,7 @@ protected function _phpClassesDataProvider()
         $allowedFiles = array_keys($classes);
         foreach ($classes as $class) {
             if (!in_array($class, $output)) {
+                // phpcs:ignore Magento2.Performance.ForeachArrayMerge
                 $output = array_merge($output, $this->_buildInheritanceHierarchyTree($class, $allowedFiles));
                 $output = array_unique($output);
             }
@@ -409,7 +411,7 @@ protected function pluginDataProvider()
                     $plugin = $node->attributes->getNamedItem('type')->nodeValue;
                     if (!in_array($plugin, $this->getPluginBlacklist())) {
                         $plugin = \Magento\Framework\App\Utility\Classes::resolveVirtualType($plugin);
-                        $plugins[] = ['plugin' => $plugin, 'intercepted type' => $type];
+                        $plugins[] = [$plugin, $type];
                     }
                 }
             }

dev/tests/static/testsuite/Magento/Test/Integrity/Magento/Framework/Api/ExtensibleInterfacesTest.php

@@ -135,7 +135,7 @@ private function checkSetExtensionAttributes(
             } else {
                 // Get the parameter name via a regular expression capture because the class may
                 // not exist which causes a fatal error
-                preg_match('/\[\s\<\w+?>\s([\w]+)/s', $methodParameters[0]->__toString(), $matches);
+                preg_match('/\[\s\<\w+?>\s([?]?[\w]+)/s', $methodParameters[0]->__toString(), $matches);
                 $isCorrectParameter = false;
                 if (isset($matches[1]) && '\\' . $matches[1] != $extensionInterfaceName) {
                     $isCorrectParameter = true;

dev/tests/static/testsuite/Magento/Test/Integrity/PublicCodeTest.php

@@ -11,6 +11,7 @@
 use PHPUnit\Framework\TestCase;
 use ReflectionClass;
 use ReflectionException;
+use ReflectionMethod;
 use ReflectionParameter;
 
 /**
@@ -111,9 +112,9 @@ public function testAllPHPClassesReferencedFromPublicClassesArePublic($class)
     {
         $nonPublishedClasses = [];
         $reflection = new \ReflectionClass($class);
-        $filter = \ReflectionMethod::IS_PUBLIC;
+        $filter = ReflectionMethod::IS_PUBLIC;
         if ($reflection->isAbstract()) {
-            $filter = $filter | \ReflectionMethod::IS_PROTECTED;
+            $filter = $filter | ReflectionMethod::IS_PROTECTED;
         }
         $methods = $reflection->getMethods($filter);
         foreach ($methods as $method) {
@@ -125,8 +126,11 @@ public function testAllPHPClassesReferencedFromPublicClassesArePublic($class)
              is written on early php 7 when return types are not actively used */
             $returnTypes = [];
             if ($method->hasReturnType()) {
-                if (!$method->getReturnType()->isBuiltin()) {
-                    $returnTypes = [trim($method->getReturnType()->getName(), '?[]')];
+                $methodReturnType = $method->getReturnType();
+                // For PHP 8.0 - ReflectionUnionType doesn't have isBuiltin method.
+                if (method_exists($methodReturnType, 'isBuiltin')
+                    && !$methodReturnType->isBuiltin()) {
+                    $returnTypes = [trim($methodReturnType->getName(), '?[]')];
                 }
             } else {
                 $returnTypes = $this->getReturnTypesFromDocComment($method->getDocComment());
@@ -260,16 +264,20 @@ private function checkReturnValues($class, array $returnTypes, array $nonPublish
 
     /**
      * Check if all method parameters are public
+     *
      * @param string $class
-     * @param \ReflectionMethod $method
+     * @param ReflectionMethod $method
      * @param array $nonPublishedClasses
+     *
      * @return array
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity)
      */
-    private function checkParameters($class, \ReflectionMethod $method, array $nonPublishedClasses)
+    private function checkParameters($class, ReflectionMethod $method, array $nonPublishedClasses)
     {
         /* Ignoring docblocks for argument types */
         foreach ($method->getParameters() as $parameter) {
             if ($parameter->hasType()
+                && method_exists($parameter->getType(), 'isBuiltin')
                 && !$parameter->getType()->isBuiltin()
                 && !$this->isGenerated($parameter->getType()->getName())
             ) {

dev/tests/static/testsuite/Magento/Test/Integrity/Readme/ReadmeTest.php

@@ -57,7 +57,7 @@ private function getDirectories()
         $directories = [];
         foreach ($this->scanList as $dir) {
             if (!$this->isInBlacklist($dir)) {
-                $directories[][$dir] = $dir;
+                $directories[][] = $dir;
             }
         }