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

Redesign the user experience about running/debugging with configuration #524

Closed
jdneo opened this issue Dec 14, 2018 · 3 comments
Closed
Milestone

Comments

@jdneo
Copy link
Member

jdneo commented Dec 14, 2018

Related Issues

#496, #476, #438
#458, #372, #354, ...

Description

Currently, the extension will generate the launch.test.json for users who want to run/debug with configuration for his test cases. While this approach has a lot of problems that hurt the user experices, including:

  • No documents
  • The schema is hard to understand
  • Could not resolve variables such as ${workspaceFolders}
  • Ruin users' source control
  • Obscure implementation logic

Plan

Add several extension settings whose scope are workspace folder, the schema looks like below:

"java.test.config": [
    {
        /* Specify the working directory when running the tests */
        "java.test.cwd": "${workspaceFolder}"

        /* Specify the command line arguments passed to the runner */
        "java.test.args": [ "" ]

        /* Specify the extra options and system properties for the JVM (e.g. -Xms<size> -Xmx<size> - D<name>=<value>) */
        "java.test.vmargs": [ "" ]

        /* Specify the environment variable when running the tests */
        "java.test.env": { "aaa": "bbb" }
    },
    {
        ...  // Another new config
    }
]

Note: The first config in the array will always be picked as the default one.

@llgcode
Copy link

llgcode commented Jan 11, 2019

In my case I have a big eclipse project for all my code with multiple test configurations in it, it could be great if I can list all tests attached to a specific run configuration or something equivalent.

@jdneo jdneo closed this as completed Jan 16, 2019
@chuckb
Copy link

chuckb commented Jan 29, 2019

So does the ${workspaceFolder} expansion work for vmargs or env? It does not seem to. Leaves me in the same state I was prior to this change. I want to add a reference for JNI dlls so that the code lens test runner will work, which reside in my build folder. I can put a relative path in env and that does seem to work.

@jdneo
Copy link
Member Author

jdneo commented Jan 29, 2019

Thank you for reporting @chuckb. Tracking issue: #602

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

No branches or pull requests

3 participants