Feature Positional parameters (#96)

* Rework input parsing (#94)

* Empty option prefixes are not allowed

* Short options need to have a length of 1

* Return results by yielding

* Add API obsolete safety test

* Add second implementation of argument manager that will work better

* Add command context to keep track of the current command

* Add logging

* Test run with ArgumentManager2 implementation

* Add logging in tests

* Implement input parsing feature v2 fixes #89

* Use new ArgumentManager everywhere

* Remove old ArgumentManager

* Inject IArgumentManager

* Remove non async versions

* Implement order attribute

* Rename to ArgumentManager

* Fix ordered options need to be processed back to back or an exception will be thrown.

* Fix codefactor

* Add more tests

Author: Matthiee

CommandLineParser.Tests/Command/CommandInModelTests.cs

@@ -15,7 +15,7 @@ public CommandInModelTests(ITestOutputHelper testOutputHelper) : base(testOutput
         #region FindCommandsInModel
 
         [Fact]
-        public async Task FindCommandsInModel()
+        public void FindCommandsInModel()
         {
             var parser = new CommandLineParser<ModelWithCommands>(Services);
 
@@ -64,7 +64,7 @@ public override void OnConfigure(ICommandConfigurationBuilder builder)
         #region SubCommandFindCommandsInModel
 
         [Fact]
-        public async Task FindCommandsInCommandModel()
+        public void FindCommandsInCommandModel()
         {
             var parser = new CommandLineParser(Services);
 

CommandLineParser.Tests/Command/SubCommandTests.cs

@@ -15,37 +15,6 @@ public SubCommandTests(ITestOutputHelper testOutputHelper) : base(testOutputHelp
         {
         }
 
-        [Theory]
-        [InlineData(true, "something", 15, -1)]
-        [InlineData(false, "something", 15, -1)]
-        [InlineData(true, "", 15, -1)]
-        public void TestSubCommandWorksCorrectlyInModel(bool autoExecute, string bla, int i, int n)
-        {
-            var lock1 = new ManualResetEventSlim();
-            var lock2 = new ManualResetEventSlim();
-
-            Services.AddSingleton(new MainCommand(lock1, autoExecute, bla, i, n));
-            Services.AddSingleton(new SubCommand(lock2, autoExecute, bla, i, n));
-
-            var parser = new CommandLineParser<MainModel>(Services);
-
-            var result = parser.Parse(new[] { "main", "-b", bla, "sub", "-i", i.ToString(), "-n", n.ToString() });
-
-            result.AssertNoErrors();
-
-            if (!autoExecute)
-            {
-                Assert.All(result.CommandResults.Select(r => r.Executed), Assert.False);
-
-                result.ExecuteCommands();
-            }
-
-            Assert.True(lock1.Wait(1000), "MainCommand didn't execute in time.");
-            Assert.True(lock2.Wait(1000), "SubCommand didn't execute in time.");
-
-            Assert.All(result.CommandResults.Select(r => r.Executed), Assert.True);
-        }
-
         [Theory]
         [InlineData(true, "something", 15, -1)]
         [InlineData(false, "something", 15, -1)]
@@ -68,7 +37,7 @@ public async Task TestSubCommandWorksCorrectlyInModelAsync(bool autoExecute, str
             {
                 Assert.All(result.CommandResults.Select(r => r.Executed), Assert.False);
 
-                result.ExecuteCommands();
+                await result.ExecuteCommandsAsync(default);
             }
 
             Assert.True(lock1.Wait(1000), "MainCommand didn't execute in time.");

CommandLineParser.Tests/CommandLineModelTests.cs

@@ -1,15 +1,20 @@
 using MatthiWare.CommandLine.Core.Attributes;
 
 using Xunit;
+using Xunit.Abstractions;
 
 namespace MatthiWare.CommandLine.Tests
 {
-    public class CommandLineModelTests
+    public class CommandLineModelTests : TestBase
     {
+        public CommandLineModelTests(ITestOutputHelper testOutputHelper) : base(testOutputHelper)
+        {
+        }
+
         [Fact]
         public void TestBasicModel()
         {
-            var parser = new CommandLineParser<Model>();
+            var parser = new CommandLineParser<Model>(Services);
 
             Assert.Equal(1, parser.Options.Count);
 
@@ -31,7 +36,7 @@ public void TestBasicModel()
         [Fact]
         public void TestBasicModelWithOverwritingUsingFluentApi()
         {
-            var parser = new CommandLineParser<Model>();
+            var parser = new CommandLineParser<Model>(Services);
 
             parser.Configure(_ => _.Message)
                 .Required(false)