Skip to content

Use test database #12779

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 7, 2018
Merged

Use test database #12779

merged 1 commit into from
Jul 7, 2018

Conversation

c960657
Copy link
Contributor

@c960657 c960657 commented Apr 28, 2018

Piwik\Plugins\CoreAdminHome\tests\Integration\Commands\SetConfigTest tries to access the main database. This may fail due to lack of permission.

1) Piwik\Plugins\CoreAdminHome\tests\Integration\Commands\SetConfigTest
Failed to setup fixture: TEST INITIALIZATION FAILED: SQLSTATE[42000]: Syntax error or access violation: 1044 Access denied for user 'matomo_dev'@'localhost' to database 'matomo_test'
#0 /var/www/matomo/core/Db.php(168): Zend_Db_Adapter_Pdo_Abstract->exec('CREATE DATABASE...')
#1 /var/www/matomo/core/Db/Schema/Mysql.php(386): Piwik\Db::exec('CREATE DATABASE...')
#2 /var/www/matomo/core/Db/Schema.php(117): Piwik\Db\Schema\Mysql->createDatabase('matomo_test')
#3 /var/www/matomo/core/DbHelper.php(152): Piwik\Db\Schema->createDatabase('matomo_test')
#4 /var/www/matomo/tests/PHPUnit/Framework/Fixture.php(235): Piwik\DbHelper::createDatabase('matomo_test')
#5 /var/www/matomo/tests/PHPUnit/Framework/TestCase/SystemTestCase.php(76): Piwik\Tests\Framework\Fixture->performSetUp()
#6 /var/www/matomo/plugins/CoreAdminHome/tests/Integration/SetConfigTest.php(29): Piwik\Tests\Framework\TestCase\SystemTestCase::setUpBeforeClass()
#7 /var/www/matomo/vendor/phpunit/phpunit/src/Framework/TestSuite.php(672): Piwik\Plugins\CoreAdminHome\tests\Integration\Commands\SetConfigTest::setUpBeforeClass()
#8 /var/www/matomo/vendor/phpunit/phpunit/src/TextUI/TestRunner.php(440): PHPUnit_Framework_TestSuite->run(Object(PHPUnit_Framework_TestResult))
#9 /var/www/matomo/vendor/phpunit/phpunit/src/TextUI/Command.php(149): PHPUnit_TextUI_TestRunner->doRun(Object(PHPUnit_Framework_TestSuite), Array)
#10 /var/www/matomo/vendor/phpunit/phpunit/src/TextUI/Command.php(100): PHPUnit_TextUI_Command->run(Array, true)
#11 /var/www/matomo/vendor/phpunit/phpunit/phpunit(52): PHPUnit_TextUI_Command::main()
#12 {main}

@c960657 c960657 force-pushed the set-config-test branch from b819c8d to 0a51d3d Compare May 22, 2018 06:34
@mattab mattab added Needs Review PRs that need a code review and removed Needs Review PRs that need a code review labels Jun 26, 2018
@mattab mattab added this to the 3.6.0 milestone Jun 26, 2018
@sgiehl sgiehl merged commit 6492de6 into matomo-org:3.x-dev Jul 7, 2018
@c960657 c960657 deleted the set-config-test branch July 27, 2018 08:57
InfinityVoid pushed a commit to InfinityVoid/matomo that referenced this pull request Oct 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review
Development

Successfully merging this pull request may close these issues.

3 participants