Skip to content

Commit

Permalink
Fix executable args for phpdbg (#572)
Browse files Browse the repository at this point in the history
* Fix executable args for phpdbg

* Refactor php executable args

* Fix arguments search in WrapperWorker as well
  • Loading branch information
Arthur-Sk authored Dec 15, 2020
1 parent 30e8d22 commit 235db99
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
5 changes: 4 additions & 1 deletion src/Runners/PHPUnit/Worker/RunnerWorker.php
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,10 @@ public function __construct(ExecutableTest $executableTest, Options $options, in
{
$this->executableTest = $executableTest;

$args = [(new PhpExecutableFinder())->find()];
$phpFinder = new PhpExecutableFinder();
$args = [$phpFinder->find(false)];
$args = array_merge($args, $phpFinder->findArguments());

if (($passthruPhp = $options->passthruPhp()) !== null) {
$args = array_merge($args, $passthruPhp);
}
Expand Down
5 changes: 4 additions & 1 deletion src/Runners/PHPUnit/Worker/WrapperWorker.php
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,10 @@ public function __construct(OutputInterface $output, Options $options, int $toke
);
touch($this->writeToPathname);

$parameters = [(new PhpExecutableFinder())->find()];
$phpFinder = new PhpExecutableFinder();
$parameters = [$phpFinder->find(false)];
$parameters = array_merge($parameters, $phpFinder->findArguments());

if (($passthruPhp = $options->passthruPhp()) !== null) {
$parameters = array_merge($parameters, $passthruPhp);
}
Expand Down

0 comments on commit 235db99

Please sign in to comment.