Closed Nithanim-Work closed 5 months ago
Yarn PnP is not supported yet.
Also, please provide a minimal Yarn PnP project.
Hello @Nithanim-Work. Please provide a minimal reproduction using a GitHub repository or StackBlitz (you can also use examples). Issues marked with needs reproduction
will be closed if they have no activity within 3 days.
Describe the bug
Using this extension v0.5.6 just shows "Resolving Vitest..." in the "Testing" tab of VS Code forever. We are using yarn with the pnp structure.
I tried debugging it (using latest master dad5884, sorry) and finally traced it back to https://github.com/vitest-dev/vscode/blob/main/src/api.ts#L276
I am not sure how this works but somehow the case is not handled where node does something (and exits?) but not starting any messaging. Capturing the stderr reveals the following:
The "options" to the
fork
function was:Back where this comes from should be https://github.com/vitest-dev/vscode/blob/dad58842672344c686a63eedd80b48aa662dc1ec/src/api/resolve.ts#L55-L57 There,
pnpPath
isc:\Users\redacted\repo\.pnp.cjs
andfolder.uri.fsPath
isc:\Users\redacted\repo
.Honestly, I am not sure, how this works on non-Windows platforms. I would assume that the path is also absolute but then again it complains that it is not
file://
?Thank you in advance!
(Unrelated note: The "Contributing" link for the issue template is broken.)
Reproduction
Any yarn pnp project on windows, I assume.
System Info
Used Package Manager
yarn
Validations