refactor: Simplify handling of no arguments by removing redundant met…

…hod and directly writing help output
lekman · Dec 2, 2024 · 053804e · 053804e
1 parent 9016337
commit 053804e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 15 deletions.
diff --git a/AzureLiquid.Preview/PreviewProcess.cs b/AzureLiquid.Preview/PreviewProcess.cs
@@ -105,7 +105,10 @@ public static PreviewProcess Create(string[] args)
         preview.Output = preview._args.ParsePath(args, "output");
         preview.ShouldWatch = PreviewProcessArguments.HasArgument(args, "watch");
 
-        HandleNoArgumentsPassed(args, preview);
+        if (args.Length == 0)
+        {
+            preview.WriteHelpOutput();
+        }
 
         if (preview.CanRender)
         {
@@ -155,19 +158,6 @@ private static void LogMissingFiles(PreviewProcess preview)
         preview.LogMessage();
     }
 
-    /// <summary>
-    /// Handles the scenario where no arguments are passed to the application.
-    /// </summary>
-    /// <param name="args">The array of arguments passed to the application.</param>
-    /// <param name="preview">The instance of <see cref="PreviewProcess" /> to handle the output.</param>
-    private static void HandleNoArgumentsPassed(string[] args, PreviewProcess preview)
-    {
-        if (args == null || args.Length == 0)
-        {
-            preview.WriteHelpOutput();
-        }
-    }
-
     /// <summary>
     /// Writes the help output.
     /// </summary>

diff --git a/AzureLiquid.Tests/PreviewProcessTests.cs b/AzureLiquid.Tests/PreviewProcessTests.cs
@@ -187,7 +187,8 @@ public void EnsureFileReadExceptionHandling()
     public void EnsureHelpMessageShown()
     {
         // Arrange
-        var instance = PreviewProcess.Create(null!);
+        var empty = new string[0];
+        var instance = PreviewProcess.Create(empty);
 
         // Act
         var result = instance.Render();