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

Follow up to #7644, add test to CronArchive (cli + web) that ensures correct domain config is used #7856

Closed
diosmosis opened this issue May 7, 2015 · 0 comments
Labels
c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. wontfix If you can reproduce this issue, please reopen the issue or create a new one describing it.

Comments

@diosmosis
Copy link
Member

As title, there is currently no test that ensures ./console core:archive --piwik-domain=... uses the right config and that a request to archive.php w/ a different Host: HTTP header uses the right config.

These tests should be added as system tests, but before they are added, a --dry-run parameter must be added to CronArchive.php so the tests do not take forever.

@diosmosis diosmosis added the c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. label May 7, 2015
@diosmosis diosmosis added this to the Short term milestone May 7, 2015
@mattab mattab closed this as completed Jul 8, 2016
@mattab mattab added the wontfix If you can reproduce this issue, please reopen the issue or create a new one describing it. label Jul 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: Tests & QA For issues related to automated tests or making it easier to QA & test issues. wontfix If you can reproduce this issue, please reopen the issue or create a new one describing it.
Projects
None yet
Development

No branches or pull requests

2 participants