Testhost Diag log file name format change #303
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
smadala
commented
Dec 23, 2016
smadala
commented
- Change testhost diag log file name format, to avoid overwrite in parallel execution
|
👍 |
harshjain2
added a commit
that referenced
this pull request
Dec 27, 2016
* Allow TranslationLayer to specify Diag parameters Log message for logging file of test host * Passing runsettings as command line arguments. * Removed unused variable. * Added regex, testcases. * Added new parameter VSTestCLIRunSettings in VSTest Task. * Added htlm encoding to the RunSettings values coming as CLI arguments. * Style fixes * Fix typo * Work done under this commit: (#302) 1) Create xlf file for all language 2) Create language resx file from xlf file 3) create satellite dll * Testhost Diag log file name format change (#303) Update diag log format for testhost. New format ensures log files are unique so that we don't run multiple testhosts with same file name in case of parallel runs. * Changes for passing RunSettings args as array of string. * Doc changes. * test fixes. * Removed unused namespaces, improved documentation, added trim to key. * Removed HTML Encoding as it is already done while validating xml. * Removed ununsed namespace and package.run
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.