fix: ensure clustering options is detected correctly when subcommands…

… added via AddSubcommand (#393)
natemcmaster · Sep 13, 2020 · 21a73b3 · 21a73b3
1 parent 0c9fe6b
commit 21a73b3
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/src/CommandLineUtils/CommandLineApplication.cs b/src/CommandLineUtils/CommandLineApplication.cs
@@ -458,6 +458,8 @@ public void AddSubcommand(CommandLineApplication subcommand)
                 AssertCommandNameIsUnique(name, null);
             }
 
+            subcommand.Parent = this;
+
             Commands.Add(subcommand);
         }
 

diff --git a/test/CommandLineUtils.Tests/CommandLineProcessorTests.cs b/test/CommandLineUtils.Tests/CommandLineProcessorTests.cs
@@ -159,6 +159,21 @@ public void ItInfersClusterOptionsCannotBeUsed_MultiCharShortNameInSubcommand()
             Assert.False(app.ClusterOptions);
         }
 
+        [Fact]
+        public void ItInfersClusterOptionsCannotBeUsed_MultiCharShortNameViaAddedSubcommand()
+        {
+            var app = new CommandLineApplication();
+            var subcmd = new CommandLineApplication
+            {
+                Name = "sub"
+            };
+            subcmd.Option("-au|--auth", "Verbose output", CommandOptionType.NoValue);
+            app.AddSubcommand(subcmd);
+            Assert.False(app.ClusterOptionsWasSetExplicitly);
+            app.Parse("sub", "-au");
+            Assert.False(app.ClusterOptions);
+        }
+
         [Fact]
         public void ItInfersClusterOptionsCannotBeUsed_MultiCharShortNameAttribute()
         {