[aik099]

Follow-up to minkphp/webdriver-classic-driver#44 (comment)

AbstractConfig now reads WEB_FIXTURES_HOST, TEST_MACHINE_BASE_PATH and DRIVER_MACHINE_BASE_PATH through a new getEnvVar() method: getenv() first, then $_SERVER.

Right now a <server> in a driver's phpunit.xml.dist overwrites the same variable given from the shell/CI, so it can't be overridden without a custom phpunit.xml. With this change:

• the shell/CI value wins over the XML one;
• drivers that use <server> keep working as before;
• drivers can switch their XML to <env> whenever they want (it doesn't write $_SERVER, so that only works after this change).

Once this is merged, the draft PRs minkphp/webdriver-classic-driver#66 and minkphp/MinkSelenium2Driver#415 would use getEnvVar() instead of reading $_SERVER, and replace all <server> with <env> in their phpunit*.xml files, with no commented-out <server> left.

getenv() isn't thread safe, but this is test-suite code only, so I think that's fine here.

Tested locally: all 4 combinations of XML value (set or not) and shell value (set or not), PHPStan, and the MinkBrowserKitDriver suite.

🤖 Generated with Claude Code