Merge pull request #867 from pascalberger/FixArgumentParsing2

Fix argument parsing
GitTools · May 18, 2016 · 28b1572 · 28b1572
2 parents 6308f89 + 86bfac3
commit 28b1572
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/GitVersionExe.Tests/ArgumentParserTests.cs b/src/GitVersionExe.Tests/ArgumentParserTests.cs
@@ -291,4 +291,12 @@ public void log_path_can_contain_forward_slash()
         var arguments = ArgumentParser.ParseArguments("-l /some/path");
         arguments.LogFilePath.ShouldBe("/some/path");
     }
+
+    [Test]
+    public void boolean_argument_handling()
+    {
+        var arguments = ArgumentParser.ParseArguments("/nofetch /updateassemblyinfo true");
+        arguments.NoFetch.ShouldBe(true);
+        arguments.UpdateAssemblyInfo.ShouldBe(true);
+    }
 }
diff --git a/src/GitVersionExe/ArgumentParser.cs b/src/GitVersionExe/ArgumentParser.cs
@@ -393,7 +393,7 @@ static bool IsBooleanArgument(string switchName)
                 "nofetch"
             };
 
-            return booleanArguments.Contains(switchName, StringComparer.OrdinalIgnoreCase);
+            return booleanArguments.Contains(switchName.Substring(1), StringComparer.OrdinalIgnoreCase);
         }
     }
 }