phpunit-watcher failed to execute phpunit about phpunit-watcher HOT 5 CLOSED

Fixed in latest version.

from phpunit-watcher.

I have the same issue. I'm using PHP 7.2, everything ele is the same as @yjv-XCV.

from phpunit-watcher.

I actually found where the problems came from, but have no idea how to fix it.
in vendor/symfony/process/Process.php:229

$commandline = 'exec '.$commandline;

The command line is received from
vendor/spatie/phpunit-watcher/src/Screens/Phpunit.php:93
which is combination of phpunitBinaryPath and phpunitArguments.
However the problem lies when it execute exec, it take the space, and the arguments as part of the file name.
When I didn't apply any phpunit arguments, it will have a space after the phpunitBinaryPath in the $commandline, and it works after I remove the space from it.
From

$commandline; // output: "exec /myprojectPath/vendor/bin/phpunit "

change to

$commandline; // output: "exec /myprojectPath/vendor/bin/phpunit"

Please advise, if any setting need to be done to make the execution will escape the space, and treat the arguments properly.

from phpunit-watcher.

After I modify
vendor/spatie/phpunit-watcher/src/Screens/Phpunit.php:93
from

$result = (new Process(["{$this->phpunitBinaryPath} {$this->phpunitArguments}"]))

to

$result = (new Process(["{$this->phpunitBinaryPath}", "{$this->phpunitArguments}"]))

it works fine.
Not sure if it will break other logic of the application.

from phpunit-watcher.

That's great @yjv-XCV. This change works for me too.

I dug a little deeper in the docs for Symfony Process and according to the documentation since v3.4 that's how it should be, the command separate from the arguments. Only from v3.3 below do we find the signature that is in phpunit-watcher.

I would suggest you open a PR with the change, although we no longer need the string wrap on the arguments:

$result = (new Process([$this->phpunitBinaryPath, $this->phpunitArguments]));

from phpunit-watcher.