Fix DotNetNugetDeleteOptions to use separate positional arguments (#1262)

* Initial plan

* Fix DotNetNugetDeleteOptions to use separate package name and version arguments

Co-authored-by: thomhurst <30480171+thomhurst@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: thomhurst <30480171+thomhurst@users.noreply.github.com>

Author: Copilot

src/ModularPipelines.DotNet/Options/DotNetNugetDeleteOptions.cs

@@ -7,21 +7,27 @@ namespace ModularPipelines.DotNet.Options;
 public record DotNetNugetDeleteOptions : DotNetOptions
 {
     public DotNetNugetDeleteOptions(
-        string packageNamePackageVersion
+        string packageName,
+        string packageVersion
     )
     {
-        CommandParts = ["nuget", "delete", "[<PACKAGE_NAME> <PACKAGE_VERSION>]"];
+        CommandParts = ["nuget", "delete", "<PACKAGE_NAME>", "<PACKAGE_VERSION>"];
 
-        PackageNamePackageVersion = packageNamePackageVersion;
+        PackageName = packageName;
+
+        PackageVersion = packageVersion;
     }
 
     public DotNetNugetDeleteOptions()
     {
-        CommandParts = ["nuget", "delete", "[<PACKAGE_NAME> <PACKAGE_VERSION>]"];
+        CommandParts = ["nuget", "delete", "<PACKAGE_NAME>", "<PACKAGE_VERSION>"];
     }
 
-    [PositionalArgument(PlaceholderName = "[<PACKAGE_NAME> <PACKAGE_VERSION>]")]
-    public string? PackageNamePackageVersion { get; set; }
+    [PositionalArgument(PlaceholderName = "<PACKAGE_NAME>")]
+    public string? PackageName { get; set; }
+
+    [PositionalArgument(PlaceholderName = "<PACKAGE_VERSION>")]
+    public string? PackageVersion { get; set; }
 
     [BooleanCommandSwitch("--force-english-output")]
     public virtual bool? ForceEnglishOutput { get; set; }

test/ModularPipelines.UnitTests/CommandParserTests.cs

@@ -159,6 +159,33 @@ public async Task No_Matching_Placeholder_Positional_Is_Appended()
         await Assert.That(result.CommandInput).IsEqualTo("dotnet add MyProject.csproj");
     }
 
+    [Test]
+    public async Task DotNet_Nuget_Delete_With_Two_Positional_Arguments()
+    {
+        var result = await GetResult(new DotNetNugetDeleteOptions("MyPackageName", "1.0.0"));
+        await Assert.That(result.CommandInput).IsEqualTo("dotnet nuget delete MyPackageName 1.0.0");
+    }
+
+    [Test]
+    public async Task DotNet_Nuget_Delete_With_Source_Option()
+    {
+        var result = await GetResult(new DotNetNugetDeleteOptions("MyPackageName", "1.0.0")
+        {
+            Source = "https://api.nuget.org/v3/index.json"
+        });
+        await Assert.That(result.CommandInput).IsEqualTo("dotnet nuget delete MyPackageName 1.0.0 --source https://api.nuget.org/v3/index.json");
+    }
+
+    [Test]
+    public async Task DotNet_Nuget_Delete_With_ApiKey_Option()
+    {
+        var result = await GetResult(new DotNetNugetDeleteOptions("MyPackageName", "1.0.0")
+        {
+            ApiKey = "my-secret-key"
+        });
+        await Assert.That(result.CommandInput).IsEqualTo("dotnet nuget delete MyPackageName 1.0.0 --api-key my-secret-key");
+    }
+
     private async Task<CommandResult> GetResult(CommandLineToolOptions options)
     {
         var command = await GetService<ICommand>();