Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Failing System test on Mysqli CI #8672

Closed
mattab opened this issue Aug 31, 2015 · 2 comments
Closed

Failing System test on Mysqli CI #8672

mattab opened this issue Aug 31, 2015 · 2 comments
Labels
c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Milestone
2.15.0

Comments

@mattab
Copy link
Member

mattab commented Aug 31, 2015 

1) Warning

The data provider specified for Piwik\Tests\System\AutoSuggestAPITest::testApi is invalid.

Unknown database 'piwik_tests'

maybe someone has some idea?
@mattab mattab added c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. labels Aug 31, 2015
@mattab mattab added this to the 2.15.0 milestone Aug 31, 2015
@diosmosis
Copy link
Member

The AutoSuggestAPITest requires the plugins to be loaded for the data provider, which requires the database to exist. Data providers are run before setUp/setUpBeforeClass. For some reason on MYSQLi, the database doesn't exist here, but on PDO it does... Not sure why, but the best fix would be to remove the need to have the database for the data provider.

@mattab mattab mentioned this issue Sep 3, 2015
Merged
@mattab
Copy link
Member Author

mattab commented Sep 3, 2015

fixed in #8703

@mattab mattab closed this as completed Sep 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Projects
None yet
Milestone
2.15.0
Development

No branches or pull requests
2 participants
@diosmosis @mattab