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

envFile: format consistency with C/C++ extension debugging configuration #307

Closed
1 task done
SpaceIm opened this issue Dec 1, 2021 · 1 comment
Closed
1 task done
Labels
enhancement New feature or request

Comments

@SpaceIm
Copy link

SpaceIm commented Dec 1, 2021

Checklist

Is your feature request related to a problem? Please describe.
Currently, this test adapter can take a "envFile" in json format (https://github.com/matepek/vscode-catch2-test-adapter/blob/master/documents/configuration/test.advancedExecutables.md#envfile). It's not consistent with envFile expected by C/C++ extension which is a dotenv file. It's not very convenient, since it requires a pre-process parsing the dotenv file to create another format.

Describe the solution you'd like
Use dotenv file instead of json in envFile property.

Describe alternatives you've considered
Either:

  • Run my tests in the terminal, so basically not relying on Test adapter.
  • after calling conan, parse generated dotenv file and produce a json file specifically for this extension.

Additional context
I'm using conan. conan can generate a dotenv file (as well as a .sh to source on Unix platforms & a .bat for windows) with runtime environment variables of dependencies (PATH/LD_LIBRARY_PATH/DYLD_LIBRARY_PATH with dependencies shared libs locations, and any specific env vars of frameworks). When those env vars are not defined while running the tests, tests depending on these dependencies obviously fail since they can't load the shared libs.

@matepek matepek added the enhancement New feature or request label Dec 1, 2021
@matepek
Copy link
Owner

matepek commented Dec 1, 2021

will be added in next release

@matepek matepek closed this as completed Dec 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants