fix(tests): align C# unit tests with buildRootCommand parameter signature

Author: DotDev262

tests/WinHome.Tests/CliBuilderTests.cs

@@ -10,13 +10,13 @@ namespace WinHome.Tests;
 
 public class CliBuilderTests
 {
-  private static Func<FileInfo, bool, string?, bool, bool, bool, bool, bool, bool, LogLevel, Task<int>> NoOpRunAction()
+  private static Func<FileInfo, bool, string?, bool, bool, bool, bool, bool, bool, bool, LogLevel, Task<int>> NoOpRunAction()
   {
-    return (_, _, _, _, _, _, _, _, _, _) => Task.FromResult(0);
+    return (_, _, _, _, _, _, _, _, _, _, _) => Task.FromResult(0);
   }
 
   private static RootCommand BuildRootCommand(
-      Func<FileInfo, bool, string?, bool, bool, bool, bool, bool, bool, LogLevel, Task<int>> runAction,
+      Func<FileInfo, bool, string?, bool, bool, bool, bool, bool, bool, bool, LogLevel, Task<int>> runAction,
       Func<FileInfo?, LogLevel, Task<int>>? generateAction = null,
       Func<string, string?, LogLevel, Task<int>>? stateAction = null)
   {
@@ -40,9 +40,10 @@ public async Task RootCommand_BindsOptions()
     bool capturedUpdate = false;
     bool capturedForce = false;
     bool capturedContinueOnError = false;
+    bool capturedAutoInstall = false;
     LogLevel capturedLevel = LogLevel.Info;
 
-    var root = BuildRootCommand((file, dryRun, profile, debug, diff, json, update, force, continueOnError, level) =>
+    var root = BuildRootCommand((file, dryRun, profile, debug, diff, json, update, force, continueOnError, autoInstall, level) =>
     {
       capturedFile = file;
       capturedDryRun = dryRun;
@@ -53,6 +54,7 @@ public async Task RootCommand_BindsOptions()
       capturedUpdate = update;
       capturedForce = force;
       capturedContinueOnError = continueOnError;
+      capturedAutoInstall = autoInstall;
       capturedLevel = level;
       return Task.FromResult(0);
     });
@@ -69,6 +71,7 @@ public async Task RootCommand_BindsOptions()
             "--update",
             "--force",
             "--continue-on-error",
+            "--auto-install-apps",
             "--verbose"
         }).InvokeAsync();
 
@@ -84,6 +87,7 @@ public async Task RootCommand_BindsOptions()
     Assert.True(capturedUpdate);
     Assert.True(capturedForce);
     Assert.True(capturedContinueOnError);
+    Assert.True(capturedAutoInstall);
     Assert.Equal(LogLevel.Trace, capturedLevel);
   }
 
@@ -94,23 +98,26 @@ public async Task RootCommand_BindsAliasOptions()
     bool capturedDryRun = false;
     string? capturedProfile = null;
     bool capturedUpdate = false;
+    bool capturedAutoInstall = false;
 
-    var root = BuildRootCommand((_, dryRun, profile, _, _, _, update, _, _, _) =>
+    var root = BuildRootCommand((_, dryRun, profile, _, _, _, update, _, _, autoInstall, _) =>
     {
       capturedDryRun = dryRun;
       capturedProfile = profile;
       capturedUpdate = update;
+      capturedAutoInstall = autoInstall;
       return Task.FromResult(0);
     });
 
     // Act
-    var exitCode = await root.Parse(new[] { "-d", "-p", "dev", "-u" }).InvokeAsync();
+    var exitCode = await root.Parse(new[] { "-d", "-p", "dev", "-u", "-i" }).InvokeAsync();
 
     // Assert
     Assert.Equal(0, exitCode);
     Assert.True(capturedDryRun);
     Assert.Equal("dev", capturedProfile);
     Assert.True(capturedUpdate);
+    Assert.True(capturedAutoInstall);
   }
 
   [Theory]
@@ -123,7 +130,7 @@ public async Task RootCommand_LogLevel_Mapping(string option, LogLevel expected)
     // Arrange
     LogLevel capturedLevel = LogLevel.Info;
 
-    var root = BuildRootCommand((_, _, _, _, _, _, _, _, _, level) =>
+    var root = BuildRootCommand((_, _, _, _, _, _, _, _, _, _, level) =>
     {
       capturedLevel = level;
       return Task.FromResult(0);
@@ -149,9 +156,10 @@ public async Task RootCommand_DefaultsToFalseAndInfo()
     bool capturedUpdate = true;
     bool capturedForce = true;
     bool capturedContinueOnError = true;
+    bool capturedAutoInstall = true;
     LogLevel capturedLevel = LogLevel.Trace;
 
-    var root = BuildRootCommand((file, dryRun, profile, debug, diff, json, update, force, continueOnError, level) =>
+    var root = BuildRootCommand((file, dryRun, profile, debug, diff, json, update, force, continueOnError, autoInstall, level) =>
     {
       capturedDryRun = dryRun;
       capturedProfile = profile;
@@ -161,6 +169,7 @@ public async Task RootCommand_DefaultsToFalseAndInfo()
       capturedUpdate = update;
       capturedForce = force;
       capturedContinueOnError = continueOnError;
+      capturedAutoInstall = autoInstall;
       capturedLevel = level;
       return Task.FromResult(0);
     });
@@ -178,6 +187,7 @@ public async Task RootCommand_DefaultsToFalseAndInfo()
     Assert.False(capturedUpdate);
     Assert.False(capturedForce);
     Assert.False(capturedContinueOnError);
+    Assert.False(capturedAutoInstall);
     Assert.Equal(LogLevel.Info, capturedLevel);
   }
 
@@ -191,7 +201,7 @@ public async Task RootCommand_DefaultConfig_UsesEnvPath()
       Environment.SetEnvironmentVariable("WINHOME_CONFIG_PATH", "env-config.yaml");
       FileInfo? capturedFile = null;
 
-      var root = BuildRootCommand((file, _, _, _, _, _, _, _, _, _) =>
+      var root = BuildRootCommand((file, _, _, _, _, _, _, _, _, _, _) =>
       {
         capturedFile = file;
         return Task.FromResult(0);
@@ -220,7 +230,7 @@ public async Task RootCommand_DefaultConfig_FallsBackToConfigYaml()
       Environment.SetEnvironmentVariable("WINHOME_CONFIG_PATH", null);
       FileInfo? capturedFile = null;
 
-      var root = BuildRootCommand((file, _, _, _, _, _, _, _, _, _) =>
+      var root = BuildRootCommand((file, _, _, _, _, _, _, _, _, _, _) =>
       {
         capturedFile = file;
         return Task.FromResult(0);
@@ -245,7 +255,7 @@ public async Task RootCommand_RejectsVerboseAndQuiet()
     // Arrange
     bool runCalled = false;
 
-    var root = BuildRootCommand((_, _, _, _, _, _, _, _, _, _) =>
+    var root = BuildRootCommand((_, _, _, _, _, _, _, _, _, _, _) =>
     {
       runCalled = true;
       return Task.FromResult(0);

tests/WinHome.Tests/ShellCompletionGeneratorTests.cs

@@ -41,7 +41,7 @@ private static RootCommand BuildTestRootCommand()
   private static RootCommand BuildRealRootCommand()
   {
     return CliBuilder.BuildRootCommand(
-        runAction: (file, dryRun, profile, debug, diff, json, update, forceReapply, continueOnError, logLevel) => Task.FromResult(0),
+        runAction: (_, _, _, _, _, _, _, _, _, _, _) => Task.FromResult(0),
         generateAction: (output, logLevel) => Task.FromResult(0),
         stateAction: (command, path, logLevel) => Task.FromResult(0)
     );

tests/WinHome.Tests/StateCommandTests.cs

@@ -25,7 +25,7 @@ public StateCommandTests()
   private RootCommand BuildRealCommand(Func<string, string?, LogLevel, Task<int>> stateAction)
   {
     return CliBuilder.BuildRootCommand(
-        runAction: (file, dryRun, profile, debug, diff, json, update, forceReapply, continueOnError, logLevel) => Task.FromResult(0),
+        runAction: (_, _, _, _, _, _, _, _, _, _, _) => Task.FromResult(0),
         generateAction: (output, logLevel) => Task.FromResult(0),
         stateAction: stateAction
     );