Fix #207 - Additional command names are now case-sensitive (#210)

Author: jcaillon

CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [Unreleased]
+
+Enhancements:
+
+Bugs fixed:
+* Fix [#207](https://github.com/natemcmaster/CommandLineUtils/issues/207) by [@jcaillon]: Option for the case sensitivity of command names
+
 ## [v2.3.1]
 
 Bugs fixed:

src/CommandLineUtils/CommandLineApplication.cs

@@ -29,7 +29,7 @@ public partial class CommandLineApplication : IServiceProvider, IDisposable
         private List<Action<ParseResult>> _onParsingComplete;
         internal readonly Dictionary<string, PropertyInfo> _shortOptions = new Dictionary<string, PropertyInfo>();
         internal readonly Dictionary<string, PropertyInfo> _longOptions = new Dictionary<string, PropertyInfo>();
-        private readonly HashSet<string> _names = new HashSet<string>(StringComparer.Ordinal);
+        private readonly HashSet<string> _names = new HashSet<string>(StringComparer.OrdinalIgnoreCase);
         private string _primaryCommandName;
         internal CommandLineContext _context;
         private IHelpTextGenerator _helpTextGenerator;
@@ -1003,7 +1003,6 @@ public void ShowRootCommandFullNameAndVersion()
 
         internal bool MatchesName(string name)
         {
-            // TODO: make this case-sensitive by default and add a parser setting
             if (string.Equals(name, Name, StringComparison.OrdinalIgnoreCase))
             {
                 return true;

src/CommandLineUtils/releasenotes.props

@@ -1,5 +1,9 @@
 <Project>
   <PropertyGroup>
+    <PackageReleaseNotes Condition="'$(VersionPrefix)' == '2.3.2'">
+Bugs fixed:
+* @jcaillon: Option for the case sensitivity of command names.
+    </PackageReleaseNotes>
     <PackageReleaseNotes Condition="'$(VersionPrefix)' == '2.3.1'">
 Bugs fixed:
 * Fix for InvalidOperationException thrown during help text generation on Mono

test/CommandLineUtils.Tests/CommandLineApplicationTests.cs

@@ -40,6 +40,21 @@ public void CommandNameCanBeMatched()
             Assert.True(called);
         }
 
+        [Fact]
+        public void CommandsNamesAreCaseInsensitive()
+        {
+            var app = new CommandLineApplication();
+            var cmd = app.Command("TEST", c => {
+                c.OnExecute(() => 5);
+            });
+            cmd.AddName("TE");
+
+            Assert.Equal(5, app.Execute("test"));
+            Assert.Equal(5, app.Execute("TEST"));
+            Assert.Equal(5, app.Execute("te"));
+            Assert.Equal(5, app.Execute("TE"));
+        }
+
         [Fact]
         public void RemainingArgsArePassed()
         {