Adding tests, fixes

gsscoder · gsscoder · commit 2ffc2805eda7 · 2015-07-27T08:16:25.000+02:00
diff --git a/src/CommandLine.Tests/Unit/UnParserExtensionsTests.cs b/src/CommandLine.Tests/Unit/UnParserExtensionsTests.cs
@@ -39,6 +39,15 @@ public static void UnParsing_instance_with_fsharp_option_returns_command_line(Fa
                 .ShouldBeEquivalentTo(result);
         }
 
+        [Fact]
+        public static void UnParsing_instance_with_group_switches_returns_command_line_with_switches_grouped()
+        {
+            var options = new FakeOptionsWithSwitches { InputFile = "input.bin", HumanReadable = true, IgnoreWarnings = true };
+            new Parser()
+                .FormatCommandLine(options, config => config.GroupSwitches = true)
+                .ShouldBeEquivalentTo("-hi --input input.bin");
+        }
+
         public static IEnumerable<object> UnParseData
         {            get            {
                 yield return new object[] { new FakeOptions(), "" };
@@ -83,5 +92,6 @@ public static IEnumerable<object> UnParseDataFSharpOption
                 yield return new object[] { new FakeOptionsWithFSharpOption { Offset = FSharpOption<int>.Some(123456789) }, "123456789" };
                 yield return new object[] { new FakeOptionsWithFSharpOption { FileName = FSharpOption<string>.Some("myfile.bin"), Offset = FSharpOption<int>.Some(123456789) }, "--filename myfile.bin 123456789" };
             }
-        }    }
+        }
+    }
 }
diff --git a/src/CommandLine/UnParserExtensions.cs b/src/CommandLine/UnParserExtensions.cs
@@ -115,7 +115,8 @@ orderby v.Index
                 select info;
 
             builder = settings.GroupSwitches && shortSwitches.Any()
-                ? builder.Append('-').Append(shortSwitches.Select(info => ((OptionSpecification)info.Specification).ShortName)).Append(' ')
+                ? builder.Append('-').Append(string.Join(string.Empty, shortSwitches.Select(
+                    info => ((OptionSpecification)info.Specification).ShortName).ToArray())).Append(' ')
                 : builder;
             builder
                 .TrimEndIfMatchWhen(!optSpecs.Any() || builder.TrailingSpaces() > 1, ' ');
