dotnet format fix (#561)

* dotnet format fix

* Fix No_Matching_Placeholder_Positional_Is_Appended

Author: thomhurst

src/ModularPipelines.Build/ReleaseNotes.md

@@ -1 +1 @@
-null
+Fix running `dotnet format` via `context.DotNet().Format()`

src/ModularPipelines.DotNet/Options/DotNetFormatOptions.cs

@@ -10,14 +10,14 @@ public DotNetFormatOptions(
         string projectSolution
     )
     {
-        CommandParts = ["format", "[options]", "[<PROJECT | SOLUTION>]"];
+        CommandParts = ["format", "[<PROJECT | SOLUTION>]"];
 
         ProjectSolution = projectSolution;
     }
 
     public DotNetFormatOptions()
     {
-        CommandParts = ["format", "[options]", "[<PROJECT | SOLUTION>]"];
+        CommandParts = ["format"];
     }
 
     [PositionalArgument(PlaceholderName = "[<PROJECT | SOLUTION>]")]

src/ModularPipelines/Helpers/CommandOptionsObjectArgumentParser.cs

@@ -50,9 +50,10 @@ private static void AddPlaceholderArguments(List<string> precedingArguments, obj
             var indexOfMatchingPrecedingArgumentPlaceholder =
                 precedingArguments.FindIndex(x => x == placeholderName);
 
-            if (indexOfMatchingPrecedingArgumentPlaceholder < 0)
+            if (indexOfMatchingPrecedingArgumentPlaceholder < 0 && !string.IsNullOrEmpty(value))
             {
-                throw new ArgumentException($"No matching placeholder found for property {positionalPlaceholderArgument.Name}");
+                precedingArguments.Add(value);
+                continue;
             }
 
             if (string.IsNullOrWhiteSpace(value) && precedingArguments[indexOfMatchingPrecedingArgumentPlaceholder].StartsWith('<'))

test/ModularPipelines.UnitTests/CommandParserTests.cs

@@ -148,12 +148,14 @@ await Assert.That(() => GetResult(new PlaceholderToolOptions(package!, "MyProjec
     }
 
     [Test]
-    public async Task No_Matching_Placeholder_Positional_Throws()
+    public async Task No_Matching_Placeholder_Positional_Is_Appended()
     {
-        await Assert.That(() => GetResult(new PlaceholderToolOptions2("ThisPackage", "MyProject.csproj")
+        var result = await GetResult(new PlaceholderToolOptions3()
         {
-            Source = "nuget.org"
-        })).Throws.Exception().OfType<ArgumentException>();
+            Project = "MyProject.csproj"
+        });
+        
+        await Assert.That(result.CommandInput).Is.EqualTo("dotnet add MyProject.csproj");
     }
 
     private async Task<CommandResult> GetResult(CommandLineToolOptions options)
@@ -218,4 +220,14 @@ private record PlaceholderToolOptions2(string Package, string Project) : Command
         [CommandSwitch("--source")]
         public string? Source { get; set; }
     }
+    
+    [CommandPrecedingArguments("add")]
+    private record PlaceholderToolOptions3() : CommandLineToolOptions("dotnet")
+    {
+        [PositionalArgument(PlaceholderName = "[<PROJECT>]")]
+        public string? Project { get; set; }
+
+        [CommandSwitch("--source")]
+        public string? Source { get; set; }
+    }
 }

test/ModularPipelines.UnitTests/Helpers/DotNetTests.cs

@@ -21,6 +21,17 @@ private class DotNetVersionModule : Module<CommandResult>
             }, token: cancellationToken);
         }
     }
+    
+    private class DotNetFormatModule : Module<CommandResult>
+    {
+        protected override async Task<CommandResult?> ExecuteAsync(IPipelineContext context, CancellationToken cancellationToken)
+        {
+            return await context.DotNet().Format(new DotNetFormatOptions
+            {
+                ProjectSolution = context.Git().RootDirectory.FindFile(x => x.Name.Contains("TestsForTests")).AssertExists(),
+            }, token: cancellationToken);
+        }
+    }
 
     [Test]
     public async Task Has_Not_Errored()
@@ -36,6 +47,21 @@ public async Task Has_Not_Errored()
             await Assert.That(moduleResult.Value).Is.Not.Null();
         }
     }
+    
+    [Test]
+    public async Task Format_Has_Not_Errored()
+    {
+        var module = await RunModule<DotNetFormatModule>();
+
+        var moduleResult = await module;
+        
+        await using (Assert.Multiple())
+        {
+            await Assert.That(moduleResult.ModuleResultType).Is.EqualTo(ModuleResultType.Success);
+            await Assert.That(moduleResult.Exception).Is.Null();
+            await Assert.That(moduleResult.Value).Is.Not.Null();
+        }
+    }
 
     [Test]
     public async Task Standard_Output_Starts_With_Git_Version()