Add ResultsFile Property to MSTestSettings

Related issue: cake-build#730
robertmuehsig · Jul 13, 2016 · 7945eb2 · 7945eb2
1 parent cee3dd8
commit 7945eb2
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 0 deletions.
diff --git a/src/Cake.Common.Tests/Unit/Tools/MSTest/MSTestRunnerTests.cs b/src/Cake.Common.Tests/Unit/Tools/MSTest/MSTestRunnerTests.cs
@@ -190,6 +190,19 @@ public void Should_Use_TestCategoryFilter_If_Provided()
             Assert.Equal("\"/testcontainer:/Working/Test1.dll\" /category:\"!Network\" /noisolation", result.Args);
         }
 
+        [Fact]
+        public void Should_Use_TestResultsFile_If_Provided()
+        {
+            //Given
+            var fixture = new MSTestRunnerFixture();
+            fixture.Settings.ResultsFile = @"c:\temp\myresults.trx";
+
+            //When
+            var result = fixture.Run();
+
+            Assert.Equal("\"/testcontainer:/Working/Test1.dll\" /resultsfile:\"c:\\temp\\myresults.trx\" /noisolation", result.Args);
+        }
+
         [Fact]
         public void Should_Not_Use_TestCategoryFilter_If_Not_Provided()
         {

diff --git a/src/Cake.Common/Tools/MSTest/MSTestRunner.cs b/src/Cake.Common/Tools/MSTest/MSTestRunner.cs
@@ -69,6 +69,11 @@ private ProcessArgumentBuilder GetArguments(IEnumerable<FilePath> assemblyPaths,
                 builder.Append(string.Concat("/category:", settings.Category.Quote()));
             }
 
+            if (!string.IsNullOrEmpty(settings.ResultsFile))
+            {
+                builder.Append(string.Concat("/resultsfile:", settings.ResultsFile.Quote()));
+            }
+
             if (settings.NoIsolation)
             {
                 builder.Append("/noisolation");

diff --git a/src/Cake.Common/Tools/MSTest/MSTestSettings.cs b/src/Cake.Common/Tools/MSTest/MSTestSettings.cs
@@ -28,6 +28,12 @@ public sealed class MSTestSettings : ToolSettings
         /// </summary>
         public string Category { get; set; }
 
+        /// <summary>
+        /// Gets or sets the filepath for a named resulting test file. 
+        /// MSTest.exe flag <see href="https://msdn.microsoft.com/en-us/library/ms182489.aspx#resultsfile">/resultsfile</see>.
+        /// </summary>
+        public string ResultsFile { get; set; }
+
         /// <summary>
         /// Gets or sets the test settings file to pass to MSTest.exe flag <see href="https://msdn.microsoft.com/en-us/library/ms182489.aspx#testsettings">/testsettings</see>.
         /// </summary>