@diosmosis opened this Pull Request on September 26th 2017 Member

…so correct INI config will be used in tests.

I'm using multiple INI configs to separate different environments (primarily docker vs local). Unfortunately, --piwik-domain option has no effect on tests:run, since that invokes phpunit directly (so the normal config.ini.php file is used).

This PR works around that by passing the piwik domain via environment variable to phpunit, and handling it in bootstrap.php (if it's found).

This change would make development more convenient for me, but it's not life or death. It means if I switch contexts, I don't have to manually change config.ini.php.

This Pull Request was closed on September 26th 2017
Powered by GitHub Issue Mirror