#1714 fix "Run Test with coverage" doesn't work: "Could not find or load main class"

[YuyaYokoyama8864]

close #1714
We have confirmed that it works in this environment and others on Issue.
We are just removing the double quotations, please confirm.

[jdneo]

Thank you @YuyaYokoyama8864

Does the agent path in your case contains spaces?

I would like to understand why having " causing the failure

[YuyaYokoyama8864]

@jdneo

Does the agent path in your case contains spaces?

The space was not included.

Maybe, but it could happen if I'm already passing vm arguments.
I set java.test.config.vmArg in VSCode's setting.json and passed VM arguments, but when I looked at the contents of debugConfiguration.vmArgs, other VM arguments except for this jacoco were not marked with double quotation marks.

I'm thinking that maybe it's not happening if you're not passing vm arguments and it's happening if you're passing other things you need like I am.

Considering the space, it is certainly not a good idea....

[YuyaYokoyama8864]

@jdneo
I had a little more time so I tried it on a Path that included spaces and it went through without a problem.

It seems that VSCode handles this, or perhaps it is not necessary to add double quotation marks.

[Maikel-Nait]

Do we have any plans to merge this PR ? Some teams in our org are getting blocked by this issue under their Windows machines. (and we can't use our own .vsix build for safety considerations) Thanks.