From 3a49cb33bbddbdc506ff493364186f57d11baefd Mon Sep 17 00:00:00 2001
From: William Desportes <williamdes@wdes.fr>
Date: Thu, 21 May 2020 13:35:25 +0200
Subject: [PATCH] Move defining envs to defineTestingGlobals

Signed-off-by: William Desportes <williamdes@wdes.fr>
---
 test/bootstrap-dist.php           | 28 ----------------------------
 test/classes/AbstractTestCase.php | 31 +++++++++++++++++++++++++++++++
 test/classes/EnvironmentTest.php  |  1 +
 test/selenium/TestBase.php        |  2 ++
 4 files changed, 34 insertions(+), 28 deletions(-)

diff --git a/test/bootstrap-dist.php b/test/bootstrap-dist.php
index 137cde2445..e48cc7d142 100644
--- a/test/bootstrap-dist.php
+++ b/test/bootstrap-dist.php
@@ -41,34 +41,6 @@ if (! defined('PHPMYADMIN')) {
 }
 // phpcs:enable
 
-// Selenium tests setup
-$test_defaults = [
-    'TESTSUITE_SERVER' => 'localhost',
-    'TESTSUITE_USER' => 'root',
-    'TESTSUITE_PASSWORD' => '',
-    'TESTSUITE_DATABASE' => 'test',
-    'TESTSUITE_PORT' => 3306,
-    'TESTSUITE_URL' => 'http://localhost/phpmyadmin/',
-    'TESTSUITE_SELENIUM_HOST' => '',
-    'TESTSUITE_SELENIUM_PORT' => '4444',
-    'TESTSUITE_SELENIUM_BROWSER' => 'firefox',
-    'TESTSUITE_SELENIUM_COVERAGE' => '',
-    'TESTSUITE_BROWSERSTACK_USER' => '',
-    'TESTSUITE_BROWSERSTACK_KEY' => '',
-    'TESTSUITE_FULL' => '',
-    'CI_MODE' => '',
-];
-if (PHP_SAPI == 'cli') {
-    foreach ($test_defaults as $varname => $defvalue) {
-        $envvar = getenv($varname);
-        if ($envvar) {
-            $GLOBALS[$varname] = $envvar;
-        } else {
-            $GLOBALS[$varname] = $defvalue;
-        }
-    }
-}
-
 require_once ROOT_PATH . 'libraries/vendor_config.php';
 require_once AUTOLOAD_FILE;
 Loader::loadFunctions();
diff --git a/test/classes/AbstractTestCase.php b/test/classes/AbstractTestCase.php
index a44c08095e..b62b85d9b3 100644
--- a/test/classes/AbstractTestCase.php
+++ b/test/classes/AbstractTestCase.php
@@ -137,6 +137,37 @@ abstract class AbstractTestCase extends TestCase
         // phpcs:enable
     }
 
+    public static function defineTestingGlobals(): void
+    {
+        // Selenium tests setup
+        $test_defaults = [
+            'TESTSUITE_SERVER' => 'localhost',
+            'TESTSUITE_USER' => 'root',
+            'TESTSUITE_PASSWORD' => '',
+            'TESTSUITE_DATABASE' => 'test',
+            'TESTSUITE_PORT' => 3306,
+            'TESTSUITE_URL' => 'http://localhost/phpmyadmin/',
+            'TESTSUITE_SELENIUM_HOST' => '',
+            'TESTSUITE_SELENIUM_PORT' => '4444',
+            'TESTSUITE_SELENIUM_BROWSER' => 'firefox',
+            'TESTSUITE_SELENIUM_COVERAGE' => '',
+            'TESTSUITE_BROWSERSTACK_USER' => '',
+            'TESTSUITE_BROWSERSTACK_KEY' => '',
+            'TESTSUITE_FULL' => '',
+            'CI_MODE' => '',
+        ];
+        if (PHP_SAPI == 'cli') {
+            foreach ($test_defaults as $varname => $defvalue) {
+                $envvar = getenv($varname);
+                if ($envvar) {
+                    $GLOBALS[$varname] = $envvar;
+                } else {
+                    $GLOBALS[$varname] = $defvalue;
+                }
+            }
+        }
+    }
+
     /**
      * Desctroys the environment built for the test.
      * Clean all variables
diff --git a/test/classes/EnvironmentTest.php b/test/classes/EnvironmentTest.php
index 250a3d9355..31746cc8bf 100644
--- a/test/classes/EnvironmentTest.php
+++ b/test/classes/EnvironmentTest.php
@@ -39,6 +39,7 @@ class EnvironmentTest extends AbstractTestCase
      */
     public function testMySQL()
     {
+        AbstractTestCase::defineTestingGlobals();
         try {
             $pdo = new PDO(
                 'mysql:host=' . $GLOBALS['TESTSUITE_SERVER'] . ';port=' . $GLOBALS['TESTSUITE_PORT'],
diff --git a/test/selenium/TestBase.php b/test/selenium/TestBase.php
index 8f482a528a..3ff66d6848 100644
--- a/test/selenium/TestBase.php
+++ b/test/selenium/TestBase.php
@@ -21,6 +21,7 @@ use Facebook\WebDriver\WebDriverSelect;
 use InvalidArgumentException;
 use mysqli;
 use mysqli_result;
+use PhpMyAdmin\Tests\AbstractTestCase;
 use PHPUnit\Framework\TestCase;
 use Throwable;
 use const CURLOPT_CUSTOMREQUEST;
@@ -93,6 +94,7 @@ abstract class TestBase extends TestCase
      */
     protected function setUp(): void
     {
+        AbstractTestCase::defineTestingGlobals();
         /**
          * Needs to be implemented
          *