fix(ConsoleService): consume enum tokens in sequential command args (#1105)

Enum parameters were parsed from args[0] without removing the token. Sequential commands such as broadcast getdata TX <hash> then fed TX into UInt256.Parse.

Co-authored-by: Shargon <shargon@gmail.com>

Author: cschuchardt88

src/Neo.ConsoleService/ConsoleServiceBase.cs

@@ -200,7 +200,7 @@ private bool TryProcessValue(Type parameterType, IList<CommandToken> args, bool
 
             if (parameterType.IsEnum)
             {
-                value = Enum.Parse(parameterType, args[0].Value, true);
+                value = Enum.Parse(parameterType, consumeAll ? args.ConsumeAll() : args.Consume(), true);
                 return true;
             }
         }

tests/Neo.ConsoleService.Tests/UT_CommandServiceBase.cs

@@ -28,6 +28,9 @@ public void TestMethod(string strParam, uint intParam, bool boolParam, string op
         [ConsoleCommand("testenum", Category = "Test Commands")]
         public void TestEnumMethod(TestEnum enumParam) { }
 
+        [ConsoleCommand("testenumrest", Category = "Test Commands")]
+        public void TestEnumThenString(TestEnum enumParam, string rest) { }
+
         [ConsoleCommand("testversion", Category = "Test Commands")]
         public Version TestMethodVersion() { return new Version("1.0.0"); }
 
@@ -133,6 +136,14 @@ public void TestParseSequentialArguments()
         Assert.HasCount(1, result3);
         Assert.AreEqual(TestConsoleService.TestEnum.Value1, result3[0]);
 
+        // Sequential enum must consume the token so later args parse.
+        var enumRestMethod = typeof(TestConsoleService).GetMethod("TestEnumThenString");
+        var argsEnumRest = "testenumrest Value2 leftover".Tokenize();
+        var resultEnumRest = service.ParseSequentialArguments(enumRestMethod, argsEnumRest[1..].Trim());
+        Assert.HasCount(2, resultEnumRest);
+        Assert.AreEqual(TestConsoleService.TestEnum.Value2, resultEnumRest[0]);
+        Assert.AreEqual("leftover", resultEnumRest[1]);
+
         // Test case 4: Missing required parameter should throw exception
         var args4 = "test hello".Tokenize();
         Assert.ThrowsExactly<ArgumentException>(() => service.ParseSequentialArguments(method, args4[1..].Trim()));