-
Notifications
You must be signed in to change notification settings - Fork 28
Open
Description
Shouldn't these two points use the same environment variable (at least for the port)?
driver-testsuite/tests/AbstractConfig.php
Line 42 in 2ef2bff
if (!isset($_SERVER['WEB_FIXTURES_HOST'])) { |
driver-testsuite/bin/mink-test-server
Line 9 in 2ef2bff
host=${MINK_HOST-localhost:8002} |
In minkphp/webdriver-classic-driver#23, I was considering to use the mink-test-server
script instead of calling PHP directly in the bootstrap.php file, but from what I understand it wouldn't work, unless I force MINK_HOST
with:
$parsedUrl = parse_url(WebdriverClassicConfig::getInstance()->getWebFixturesUrl());
$process = new Process(..., ['MINK_HOST' => $parsedUrl['host'] . ':' . $parsedUrl['port']])`
@aik099 that's another reason I wasn't relying on the old/current (broken) approach.
Metadata
Metadata
Assignees
Labels
No labels