CICD-2390: Speed up L2 by run test in batch

Author: kandy

dev/tests/integration/framework/Magento/TestFramework/Event/Transaction.php

@@ -113,6 +113,7 @@ protected function _rollbackTransaction()
             $this->_getConnection()->rollbackTransparentTransaction();
             $this->_isTransactionActive = false;
             $this->_eventManager->fireEvent('rollbackTransaction');
+            $this->_getConnection()->closeConnection();
         }
     }
 

dev/tests/integration/framework/Magento/TestFramework/Workaround/Cleanup/StaticProperties.php

@@ -33,7 +33,6 @@ class StaticProperties
      * @var array
      */
     protected static $_classesToSkip = [
-        'Mage',
         \Magento\Framework\App\ObjectManager::class,
         \Magento\TestFramework\Helper\Bootstrap::class,
         \Magento\TestFramework\Event\Magento::class,
@@ -107,14 +106,31 @@ protected static function _isClassInCleanableFolders($classFile)
         return false; // File is not in an "include" directory
     }
 
+    /**
+     * @var \ReflectionClass[]
+     */
+    static protected $classes = [];
+
+    /**
+     * @param string $class
+     * @return \ReflectionClass
+     */
+    private static function getReflectionClass($class)
+    {
+        if (!isset(self::$classes[$class])) {
+            self::$classes[$class] = new \ReflectionClass($class);
+        }
+        return self::$classes[$class];
+    }
+
     /**
      * Restore static variables (after running controller test case)
      * @TODO: refactor all code where objects are stored to static variables to use object manager instead
      */
     public static function restoreStaticVariables()
     {
         foreach (array_keys(self::$backupStaticVariables) as $class) {
-            $reflectionClass = new \ReflectionClass($class);
+            $reflectionClass = self::getReflectionClass($class);
             $staticProperties = $reflectionClass->getProperties(\ReflectionProperty::IS_STATIC);
             foreach ($staticProperties as $staticProperty) {
                 $staticProperty->setAccessible(true);
@@ -129,44 +145,50 @@ public static function restoreStaticVariables()
      */
     public static function backupStaticVariables()
     {
-        $classFiles = Files::init()->getPhpFiles(
-            Files::INCLUDE_APP_CODE
-            | Files::INCLUDE_LIBS
-            | Files::INCLUDE_TESTS
+        if (count(self::$backupStaticVariables) > 0) {
+            return;
+        }
+        $classFiles = array_filter(
+            Files::init()->getPhpFiles(
+                Files::INCLUDE_APP_CODE
+                | Files::INCLUDE_LIBS
+                | Files::INCLUDE_TESTS
+            ),
+            function ($classFile) {
+                return StaticProperties::_isClassInCleanableFolders($classFile)
+                && strpos(file_get_contents($classFile), ' static ')  > 0;
+            }
         );
         $namespacePattern = '/namespace [a-zA-Z0-9\\\\]+;/';
         $classPattern = '/\nclass [a-zA-Z0-9_]+/';
         foreach ($classFiles as $classFile) {
-            if (self::_isClassInCleanableFolders($classFile)) {
-                $file = @fopen($classFile, 'r');
-                $code = fread($file, 4096);
-                preg_match($namespacePattern, $code, $namespace);
-                preg_match($classPattern, $code, $class);
-                if (!isset($namespace[0]) || !isset($class[0])) {
-                    fclose($file);
-                    continue;
-                }
-                // trim namespace and class name
-                $namespace = substr($namespace[0], 10, strlen($namespace[0]) - 11);
-                $class = substr($class[0], 7, strlen($class[0]) - 7);
-                $className = $namespace . '\\' . $class;
-
-                try {
-                    $reflectionClass = new \ReflectionClass($className);
-                } catch (\Exception $e) {
-                    fclose($file);
-                    continue;
-                }
-                if (self::_isClassCleanable($reflectionClass)) {
-                    $staticProperties = $reflectionClass->getProperties(\ReflectionProperty::IS_STATIC);
-                    foreach ($staticProperties as $staticProperty) {
-                        $staticProperty->setAccessible(true);
-                        $value = $staticProperty->getValue();
-                        self::$backupStaticVariables[$className][$staticProperty->getName()] = $value;
-                    }
+            $file = @fopen($classFile, 'r');
+            $code = fread($file, 4096);
+            fclose($file);
+            preg_match($namespacePattern, $code, $namespace);
+            preg_match($classPattern, $code, $class);
+            if (!isset($namespace[0]) || !isset($class[0])) {
+                continue;
+            }
+            // trim namespace and class name
+            $namespace = substr($namespace[0], 10, strlen($namespace[0]) - 11);
+            $class = substr($class[0], 7, strlen($class[0]) - 7);
+            $className = $namespace . '\\' . $class;
+
+            try {
+                $reflectionClass = self::getReflectionClass($className);
+            } catch (\Exception $e) {
+                continue;
+            }
+            if (self::_isClassCleanable($reflectionClass)) {
+                $staticProperties = $reflectionClass->getProperties(\ReflectionProperty::IS_STATIC);
+                foreach ($staticProperties as $staticProperty) {
+                    $staticProperty->setAccessible(true);
+                    $value = $staticProperty->getValue();
+                    self::$backupStaticVariables[$className][$staticProperty->getName()] = $value;
                 }
-                fclose($file);
             }
+
         }
     }