Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix debugging jest tests in windows #165

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

fix debugging jest tests in windows #165

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
2b702c7
fix debugging jest tests in windows
roguib Feb 17, 2021
7f6e7fe
address changes
roguib Feb 23, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 5 additions & 9 deletions .vscode/launch.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,16 +5,12 @@
"name": "Debug Jest Tests",
"type": "node",
"request": "launch",
"runtimeArgs": [
"--inspect-brk",
"${workspaceRoot}/node_modules/.bin/jest",
"--runInBand",
"end-to-end",
"-t",
"shared-with-dependencies"
],
"args": ["--runInBand"],
"console": "integratedTerminal",
"internalConsoleOptions": "neverOpen"
"internalConsoleOptions": "neverOpen",
"windows": {
"program": "${workspaceFolder}/node_modules/jest/bin/jest"
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you try adding this windows part to the current config and see if you're still getting the error?

Copy link
Author

@roguib roguib Feb 19, 2021 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately, only adding windows in the Debug Jest Tests configuration doesn't solve the issue unless runtimeArgs is modified like this:

"args": ["--runInBand"],
"windows": {
  "program": "${workspaceFolder}/node_modules/jest/bin/jest"
}

What do you think about using only the first configuration but changing these two parameters?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I think it'd be preferable to fix the existing config

}
]
}