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.

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

Running testsuite local (IntegrationTests ) #8846

Closed
ThaDafinser opened this issue Sep 23, 2015 · 7 comments
Closed

Running testsuite local (IntegrationTests ) #8846

ThaDafinser opened this issue Sep 23, 2015 · 7 comments
Labels
answered For when a question was asked and we referred to forum or answered it. Bug For errors / faults / flaws / inconsistencies etc. c: Tests & QA For issues related to automated tests or making it easier to QA & test issues.

Comments

@ThaDafinser
Copy link
Contributor

Hello,

to get the IntegrationTests up and running i needed to manually adjust some things.

After going through https://developer.piwik.org/guides/getting-started-part-1

I needed to adjust the config https://github.com/piwik/piwik/blob/b2f5b5489ceca056217fc6663ec6c2c6b052cd42/tests/PHPUnit/config.ini.travis.php

[tests]
request_uri = "/"

[log]
log_writers[] = file
log_level = info

Would it make sense to automatically do this on ./console development:enable ?

Else maybe just mention it on getting-started-part-1 guide?

@ThaDafinser ThaDafinser changed the title Running testsuite local Running testsuite local (IntegrationTests ) Sep 23, 2015
@tsteur
Copy link
Member

tsteur commented Sep 24, 2015

It's possibly introduced by the new LogViewer plugin that I added recently. Can you let me know which tests fail for you and I'd try to write them in a way that they do not fail and do not need any custom adjustments.

@ThaDafinser
Copy link
Contributor Author

@tsteur nope LogViewer did not introduced this.

There is just no documentation, what settings you need to run this testsuite.

I use those settings now (mostly from the travis config link above)

[database_tests]
host = 127.0.0.1
username = piwik_developmen
password =test
dbname = piwik_tests
adapter = PDO\MYSQL
; no table prefix for tests on travis
tables_prefix =

[tests]
http_host = "http://localhost/GitHub/piwik"
request_uri = "/"

[log]
log_writers[] = file
log_level = info

[Development]
enabled = 1
disable_merged_assets = 1

@tsteur
Copy link
Member

tsteur commented Sep 25, 2015

Oh, this sounds like a bug then. Our tests should be config independent. If it is config independent and we cannot fix it for some reasons we should add list print this information and check whether config is configured as needed etc.

@tsteur tsteur added Bug For errors / faults / flaws / inconsistencies etc. c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. labels Sep 25, 2015
@ThaDafinser
Copy link
Contributor Author

Some specials i found:

  • the database name must be piwik_tests (its checked somewhere in the tests)
  • tables_prefixes must also be blank, otherwise they fail too

@mattab
Copy link
Member

mattab commented Nov 26, 2015

@tsteur what do you reckon is our next steps here?

@tsteur
Copy link
Member

tsteur commented Nov 26, 2015

I'm not sure. Ideally we have a specific case that we can reproduce and fix, not sure if this is given here.

@mattab
Copy link
Member

mattab commented Nov 27, 2015

@ThaDafinser please create a new issue with steps to reproduce issue and we'll investigate!

@mattab mattab closed this as completed Nov 27, 2015
@mattab mattab added the answered For when a question was asked and we referred to forum or answered it. label Nov 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
answered For when a question was asked and we referred to forum or answered it. Bug For errors / faults / flaws / inconsistencies etc. c: Tests & QA For issues related to automated tests or making it easier to QA & test issues.
Projects
None yet
Development

No branches or pull requests

3 participants