Fix #28: Add comprehensive test coverage for previously untested components

src/DotNetApiDiff/Program.cs

@@ -114,7 +114,8 @@ private static string GetBuildTime()
     }
 
     /// <summary>
-    /// Configures dependency injection services
+    /// Configures dependency injection services for the root container
+    /// Only includes services needed for command instantiation
     /// </summary>
     /// <param name="services">Service collection to configure</param>
     public static void ConfigureServices(IServiceCollection services)
@@ -150,45 +151,11 @@ public static void ConfigureServices(IServiceCollection services)
             builder.SetMinimumLevel(logLevel);
         });
 
-        // Register core services
-        services.AddScoped<IAssemblyLoader, AssemblyLoading.AssemblyLoader>();
-        services.AddScoped<IApiExtractor, ApiExtraction.ApiExtractor>();
-        services.AddScoped<IMemberSignatureBuilder, ApiExtraction.MemberSignatureBuilder>();
-        services.AddScoped<ITypeAnalyzer, ApiExtraction.TypeAnalyzer>();
-        services.AddScoped<IDifferenceCalculator, ApiExtraction.DifferenceCalculator>();
-
-        // Register the NameMapper
-        services.AddScoped<INameMapper>(provider =>
-        {
-            var logger = provider.GetRequiredService<ILogger<ApiExtraction.NameMapper>>();
-
-            // Create a default mapping configuration - in real usage this would be loaded from config
-            var mappingConfig = Models.Configuration.MappingConfiguration.CreateDefault();
-            return new ApiExtraction.NameMapper(mappingConfig, logger);
-        });
-
-        // Register the ChangeClassifier
-        services.AddScoped<IChangeClassifier>(provider =>
-        {
-            var logger = provider.GetRequiredService<ILogger<ApiExtraction.ChangeClassifier>>();
-
-            // Create default configurations - in real usage these would be loaded from config
-            var breakingChangeRules = Models.Configuration.BreakingChangeRules.CreateDefault();
-            var exclusionConfig = Models.Configuration.ExclusionConfiguration.CreateDefault();
-
-            return new ApiExtraction.ChangeClassifier(breakingChangeRules, exclusionConfig, logger);
-        });
-
-        // Register the ApiComparer with NameMapper
-        services.AddScoped<IApiComparer, ApiExtraction.ApiComparer>();
-
-        // Register the ReportGenerator
-        services.AddScoped<IReportGenerator, Reporting.ReportGenerator>();
-
-        // Register the ExitCodeManager
+        // Register only services needed for command instantiation
         services.AddSingleton<IExitCodeManager, ExitCodeManager>();
-
-        // Register the GlobalExceptionHandler
         services.AddSingleton<IGlobalExceptionHandler, GlobalExceptionHandler>();
+
+        // Note: Business logic services (AssemblyLoader, ApiExtractor, etc.) will be registered
+        // in the command-specific container to avoid configuration timing issues
     }
 }

src/DotNetApiDiff/Reporting/ConsoleFormatter.cs

@@ -323,7 +323,9 @@
             table.AddColumn("Breaking");
         }
 
-        table.Title = new TableTitle($"[bold {color}]{title}[/] ({changes.Count})");
+        // Ensure color is not empty or null to avoid Spectre Console parsing errors
+        var safeColor = string.IsNullOrWhiteSpace(color) ? "default" : color;
+        table.Title = new TableTitle($"[bold {safeColor}]{title}[/] ({changes.Count})");
         table.Border = TableBorder.Rounded;
 
         // Group changes by element type for better organization
@@ -336,7 +338,7 @@
                 var row = new List<string>
                 {
                     change.ElementType.ToString(),
-                    change.ElementName,
+                    string.IsNullOrWhiteSpace(change.ElementName) ? "[dim]<unnamed>[/]" : change.ElementName,
                     FormatChangeDetails(change)
                 };
 
@@ -368,25 +370,39 @@
 
     private string FormatChangeDetails(ApiDifference change)
     {
-        var details = new StringBuilder(change.Description);
+        // Escape any markup in the description to prevent parsing errors
+        var safeDescription = string.IsNullOrWhiteSpace(change.Description)
+            ? "No description available"
+            : EscapeMarkup(change.Description);
 
-        if (!string.IsNullOrEmpty(change.OldSignature) && !string.IsNullOrEmpty(change.NewSignature))
-        {
-            details.AppendLine();
-            details.AppendLine($"[red]- {change.OldSignature}[/]");
-            details.AppendLine($"[green]+ {change.NewSignature}[/]");
-        }
-        else if (!string.IsNullOrEmpty(change.OldSignature))
+        var details = new StringBuilder(safeDescription);
+
+        if (!string.IsNullOrWhiteSpace(change.OldSignature))
         {
             details.AppendLine();
-            details.AppendLine($"[red]- {change.OldSignature}[/]");
+            details.AppendLine($"[red]- {EscapeMarkup(change.OldSignature)}[/]");
         }
-        else if (!string.IsNullOrEmpty(change.NewSignature))
+
+        if (!string.IsNullOrWhiteSpace(change.NewSignature))
         {
             details.AppendLine();
-            details.AppendLine($"[green]+ {change.NewSignature}[/]");
+            details.AppendLine($"[green]+ {EscapeMarkup(change.NewSignature)}[/]");
         }
 
         return details.ToString();
     }
+
+    /// <summary>
+    /// Escapes markup characters to prevent Spectre Console parsing errors
+    /// </summary>
+    /// <param name="text">The text to escape</param>
+    /// <returns>Escaped text safe for Spectre Console</returns>
+    private string EscapeMarkup(string text)
+    {
+        if (string.IsNullOrEmpty(text))
+            return text;
+
+        // Escape square brackets which are used for markup
+        return text.Replace("[", "[[").Replace("]", "]]");
+    }
 }