Hardening ConfigUpgrade

[friendica.git] / tests / DatabaseTest.php

diff --git a/tests/DatabaseTest.php b/tests/DatabaseTest.php
index 525c9372f514cefb42f18f5a4400447ee1d0aeef..fec31b05af6f69aa49deee5c16a9c957a445b5c7 100644 (file)
--- a/tests/DatabaseTest.php
+++ b/tests/DatabaseTest.php
@@ -5,44 +5,63 @@
 
 namespace Friendica\Test;
 
-use dba;
-use Friendica\Database\DBStructure;
-use PHPUnit_Extensions_Database_DB_IDatabaseConnection;
+use Friendica\App;
+use Friendica\Database\DBA;
+use Friendica\Factory;
+use Friendica\Util\BasePath;
+use Friendica\Util\Config\ConfigFileLoader;
+use Friendica\Util\Profiler;
 use PHPUnit\DbUnit\DataSet\YamlDataSet;
 use PHPUnit\DbUnit\TestCaseTrait;
-use PHPUnit\Framework\TestCase;
+use PHPUnit_Extensions_Database_DB_IDatabaseConnection;
+
+require_once __DIR__ . '/../boot.php';
 
 /**
  * Abstract class used by tests that need a database.
  */
-abstract class DatabaseTest extends TestCase
+abstract class DatabaseTest extends MockedTest
 {
-
        use TestCaseTrait;
 
        /**
         * Get database connection.
+        *
+        * This function is executed before each test in order to get a database connection that can be used by tests.
+        * If no prior connection is available, it tries to create one using the USER, PASS and DB environment variables.
+        *
+        * If it could not connect to the database, the test is skipped.
+        *
         * @return PHPUnit_Extensions_Database_DB_IDatabaseConnection
         * @see https://phpunit.de/manual/5.7/en/database.html
         */
        protected function getConnection()
        {
-               if (!dba::$connected) {
-                       dba::connect('localhost', getenv('USER'), getenv('PASS'), getenv('DB'));
-
-                       if (dba::$connected) {
-                               $app = get_app();
-                               // We need to do this in order to disable logging
-                               $app->module = 'install';
-
-                               // Create database structure
-                               DBStructure::update(false, true, true);
-                       } else {
-                               $this->markTestSkipped('Could not connect to the database.');
-                       }
+               if (!getenv('MYSQL_DATABASE')) {
+                       $this->markTestSkipped('Please set the MYSQL_* environment variables to your test database credentials.');
+               }
+
+               $basePath = BasePath::create(dirname(__DIR__));
+               $mode = new App\Mode($basePath);
+               $configLoader = new ConfigFileLoader($basePath, $mode);
+               $config = Factory\ConfigFactory::createCache($configLoader);
+
+               $profiler = \Mockery::mock(Profiler::class);
+
+               DBA::connect(
+                       $basePath,
+                       $config,
+                       $profiler,
+                       getenv('MYSQL_HOST'),
+                       getenv('MYSQL_USERNAME'),
+                       getenv('MYSQL_PASSWORD'),
+                       getenv('MYSQL_DATABASE'));
+
+               if (!DBA::connected()) {
+                       $this->markTestSkipped('Could not connect to the database.');
                }
 
-               return $this->createDefaultDBConnection(dba::get_db(), 'friendica_test:');
+               return $this->createDefaultDBConnection(DBA::getConnection(), getenv('MYSQL_DATABASE'));
        }
 
        /**
@@ -52,8 +71,6 @@ abstract class DatabaseTest extends TestCase
         */
        protected function getDataSet()
        {
-               return new YamlDataSet(
-                       __DIR__.'/datasets/api.yml'
-               );
+               return new YamlDataSet(__DIR__ . '/datasets/api.yml');
        }
 }