options should be case sensitive

Author: siywilliams

FluentCommandLineParser.Tests/FluentCommandLineParser/when_a_new_instance_is_created.cs

@@ -34,7 +34,7 @@ public class when_a_new_instance_is_created : FluentCommandLineParserTestContext
 		{
 			It should_create_a_default_parser_engine = () => sut.ParserEngine.ShouldBeOfType(typeof(Fclp.Internals.CommandLineParserEngineMark2));
 			It should_create_a_default_option_factory = () => sut.OptionFactory.ShouldBeOfType(typeof(CommandLineOptionFactory));
-			It should_set_IsCaseSensitive_to_default_value_of_false = () => sut.StringComparison.ShouldEqual(System.StringComparison.CurrentCultureIgnoreCase);
+			It should_set_the_string_comparison_to_current_culture = () => sut.StringComparison.ShouldEqual(System.StringComparison.CurrentCulture);
 			It should_have_setup_no_options_internally = () => sut.Options.ShouldBeEmpty();
 			It should_have_a_default_option_formatter = () => sut.OptionFormatter.ShouldBeOfType(typeof(CommandLineOptionFormatter));
 		}

FluentCommandLineParser.Tests/FluentCommandLineParserTests.cs

@@ -570,7 +570,7 @@ public void Ensure_Defaults_Are_Called_When_Empty_Args_Specified()
 			bool actualBool = false;
 
 			parser.Setup<int>('i').Callback(i => actualInt = i).SetDefault(expectedInt);
-			parser.Setup<string>('s').Callback(s=> actualString = s).SetDefault(expectedString);
+			parser.Setup<string>('s').Callback(s => actualString = s).SetDefault(expectedString);
 			parser.Setup<bool>('b').Callback(b => actualBool = b).SetDefault(expectedBool);
 			parser.Setup<double>('d').Callback(d => actualDouble = d).SetDefault(expectedDouble);
 
@@ -660,7 +660,7 @@ public void Setup_Help_And_Ensure_It_Is_Called_With_Custom_Formatter()
 
 			var formatter = new Mock<ICommandLineOptionFormatter>();
 
-			var args = new[] {"/help", "i", "s"};
+			var args = new[] { "/help", "i", "s" };
 			const string expectedCallbackResult = "blah";
 			string callbackResult = null;
 
@@ -708,6 +708,49 @@ public void Setup_Help_And_Ensure_It_Is_Called()
 
 		#endregion
 
+		#region Case Sensitive
+
+		[Test]
+		public void Ensure_Short_Options_Are_Case_Sensitive()
+		{
+			var parser = CreateFluentParser();
+
+			const string expectedSValue = "my expected value";
+			string SValue;
+			bool sValue = false;
+
+			parser.Setup<string>('S').Callback(str => SValue = str).Required();
+			parser.Setup<bool>('s').Callback(b => sValue = b).Required();
+
+			var result = parser.Parse(new[] { "-S", expectedSValue, "-s" });
+
+			Assert.IsFalse(result.HasErrors);
+			Assert.AreEqual(expectedSValue, expectedSValue);
+			Assert.IsTrue(sValue);
+		}
+
+
+		[Test]
+		public void Ensure_Long_Options_Are_Case_Sensitive()
+		{
+			var parser = CreateFluentParser();
+
+			const string expectedSValue = "my expected value";
+			string SValue;
+			bool sValue = false;
+
+			parser.Setup<string>("LONGOPTION").Callback(str => SValue = str).Required();
+			parser.Setup<bool>("longoption").Callback(b => sValue = b).Required();
+
+			var result = parser.Parse(new[] { "--LONGOPTION", expectedSValue, "--longoption" });
+
+			Assert.IsFalse(result.HasErrors);
+			Assert.AreEqual(expectedSValue, expectedSValue);
+			Assert.IsTrue(sValue);
+		}
+
+		#endregion
+
 		#endregion Top Level Tests
 
 		#region Duplicate Options Tests

FluentCommandLineParser/FluentCommandLineParser.cs

@@ -47,18 +47,12 @@ public class FluentCommandLineParser : IFluentCommandLineParser
 		ICommandLineParserErrorFormatter _errorFormatter;
 		ICommandLineOptionValidator _optionValidator;
 
-		///// <summary>
-		///// Gets or sets whether the parser is case-sensitive. E.g. If <c>true</c> then <c>/a</c> will be treated as identical to <c>/A</c>.
-		///// </summary>
-		//public bool IsCaseSensitive { get; set; }
-
 		/// <summary>
 		/// Gets the <see cref="StringComparison"/> to use when matching values.
 		/// </summary>
 		internal StringComparison StringComparison
 		{
-			//get { return this.IsCaseSensitive ? StringComparison.CurrentCulture : StringComparison.CurrentCultureIgnoreCase; }
-			get { return StringComparison.CurrentCultureIgnoreCase; }
+			get { return StringComparison.CurrentCulture; }
 		}
 
 		/// <summary>