Add multi-pass environment variable resolution by john-patterson · Pull Request #2322 · microsoft/vscode-cpptools

and others added 9 commits

July 24, 2018 09:27
We were passing the root of the test folder instead of the unitTests
specific folder into the runner, so it wasn't finding unit tests.
If a variable had env, config, or workspaceFolder as substring, it was
being treated as an environment variable lookup on the reminder of the
string minus the character immediately following the substring. For
instance: envRoot was being looked up as "oot" in the configuarion
environment block.

The issue is that the regex to match the env., config., and
workspaceFolder. patterns used . instead of \.
This gets rid of linting errors and adopts a more BDD literate assert style.

sean-mcmanus

@john-patterson

sean-mcmanus

sean-mcmanus

@john-patterson

john-patterson added a commit to john-patterson/vscode-cpptools that referenced this pull request

Jul 25, 2018
Realized I had left out the CHANGELOG update. For [PR
2322](microsoft#2322).

bobbrow pushed a commit that referenced this pull request

Jul 27, 2018
Realized I had left out the CHANGELOG update. For [PR
2322](#2322).