-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Spaces in file path in windows 10 #37
Comments
Hi, could you try setting the phpunit.phpunit setting directly with quotationmarks? In the end it's just a cmd command so c:/Users/etc needs to be wrapped in " if there are spaces in the string. |
Hello, Thank you for your fast answer. I tried to set it directly in the user settings.json but it did not change the resulting execution block which is without quotation marks. Best regards, |
It should be possible to use the windows short path. If you open a Committed a fix that will wrap the |
I'm seeing this behavior in
So basically it is injecting test file path without quotes into command arguments where it breaks on the first space. My PHPUnit version is |
Ok spaces problems are back :( Are you interested in creating a PR? Think what's needed is adding the quotes in the phpunit args builder class where the file name is added and creating a test case for it to make sure it works. |
Thanks. I'll see if I can manage to submit a PR (workload tides are high after holidays). |
Thanks, no rush. I think the fix is somewhere around here
|
Could there be maybe a temporary workaround until the fix is out? Thanks in advance :) |
This should be fixed now in 4.16.0. Could you update and have a look please. |
Hello,
I saw there is already an issue concerning spaces in the path for the resource file but I am having this issue too. Is there a way I would be able to configure this correctly?
This here is the output when I try to execute a test
Any ideas how to solve this?
Thank you very much and best regards,
Stefan
The text was updated successfully, but these errors were encountered: