You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I wanted to add a different path VSTest argument and I looked at SettingsFile as a pattern to copy, but I found some issues with its current implementation.
The SettingsFile argument is not quoted. Filenames that contain spaces will cause problems. I've confirmed the expected behavior at the command line.
@gep13 pointed out AppendSwitchQuoted which is more idiomatic than the string.Format that was previously used in this method, so I changed all the switches to use AppendSwitch or AppendSwitchQuoted and that resolved the issue.
I wanted to add a different path VSTest argument and I looked at SettingsFile as a pattern to copy, but I found some issues with its current implementation.
The SettingsFile argument is not quoted. Filenames that contain spaces will cause problems. I've confirmed the expected behavior at the command line.
@gep13 pointed out
AppendSwitchQuoted
which is more idiomatic than thestring.Format
that was previously used in this method, so I changed all the switches to useAppendSwitch
orAppendSwitchQuoted
and that resolved the issue.Fix is here: #1268
The text was updated successfully, but these errors were encountered: