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.

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

feat: Add workspace root substitution for docker containers #335

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

domsleee
Copy link
Collaborator

@domsleee domsleee commented Oct 14, 2023

Suggested feature for #199

It is a proof of concept of a way of transforming the test file path, that could be used in a docker container

For example, instead of this:

node 'node_modules/.bin/jest' '/Users/user/git/vscode-jest-runner/examples.test.ts' -t 'Example'

With testPathSubstitution as /app/container (i.e. a replacement of the workspace root):

node 'node_modules/.bin/jest' '/app/container/examples.test.ts' -t 'Example'

With testPathSubstitution as ./ (i.e. relative to workspace root):

node 'node_modules/.bin/jest' './examples.test.ts' -t 'Example'

@firsttris
Copy link
Owner

Let me know when you want to merge this because it's still in draft

@firsttris
Copy link
Owner

you also have to permission to merge on your own.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants