Fix another break caused by upgrading from beta 1

Author: jnm2

src/TfvcMigrator.Tests/EntryPointTests.cs

@@ -6,10 +6,45 @@ public static class EntryPointTests
     public static async Task No_System_CommandLine_failure_for_minimal_arguments()
     {
         var arguments = await CommandVerifier.VerifyArgumentsAsync(async () =>
-            await Program.Main(new[] { "http://someurl", "$/SomePath", "--authors", "authors.txt" }));
+            await Program.Main(new[]
+            {
+                "http://someurl",
+                "$/SomePath",
+                "--authors", "authors.txt",
+            }));
 
         arguments[0].ShouldBe(new Uri("http://someurl"));
         arguments[1].ShouldBe("$/SomePath");
         arguments[2].ShouldBe("authors.txt");
     }
+
+    [Test]
+    public static async Task No_System_CommandLine_failure_for_all_arguments()
+    {
+        var arguments = await CommandVerifier.VerifyArgumentsAsync(async () =>
+            await Program.Main(new[]
+            {
+                "http://someurl",
+                "$/SomePath",
+                "--authors", "authors.txt",
+                "--out-dir", "somedir",
+                "--min-changeset", "42",
+                "--max-changeset", "43",
+                "--root-path-changes", "CS1234:$/New/Path", "CS1235:$/Another/Path",
+                "--pat", "somepat",
+            }));
+
+        arguments[0].ShouldBe(new Uri("http://someurl"));
+        arguments[1].ShouldBe("$/SomePath");
+        arguments[2].ShouldBe("authors.txt");
+        arguments[3].ShouldBe("somedir");
+        arguments[4].ShouldBe(42);
+        arguments[5].ShouldBe(43);
+        arguments[6].ShouldBeOfType<ImmutableArray<RootPathChange>>().ShouldBe(new[]
+        {
+            new RootPathChange(1234, "$/New/Path"),
+            new RootPathChange(1235, "$/Another/Path"),
+        });
+        arguments[7].ShouldBe("somepat");
+    }
 }

src/TfvcMigrator/Program.cs

@@ -31,6 +31,7 @@ public static Task Main(string[] args)
                 parseArgument: result => result.Tokens.Select(token => ParseRootPathChange(token.Value)).ToImmutableArray())
             {
                 Arity = ArgumentArity.OneOrMore,
+                AllowMultipleArgumentsPerToken = true,
                 Description = "Followed by one or more arguments with the format CS1234:$/New/Path. Changes the path that is mapped as the Git repository root to a new path during a specified changeset.",
             },
             new Option<string?>("--pat") { Description = "Optional PAT, required to access TFVC repositories hosted on Azure DevOps Services. If not provided Default Client Credentials will be used, these are only suitable for on-premise TFS/Azure DevOps Server." },

src/TfvcMigrator/RootPathChange.cs

@@ -1,7 +1,7 @@
 namespace TfvcMigrator;
 
 [DebuggerDisplay("{ToString(),nq}")]
-public sealed class RootPathChange
+public sealed class RootPathChange : IEquatable<RootPathChange?>
 {
     public RootPathChange(int changeset, string newSourceRootPath)
     {
@@ -18,5 +18,22 @@ public RootPathChange(int changeset, string newSourceRootPath)
     public int Changeset { get; }
     public string NewSourceRootPath { get; }
 
+    public override bool Equals(object? obj)
+    {
+        return Equals(obj as RootPathChange);
+    }
+
+    public bool Equals(RootPathChange? other)
+    {
+        return other != null &&
+               Changeset == other.Changeset &&
+               NewSourceRootPath == other.NewSourceRootPath;
+    }
+
+    public override int GetHashCode()
+    {
+        return HashCode.Combine(Changeset, NewSourceRootPath);
+    }
+
     public override string ToString() => $"CS{Changeset}:{NewSourceRootPath}";
 }