diff --git a/tests/Util/AppMockTrait.php b/tests/Util/AppMockTrait.php new file mode 100644 index 0000000000..72c0dc429b --- /dev/null +++ b/tests/Util/AppMockTrait.php @@ -0,0 +1,90 @@ +shouldReceive('t') + ->andReturnUsing(function ($arg) { return $arg; }); + + $this->mockConfigGet('system', 'theme', 'testtheme'); + + // Mocking App and most used functions + $this->app = \Mockery::mock(App::class); + $this->app + ->shouldReceive('getBasePath') + ->andReturn($root->url()); + + $this->app + ->shouldReceive('getConfigValue') + ->with('database', 'hostname') + ->andReturn(getenv('MYSQL_HOST')); + $this->app + ->shouldReceive('getConfigValue') + ->with('database', 'username') + ->andReturn(getenv('MYSQL_USERNAME')); + $this->app + ->shouldReceive('getConfigValue') + ->with('database', 'password') + ->andReturn(getenv('MYSQL_PASSWORD')); + $this->app + ->shouldReceive('getConfigValue') + ->with('database', 'database') + ->andReturn(getenv('MYSQL_DATABASE')); + $this->app + ->shouldReceive('getTemplateEngine') + ->andReturn(new FriendicaSmartyEngine()); + $this->app + ->shouldReceive('getCurrentTheme') + ->andReturn('Smarty3'); + $this->app + ->shouldReceive('getTemplateLeftDelimiter') + ->with('smarty3') + ->andReturn('{{'); + $this->app + ->shouldReceive('getTemplateRightDelimiter') + ->with('smarty3') + ->andReturn('}}'); + $this->app + ->shouldReceive('saveTimestamp') + ->andReturn(true); + $this->app + ->shouldReceive('getBaseUrl') + ->andReturn('http://friendica.local'); + + // Mocking the Theme + // Necessary for macro engine with template files + $themeMock = \Mockery::mock('alias:Friendica\Core\Theme'); + $themeMock + ->shouldReceive('install') + ->with('testtheme') + ->andReturn(true); + + BaseObject::setApp($this->app); + } +} diff --git a/tests/Util/ConfigMockTrait.php b/tests/Util/ConfigMockTrait.php new file mode 100644 index 0000000000..d2867a589e --- /dev/null +++ b/tests/Util/ConfigMockTrait.php @@ -0,0 +1,64 @@ +configMock)) { + $this->configMock = \Mockery::mock('alias:Friendica\Core\Config'); + } + + $this->configMock + ->shouldReceive('get') + ->times($times) + ->with($family, $key) + ->andReturn($value); + } + + /** + * Mocking setting a new config entry + * + * @param string $family The family of the config double + * @param string $key The key of the config double + * @param mixed $value The value of the config double + * @param null|int $times How often the Config will get used + * @param bool $return Return value of the set (default is true) + */ + public function mockConfigSet($family, $key, $value, $times = null, $return = true) + { + if (!isset($this->configMock)) { + $this->configMock = \Mockery::mock('alias:Friendica\Core\Config'); + } + + $this->mockConfigGet($family, $key, false, 1); + if ($return) { + $this->mockConfigGet($family, $key, $value, 1); + } + + $this->configMock + ->shouldReceive('set') + ->times($times) + ->with($family, $key, $value) + ->andReturn($return); + } +} diff --git a/tests/Util/DBAMockTrait.php b/tests/Util/DBAMockTrait.php new file mode 100644 index 0000000000..a076ac23d0 --- /dev/null +++ b/tests/Util/DBAMockTrait.php @@ -0,0 +1,52 @@ +dbaMock)) { + $this->dbaMock = \Mockery::mock('alias:Friendica\Database\DBA'); + } + + $this->dbaMock + ->shouldReceive('connect') + ->times($times) + ->andReturn($return); + } + + /** + * Mocking DBA::connected() + * + * @param bool $return True, if the DB is connected, otherwise false + * @param null|int $times How often the method will get used + */ + public function mockConnected($return = true, $times = null) + { + if (!isset($this->dbaMock)) { + $this->dbaMock = \Mockery::mock('alias:Friendica\Database\DBA'); + } + + $this->dbaMock + ->shouldReceive('connected') + ->times($times) + ->andReturn($return); + } +} diff --git a/tests/Util/DBStructureMockTrait.php b/tests/Util/DBStructureMockTrait.php new file mode 100644 index 0000000000..87c120d3f2 --- /dev/null +++ b/tests/Util/DBStructureMockTrait.php @@ -0,0 +1,56 @@ +dbStructure)) { + $this->dbStructure = \Mockery::mock('alias:Friendica\Database\DBStructure'); + } + + $this->dbStructure + ->shouldReceive('update') + ->withArgs($args) + ->times($times) + ->andReturn($return); + } + + /** + * Mocking DBStructure::existsTable() + * + * @param string $tableName The name of the table to check + * @param bool $return True, if the connect was successful, otherwise false + * @param null|int $times How often the method will get used + */ + public function mockExistsTable($tableName, $return = true, $times = null) + { + if (!isset($this->dbStructure)) { + $this->dbStructure = \Mockery::mock('alias:Friendica\Database\DBStructure'); + } + + $this->dbStructure + ->shouldReceive('existsTable') + ->with($tableName) + ->times($times) + ->andReturn($return); + } +} diff --git a/tests/Util/VFSTrait.php b/tests/Util/VFSTrait.php index d51ba5b6a5..972119134a 100644 --- a/tests/Util/VFSTrait.php +++ b/tests/Util/VFSTrait.php @@ -13,11 +13,15 @@ trait VFSTrait */ protected $root; + /** + * Sets up the Virtual File System for Friendica with common files (config, dbstructure) + */ protected function setUpVfsDir() { // the used directories inside the App class $structure = [ 'config' => [], - 'bin' => [] + 'bin' => [], + 'test' => [] ]; // create a virtual directory and copy all needed files and folders to it @@ -29,6 +33,11 @@ trait VFSTrait $this->setConfigFile('dbstructure.php'); } + /** + * Copying a config file from the file system to the Virtual File System + * + * @param string $filename The filename of the config file + */ protected function setConfigFile($filename) { $file = dirname(__DIR__) . DIRECTORY_SEPARATOR . @@ -43,6 +52,11 @@ trait VFSTrait } } + /** + * Delets a config file from the Virtual File System + * + * @param string $filename The filename of the config file + */ protected function delConfigFile($filename) { if ($this->root->hasChild('config/' . $filename)) { diff --git a/tests/datasets/ini/assert.ini.php b/tests/datasets/ini/assert.ini.php new file mode 100644 index 0000000000..39828affcb --- /dev/null +++ b/tests/datasets/ini/assert.ini.php @@ -0,0 +1,56 @@ +db_user = getenv('MYSQL_USERNAME') . getenv('MYSQL_USER'); $this->db_pass = getenv('MYSQL_PASSWORD'); - // Mocking 'DBStructure::existsTable()' because with CI, we cannot create an empty database - // therefore we temporary override the existing database - /// @todo Mocking the DB-Calls of ConsoleTest so we don't need this specific mock anymore - $existsMock = \Mockery::mock('alias:Friendica\Database\DBStructure'); - $existsMock->shouldReceive('existsTable') - ->with('user') - ->andReturn(false); + $this->mockConfigGet('config', 'php_path', false); + + $assertFile = dirname(__DIR__) . DIRECTORY_SEPARATOR . + '..' . DIRECTORY_SEPARATOR . + '..' . DIRECTORY_SEPARATOR . + 'datasets' . DIRECTORY_SEPARATOR . + 'ini' . DIRECTORY_SEPARATOR . + 'assert.ini.php'; + $this->assertFile = vfsStream::newFile('assert.ini.php') + ->at($this->root->getChild('test')) + ->setContent($this->replaceEnvironmentSettings($assertFile, false)); + $this->assertFileDb = vfsStream::newFile('assert_db.ini.php') + ->at($this->root->getChild('test')) + ->setContent($this->replaceEnvironmentSettings($assertFile, true)); } - private function assertConfig($family, $key, $value) + /** + * Replacing environment specific variables in the assertion file + * + * @param string $file The file to compare in later tests + * @param bool $withDb If true, db settings are replaced too + * @return string The file content + */ + private function replaceEnvironmentSettings($file, $withDb) { - $config = $this->execute(['config', $family, $key]); - $this->assertEquals($family . "." . $key . " => " . $value . "\n", $config); + $fileContent = file_get_contents($file); + $fileContent = str_replace("/usr/bin/php", trim(shell_exec('which php')), $fileContent); + if ($withDb) { + $fileContent = str_replace("hostname = \"\"", "hostname = \"" . $this->db_host . (!empty($this->db_port) ? ":" . $this->db_port : "") . "\"", $fileContent); + $fileContent = str_replace("username = \"\"", "username = \"" . $this->db_user . "\"", $fileContent); + $fileContent = str_replace("password = \"\"", "password = \"" . $this->db_pass . "\"", $fileContent); + $fileContent = str_replace("database = \"\"", "database = \"" . $this->db_data . "\"", $fileContent); + } + return $fileContent; } private function assertFinished($txt, $withconfig = false, $copyfile = false) @@ -113,14 +150,17 @@ FIN; $finished = <<mockConnect(true, 1); + $this->mockConnected(true, 1); + $this->mockExistsTable('user', false, 1); + $this->mockUpdate([false, true, true], null, 1); + $config = <<at($this->root) ->setContent($config); - $txt = $this->execute(['autoinstall', '-f', 'prepared.ini.php']); + $console = new AutomaticInstallation($this->consoleArgv); + $console->setOption('f', 'prepared.ini.php'); + + $txt = $this->dumpExecute($console); $this->assertFinished($txt, false, true); @@ -191,23 +239,28 @@ CONF; */ public function testWithEnvironmentAndSave() { + $this->mockConnect(true, 1); + $this->mockConnected(true, 1); + $this->mockExistsTable('user', false, 1); + $this->mockUpdate([false, true, true], null, 1); + $this->assertTrue(putenv('FRIENDICA_ADMIN_MAIL=admin@friendica.local')); $this->assertTrue(putenv('FRIENDICA_TZ=Europe/Berlin')); $this->assertTrue(putenv('FRIENDICA_LANG=de')); + $this->assertTrue(putenv('FRIENDICA_URL_PATH=/friendica')); - $txt = $this->execute(['autoinstall', '--savedb']); + $console = new AutomaticInstallation($this->consoleArgv); + $console->setOption('savedb', true); + + $txt = $this->dumpExecute($console); $this->assertFinished($txt, true); $this->assertTrue($this->root->hasChild('config' . DIRECTORY_SEPARATOR . 'local.ini.php')); - $this->assertConfig('database', 'hostname', $this->db_host . (!empty($this->db_port) ? ':' . $this->db_port : '')); - $this->assertConfig('database', 'username', $this->db_user); - $this->assertConfig('database', 'database', $this->db_data); - $this->assertConfig('config', 'admin_email', 'admin@friendica.local'); - $this->assertConfig('system', 'default_timezone', 'Europe/Berlin'); - // TODO language changes back to en - //$this->assertConfig('system', 'language', 'de'); + $this->assertFileEquals( + $this->assertFileDb->url(), + $this->root->getChild('config' . DIRECTORY_SEPARATOR . 'local.ini.php')->url()); } /** @@ -215,25 +268,27 @@ CONF; */ public function testWithEnvironmentWithoutSave() { + $this->mockConnect(true, 1); + $this->mockConnected(true, 1); + $this->mockExistsTable('user', false, 1); + $this->mockUpdate([false, true, true], null, 1); + $this->assertTrue(putenv('FRIENDICA_ADMIN_MAIL=admin@friendica.local')); $this->assertTrue(putenv('FRIENDICA_TZ=Europe/Berlin')); $this->assertTrue(putenv('FRIENDICA_LANG=de')); $this->assertTrue(putenv('FRIENDICA_URL_PATH=/friendica')); - $txt = $this->execute(['autoinstall']); + $console = new AutomaticInstallation($this->consoleArgv); + + $txt = $this->dumpExecute($console); $this->assertFinished($txt, true); $this->assertTrue($this->root->hasChild('config' . DIRECTORY_SEPARATOR . 'local.ini.php')); - $this->assertConfig('database', 'hostname', ''); - $this->assertConfig('database', 'username', ''); - $this->assertConfig('database', 'database', ''); - $this->assertConfig('config', 'admin_email', 'admin@friendica.local'); - $this->assertConfig('system', 'default_timezone', 'Europe/Berlin'); - $this->assertConfig('system', 'urlpath', '/friendica'); - // TODO language changes back to en - //$this->assertConfig('system', 'language', 'de'); + $this->assertFileEquals( + $this->assertFile->url(), + $this->root->getChild('config' . DIRECTORY_SEPARATOR . 'local.ini.php')->url()); } /** @@ -241,46 +296,38 @@ CONF; */ public function testWithArguments() { - $args = ['autoinstall']; - array_push($args, '--dbhost'); - array_push($args, $this->db_host); - array_push($args, '--dbuser'); - array_push($args, $this->db_user); + $this->mockConnect(true, 1); + $this->mockConnected(true, 1); + $this->mockExistsTable('user', false, 1); + $this->mockUpdate([false, true, true], null, 1); + + $console = new AutomaticInstallation($this->consoleArgv); + + $console->setOption('dbhost', $this->db_host); + $console->setOption('dbuser', $this->db_user); if (!empty($this->db_pass)) { - array_push($args, '--dbpass'); - array_push($args, $this->db_pass); + $console->setOption('dbpass', $this->db_pass); } if (!empty($this->db_port)) { - array_push($args, '--dbport'); - array_push($args, $this->db_port); + $console->setOption('dbport', $this->db_port); } - array_push($args, '--dbdata'); - array_push($args, $this->db_data); + $console->setOption('dbdata', $this->db_data); - array_push($args, '--admin'); - array_push($args, 'admin@friendica.local'); - array_push($args, '--tz'); - array_push($args, 'Europe/Berlin'); - array_push($args, '--lang'); - array_push($args, 'de'); + $console->setOption('admin', 'admin@friendica.local'); + $console->setOption('tz', 'Europe/Berlin'); + $console->setOption('lang', 'de'); - array_push($args, '--urlpath'); - array_push($args, '/friendica'); + $console->setOption('urlpath', '/friendica'); - $txt = $this->execute($args); + $txt = $this->dumpExecute($console); $this->assertFinished($txt, true); $this->assertTrue($this->root->hasChild('config' . DIRECTORY_SEPARATOR . 'local.ini.php')); - $this->assertConfig('database', 'hostname', $this->db_host . (!empty($this->db_port) ? ':' . $this->db_port : '')); - $this->assertConfig('database', 'username', $this->db_user); - $this->assertConfig('database', 'database', $this->db_data); - $this->assertConfig('config', 'admin_email', 'admin@friendica.local'); - $this->assertConfig('system', 'default_timezone', 'Europe/Berlin'); - $this->assertConfig('system', 'urlpath', '/friendica'); - // TODO language changes back to en - //$this->assertConfig('system', 'language', 'de'); + $this->assertFileEquals( + $this->assertFileDb->url(), + $this->root->getChild('config' . DIRECTORY_SEPARATOR . 'local.ini.php')->url()); } /** @@ -289,15 +336,11 @@ CONF; */ public function testNoDatabaseConnection() { - // TODO DBA mocking for whole console tests make this test work again - $this->markTestSkipped('DBA is already loaded, we have to mock the whole App to make it work'); + $this->mockConnect(false, 1); - $dbaMock = \Mockery::mock('alias:Friendica\Database\DBA'); - $dbaMock - ->shouldReceive('connected') - ->andReturn(false); + $console = new AutomaticInstallation($this->consoleArgv); - $txt = $this->execute(['autoinstall']); + $txt = $this->dumpExecute($console); $this->assertStuckDB($txt); } @@ -357,7 +400,10 @@ Examples HELP; - $txt = $this->execute(['autoinstall', '-h']); + $console = new AutomaticInstallation($this->consoleArgv); + $console->setOption('help', true); + + $txt = $this->dumpExecute($console); $this->assertEquals($txt, $theHelp); } diff --git a/tests/src/Core/Console/ConfigConsoleTest.php b/tests/src/Core/Console/ConfigConsoleTest.php index c4fd217770..8f845ae7b0 100644 --- a/tests/src/Core/Console/ConfigConsoleTest.php +++ b/tests/src/Core/Console/ConfigConsoleTest.php @@ -2,7 +2,7 @@ namespace Friendica\Test\src\Core\Console; -use Friendica\Database\DBA; +use Friendica\Core\Console\Config; /** * @runTestsInSeparateProcesses @@ -11,76 +11,147 @@ use Friendica\Database\DBA; */ class ConfigConsoleTest extends ConsoleTest { - public function tearDown() + protected function setUp() { - DBA::delete('config', ['k' => 'test']); + parent::setUp(); - parent::tearDown(); - } + \Mockery::getConfiguration()->setConstantsMap([ + 'Friendica\App\Mode' => [ + 'DBCONFIGAVAILABLE' => 0 + ] + ]); - private function assertGet($family, $key, $value) { - $config = $this->execute(['config', $family, $key]); - $this->assertEquals($family . "." . $key . " => " . $value . "\n", $config); - } + $mode = \Mockery::mock('alias:Friendica\App\Mode'); + $mode + ->shouldReceive('has') + ->andReturn(true); - private function assertSet($family, $key, $value) { - $config = $this->execute(['config', $family, $key, $value]); - $this->assertEquals($family . "." . $key . " <= " . $value . "\n", $config); + $this->app + ->shouldReceive('getMode') + ->andReturn($mode); } function testSetGetKeyValue() { - $this->assertSet( 'config', 'test', 'now'); - $this->assertGet('config', 'test', 'now'); - $this->assertSet('config', 'test', ''); - $this->assertGet('config', 'test', ''); - DBA::delete('config', ['k' => 'test']); - $this->assertGet('config', 'test', null); + $this->mockConfigSet('config', 'test', 'now', 1); + $console = new Config($this->consoleArgv); + $console->setArgument(0, 'config'); + $console->setArgument(1, 'test'); + $console->setArgument(2, 'now'); + $txt = $this->dumpExecute($console); + $this->assertEquals("config.test <= now\n", $txt); + + $this->mockConfigGet('config', 'test', 'now', 1); + $console = new Config($this->consoleArgv); + $console->setArgument(0, 'config'); + $console->setArgument(1, 'test'); + $txt = $this->dumpExecute($console); + $this->assertEquals("config.test => now\n", $txt); + + $this->mockConfigGet('config', 'test', null, 1); + $console = new Config($this->consoleArgv); + $console->setArgument(0, 'config'); + $console->setArgument(1, 'test'); + $txt = $this->dumpExecute($console); + $this->assertEquals("config.test => \n", $txt); } function testSetArrayValue() { $testArray = [1, 2, 3]; - DBA::insert('config', ['cat' => 'config', 'k' => 'test', 'v' => serialize($testArray)]); + $this->mockConfigGet('config', 'test', $testArray, 1); - $txt = $this->execute(['config', 'config', 'test', 'now']); + $console = new Config($this->consoleArgv); + $console->setArgument(0, 'config'); + $console->setArgument(1, 'test'); + $console->setArgument(2, 'now'); + $txt = $this->dumpExecute($console); $this->assertEquals("[Error] config.test is an array and can't be set using this command.\n", $txt); } function testTooManyArguments() { - $txt = $this->execute(['config', 'config', 'test', 'it', 'now']); + $console = new Config($this->consoleArgv); + $console->setArgument(0, 'config'); + $console->setArgument(1, 'test'); + $console->setArgument(2, 'it'); + $console->setArgument(3, 'now'); + $txt = $this->dumpExecute($console); $assertion = '[Warning] Too many arguments'; $firstline = substr($txt, 0, strlen($assertion)); - $this->assertEquals($assertion, $firstline); } function testVerbose() { - $this->assertSet('test', 'it', 'now'); - $executable = $this->getExecutablePath(); + $this->mockConfigGet('test', 'it', 'now', 1); + $console = new Config($this->consoleArgv); + $console->setArgument(0, 'test'); + $console->setArgument(1, 'it'); + $console->setOption('v', 1); + $executable = $this->consoleArgv[0]; $assertion = << 'config', - 1 => 'test', -) -Options: array ( - 'v' => 1, -) -Command: config -Executable: {$executable} Class: Friendica\Core\Console\Config Arguments: array ( 0 => 'test', + 1 => 'it', ) Options: array ( 'v' => 1, ) -[test] -it => now +test.it => now CONF; - $txt = $this->execute(['config', 'test', '-v']); - + $txt = $this->dumpExecute($console); $this->assertEquals($assertion, $txt); } + + function testUnableToSet() { + $this->mockConfigSet('test', 'it', 'now', 1, false); + $console = new Config(); + $console->setArgument(0, 'test'); + $console->setArgument(1, 'it'); + $console->setArgument(2, 'now'); + $txt = $this->dumpExecute($console); + $this->assertSame("Unable to set test.it\n", $txt); + } + + public function testGetHelp() + { + // Usable to purposely fail if new commands are added without taking tests into account + $theHelp = << [-h|--help|-?] [-v] + bin/console config [-h|--help|-?] [-v] + bin/console config [-h|--help|-?] [-v] + +Description + bin/console config + Lists all config values + + bin/console config + Lists all config values in the provided category + + bin/console config + Shows the value of the provided key in the category + + bin/console config + Sets the value of the provided key in the category + +Notes: + Setting config entries which are manually set in config/local.ini.php may result in + conflict between database settings and the manual startup settings. + +Options + -h|--help|-? Show help information + -v Show more debug information. + +HELP; + $console = new Config($this->consoleArgv); + $console->setOption('help', true); + + $txt = $this->dumpExecute($console); + + $this->assertEquals($txt, $theHelp); + } } diff --git a/tests/src/Core/Console/ConsoleTest.php b/tests/src/Core/Console/ConsoleTest.php index 75f339e8f5..0997269c06 100644 --- a/tests/src/Core/Console/ConsoleTest.php +++ b/tests/src/Core/Console/ConsoleTest.php @@ -2,71 +2,57 @@ namespace Friendica\Test\src\Core\Console; -use Friendica\App; -use Friendica\BaseObject; -use Friendica\Database\DBA; +use Asika\SimpleConsole\Console; +use Friendica\Test\Util\AppMockTrait; use Friendica\Test\Util\Intercept; use Friendica\Test\Util\VFSTrait; -use org\bovigo\vfs\vfsStream; -use org\bovigo\vfs\vfsStreamDirectory; use PHPUnit\Framework\TestCase; abstract class ConsoleTest extends TestCase { use VFSTrait; + use AppMockTrait; /** - * @var MultiUseConsole Extension of the basic Friendica Console for testing purpose + * @var array The default argv for a Console Instance */ - private $console; - /** - * @var App The Friendica App - */ - protected $app; - - protected $stdout; + protected $consoleArgv = [ 'consoleTest.php' ]; protected function setUp() { parent::setUp(); - Intercept::setUp(); - if (!getenv('MYSQL_DATABASE')) { $this->markTestSkipped('Please set the MYSQL_* environment variables to your test database credentials.'); } + Intercept::setUp(); + $this->setUpVfsDir(); - - // fake console.php for setting an executable - vfsStream::newFile('console.php') - ->at($this->root->getChild('bin')) - ->setContent('app = new App($this->root->url()); - BaseObject::setApp($this->app); - $this->console = new MultiUseConsole(); + $this->mockApp($this->root); } - public function execute($args) { - $this->app->reload(); + protected function tearDown() + { + \Mockery::close(); - array_unshift($args, $this->getExecutablePath()); - Intercept::reset(); - $this->console->reset(); - $this->console->parseTestArgv($args); - $this->console->execute(); - - $returnStr = Intercept::$cache; - Intercept::reset(); - return $returnStr; + parent::tearDown(); } /** - * @return string returns the path to the console executable during tests + * Dumps the execution of an console output to a string and returns it + * + * @param Console $console The current console instance + * + * @return string the output of the execution */ - protected function getExecutablePath() { - return $this->root->getChild('bin' . DIRECTORY_SEPARATOR . 'console.php')->url(); + protected function dumpExecute($console) + { + Intercept::reset(); + $console->execute(); + $returnStr = Intercept::$cache; + Intercept::reset(); + + return $returnStr; } } diff --git a/tests/src/Core/Console/MultiUseConsole.php b/tests/src/Core/Console/MultiUseConsole.php deleted file mode 100644 index ddcbfebc34..0000000000 --- a/tests/src/Core/Console/MultiUseConsole.php +++ /dev/null @@ -1,23 +0,0 @@ -args = []; - $this->options = []; - } - - public function parseTestArgv($argv) - { - $this->parseArgv($argv); - } -}