Allow nested searches for feature values

Author: mnadareski

SabreTools.CommandLine.Test/CommandSetTests.cs

@@ -188,7 +188,7 @@ public void GetFeature_Exists_Returns()
         }
 
         [Fact]
-        public void GetFeature_NestedExists_Null()
+        public void GetFeature_NestedExists_Returns()
         {
             var commandSet = new CommandSet();
             var child = new MockUserInput("b", "b", "b");
@@ -200,7 +200,9 @@ public void GetFeature_NestedExists_Null()
             subChild.ProcessInput(["c"], ref index);
 
             Feature? actual = commandSet.GetFeature("c");
-            Assert.Null(actual);
+            Assert.NotNull(actual);
+            Assert.Equal("c", actual.Name);
+            Assert.True(actual.Value);
         }
 
         #endregion

SabreTools.CommandLine.Test/Inputs/UserInputTests.cs

@@ -154,7 +154,7 @@ public void GetFeature_Exists_Returns()
         }
 
         [Fact]
-        public void GetFeature_NestedExists_Null()
+        public void GetFeature_NestedExists_Returns()
         {
             UserInput userInput = new MockUserInput("a", "a", "a");
             var child = new MockUserInput("b", "b", "b");
@@ -166,7 +166,9 @@ public void GetFeature_NestedExists_Null()
             subChild.ProcessInput(["c"], ref index);
 
             Feature? actual = userInput.GetFeature("c");
-            Assert.Null(actual);
+            Assert.NotNull(actual);
+            Assert.Equal("c", actual.Name);
+            Assert.True(actual.Value);
         }
 
         #endregion

SabreTools.CommandLine/CommandSet.cs

@@ -325,7 +325,13 @@ public bool TryGetFeature(string key, out Feature? value)
                 return true;
             }
 
-            // TODO: Investigate if nested features should be supported
+            // Check all children recursively
+            foreach (var child in _inputs.Values)
+            {
+                if (child.TryGetFeature(key, out value))
+                    return true;
+            }
+
             value = null;
             return false;
         }

SabreTools.CommandLine/Inputs/UserInput.cs

@@ -343,7 +343,13 @@ public bool TryGetFeature(string key, out Feature? value)
                 return true;
             }
 
-            // TODO: Investigate if nested features should be supported
+            // Check all children recursively
+            foreach (var child in Children.Values)
+            {
+                if (child.TryGetFeature(key, out value))
+                    return true;
+            }
+
             value = null;
             return false;
         }