Skip to content
This repository has been archived by the owner on Apr 20, 2023. It is now read-only.

Add results-directory switch to dotnet test #6151

Merged
merged 4 commits into from
Apr 4, 2017
Merged

Conversation

smadala
Copy link

@smadala smadala commented Mar 25, 2017

@livarcocc @codito @piotrpMSFT

@@ -109,7 +109,7 @@ public void TestWillCreateTrxLoggerInTheSpecifiedResultsDirectoryBySwitch()
// Call test with trx logger enabled and results directory explicitly specified.
CommandResult result = new DotnetTestCommand()
.WithWorkingDirectory(testProjectDirectory)
.ExecuteWithCapturedOutput("--logger:trx -r \"" + trxLoggerDirectory + "\"");
.ExecuteWithCapturedOutput("--logger trx -r \"" + trxLoggerDirectory + "\"");

This comment was marked as spam.

This comment was marked as spam.

@@ -60,5 +60,10 @@ internal class LocalizableStrings

public const string RunSettingsArgsHelpText = @"Any extra command-line runsettings arguments that should be passed to vstest. See 'dotnet vstest --help' for available options.
Example: -- RunConfiguration.ResultsDirectory=""C:\users\user\desktop\Results Directory"" MSTest.DeploymentEnabled=false";

public const string CmdResultsDirectoryDescription = @"The test results directory will be created in the specified path if it does not exist.

This comment was marked as spam.

This comment was marked as spam.

@smadala
Copy link
Author

smadala commented Mar 31, 2017

@dotnet-bot Test Ubuntu x64 Release Build
@dotnet-bot Test RHEL7.2 x64 Release Build

@smadala
Copy link
Author

smadala commented Mar 31, 2017

@dotnet-bot Test Ubuntu x64 Release Build

@smadala
Copy link
Author

smadala commented Mar 31, 2017

@dotnet-bot Test RHEL7.2 x64 Release Build

@smadala
Copy link
Author

smadala commented Mar 31, 2017

@livarcocc Can you please review again?

@livarcocc livarcocc merged commit 6c5d622 into dotnet:master Apr 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants