Merge test run parameters that have spaces

src/vstest.console/CommandLine/Executor.cs

@@ -183,6 +183,7 @@ private int GetArgumentProcessors(string[] args, out List<IArgumentProcessor> pr
                     processors.Add(cliRunSettingsProcessor);
                     break;
                 }
+
                 var processor = processorFactory.CreateArgumentProcessor(arg);
 
                 if (processor != null)

src/vstest.console/Processors/CLIRunSettingsArgumentProcessor.cs

@@ -14,6 +14,10 @@ namespace Microsoft.VisualStudio.TestPlatform.CommandLine.Processors
 
     using CommandLineResources = Microsoft.VisualStudio.TestPlatform.CommandLine.Resources.Resources;
     using System.Text.RegularExpressions;
+    using System.Collections.Generic;
+    using System.Linq;
+    using System.Reflection.Metadata;
+    using System.Runtime.ExceptionServices;
 
     /// <summary>
     /// The argument processor for runsettings passed as argument through cli
@@ -136,11 +140,53 @@ public ArgumentProcessorResult Execute()
 
         private void CreateOrOverwriteRunSettings(IRunSettingsProvider runSettingsProvider, string[] args)
         {
-            var length = args.Length;
+            var mergedArgs = new List<string>();
+            var mergedArg = string.Empty;
+            var merge = false;
+
+            foreach (var arg in args)
+            {
+                if (arg.StartsWith("TestRunParameters", StringComparison.OrdinalIgnoreCase))
+                {
+                    if (arg.EndsWith("\")")) {
+                        // this parameter is complete
+                        mergedArgs.Add(arg);
+                    }
+                    else
+                    {
+                        // this parameter needs merging
+                        merge = true;
+                    }
+                }
+
+                if (merge)
+                {
+                    mergedArg += string.IsNullOrWhiteSpace(mergedArg) ? arg : $" {arg}";
+                }
+                else
+                {
+                    mergedArgs.Add(arg);
+                }
+
+                if (merge && arg.EndsWith("\")")) {
+                    mergedArgs.Add(mergedArg);
+                    mergedArg = string.Empty;
+                    merge = false;
+                }
+            }
+
+            if (merge == true)
+            {
+                // we tried to merge but never finished, add what we merged up until now
+                mergedArgs.Add(mergedArg);
+            }
+
+
+            var length = mergedArgs.Count;
 
             for (int index = 0; index < length; index++)
             {
-                var arg = args[index];
+                var arg = mergedArgs[index];
 
                 if (UpdateTestRunParameterNode(runSettingsProvider, arg))
                 {

test/vstest.console.UnitTests/Processors/CLIRunSettingsArgumentProcessorTests.cs

@@ -347,5 +347,46 @@ public static IEnumerable<object[]> TestRunParameterArgValidTestCases()
             },
         };
         #endregion
+
+        [TestMethod]
+        public void InitializeShouldMergeTestRunParametersWithSpaces()
+        {
+            // in powershell call: ConsoleApp1.exe --% --TestRunParameters.Parameter(name =\"myParam\", value=\"myValue\")
+            // args:
+            //--
+            //TestRunParameters.Parameter(name = "myParam",
+            //value = "myValue")
+
+            // in cmd: ConsoleApp1.exe -- TestRunParameters.Parameter(name=\"myParam\", value=\"myValue\")
+            // args:
+            //--
+            //TestRunParameters.Parameter(name = "myParam",
+            //value = "myValue")
+
+            // in ubuntu wsl without escaping the space: ConsoleApp1.exe-- TestRunParameters.Parameter\(name =\"myParam\", value=\"myValue\"\)
+            // args:
+            //--
+            //TestRunParameters.Parameter(name = "myParam",
+            //value = "myValue")
+
+            // in ubuntu wsl with escaping the space: ConsoleApp1.exe-- TestRunParameters.Parameter\(name =\"myParam\",\ value=\"myValue\"\)
+            // args:
+            //--
+            //TestRunParameters.Parameter(name = "myParam", value = "myValue")
+
+            var args = new string[] {
+                "--",
+                "TestRunParameters.Parameter(name=\"myParam\",",
+                "value=\"myValue\")",
+                "TestRunParameters.Parameter(name=\"myParam2\",",
+                "value=\"myValue 2\")",
+            };
+
+            var runsettings = "<?xml version=\"1.0\" encoding=\"utf-16\"?>\r\n<RunSettings>\r\n  <DataCollectionRunSettings>\r\n    <DataCollectors />\r\n  </DataCollectionRunSettings>\r\n  <TestRunParameters>\r\n    <Parameter name=\"myParam\" value=\"myValue\" />\r\n    <Parameter name=\"myParam2\" value=\"myValue 2\" />\r\n  </TestRunParameters>\r\n</RunSettings>";
+            this.executor.Initialize(args);
+
+            Assert.IsNotNull(this.settingsProvider.ActiveRunSettings);
+            Assert.AreEqual(runsettings, settingsProvider.ActiveRunSettings.SettingsXml);
+        }
     }
 }