refactor: update test project dependencies and improve error handling in SolutionAnalyzer

Author: magic5644

CodeLineCounter.Tests/CodeLineCounter.Tests.csproj

@@ -14,10 +14,10 @@
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
       <PrivateAssets>all</PrivateAssets>
     </PackageReference>
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.12.0" />
+    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.13.0" />
     <PackageReference Include="Moq" Version="4.20.72" />
     <PackageReference Include="xunit" Version="2.9.3" />
-    <PackageReference Include="xunit.runner.visualstudio" Version="3.0.1">
+    <PackageReference Include="xunit.runner.visualstudio" Version="3.0.2">
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
       <PrivateAssets>all</PrivateAssets>
     </PackageReference>

CodeLineCounter.Tests/SolutionAnalyzerTests.cs

@@ -232,11 +232,21 @@ public void export_results_with_valid_input_exports_all_files()
                 var solutionPath = Path.Combine(baseSolutionPath, "CodelineCounter.sln");
                 var format = CoreUtils.ExportFormat.JSON;
                 Console.SetOut(sw);
-                SolutionAnalyzer.ExportResults(result, solutionPath, format, baseSolutionPath);
-                // Assert
-                Assert.True(File.Exists(Path.Combine(baseSolutionPath, "CodelineCounter.CodeMetrics.json")));
-                Assert.True(File.Exists(Path.Combine(baseSolutionPath, "CodelineCounter.CodeDuplications.json")));
-                Assert.True(File.Exists(Path.Combine(baseSolutionPath, "CodelineCounter.Dependencies.dot")));
+
+                try
+                {
+                    SolutionAnalyzer.ExportResults(result, solutionPath, format, baseSolutionPath);
+                    // Assert
+                    Assert.True(File.Exists(Path.Combine(baseSolutionPath, "CodelineCounter.CodeMetrics.json")));
+                    Assert.True(File.Exists(Path.Combine(baseSolutionPath, "CodelineCounter.CodeDuplications.json")));
+                    Assert.True(File.Exists(Path.Combine(baseSolutionPath, "CodelineCounter.Dependencies.dot")));
+                }
+                finally
+                {
+                    File.Delete(Path.Combine(baseSolutionPath, "CodelineCounter.CodeMetrics.json"));
+                    File.Delete(Path.Combine(baseSolutionPath, "CodelineCounter.CodeDuplications.json")); 
+                    File.Delete(Path.Combine(baseSolutionPath, "CodelineCounter.Dependencies.dot"));
+                }
             }
 
 

CodeLineCounter/Services/SolutionAnalyzer.cs

@@ -7,15 +7,23 @@ namespace CodeLineCounter.Services
 {
     public static partial class SolutionAnalyzer
     {
-
         public static void AnalyzeAndExportSolution(string solutionPath, bool verbose, CoreUtils.ExportFormat format, string? outputPath = null)
         {
             try
             {
                 var analysisResult = PerformAnalysis(solutionPath);
                 OutputAnalysisResults(analysisResult, verbose);
                 ExportResults(analysisResult, solutionPath, format, outputPath);
-
+            }
+            catch (UnauthorizedAccessException ex)
+            {
+                Console.Error.WriteLine($"Access denied: {ex.Message}");
+                throw;
+            }
+            catch (FileNotFoundException ex)
+            {
+                Console.Error.WriteLine($"File not found: {ex.Message}");
+                throw;
             }
             catch (Exception ex)
             {
@@ -56,7 +64,7 @@ public static void OutputAnalysisResults(AnalysisResult result, bool verbose)
             }
 
             var percentageDuplication = (result.DuplicatedLines / (double)result.TotalLines) * 100;
-            NumberFormatInfo nfi = new System.Globalization.CultureInfo( "en-US", false ).NumberFormat;
+            NumberFormatInfo nfi = new System.Globalization.CultureInfo("en-US", false).NumberFormat;
 
             Console.WriteLine($"Processing completed, number of source files processed: {result.TotalFiles}");
             Console.WriteLine($"Total lines of code: {result.TotalLines}");
@@ -68,52 +76,23 @@ public static void OutputAnalysisResults(AnalysisResult result, bool verbose)
         public static void ExportResults(AnalysisResult result, string solutionPath, CoreUtils.ExportFormat format, string? outputPath = null)
         {
             string baseFileName = Path.GetFileNameWithoutExtension(solutionPath);
-            
-            // Export metrics
             string metricsFileName = $"{baseFileName}.CodeMetrics.json";
             metricsFileName = CoreUtils.GetExportFileNameWithExtension(metricsFileName, format);
-            if (string.IsNullOrEmpty(outputPath))
-            {
-                outputPath = ".";
-            }   
+            outputPath ??= ".";
 
-            string metricsOutputPath = outputPath != null 
-                ? Path.Combine(outputPath, metricsFileName)
-                : metricsFileName;
-
-            // Export duplications
+            string metricsOutputPath = Path.Combine(outputPath, metricsFileName);
             string duplicationsFileName = $"{baseFileName}.CodeDuplications.json";
             duplicationsFileName = CoreUtils.GetExportFileNameWithExtension(duplicationsFileName, format);
-            string duplicationsOutputPath = outputPath != null 
-                ? Path.Combine(outputPath, duplicationsFileName)
-                : duplicationsFileName;
-            // Export des duplications...
-
-            // Export dependencies graph
+            string duplicationsOutputPath = Path.Combine(outputPath, duplicationsFileName);
             string graphFileName = $"{baseFileName}.Dependencies.dot";
-            string graphOutputPath = outputPath != null 
-                ? Path.Combine(outputPath, graphFileName)
-                : graphFileName;
+            string graphOutputPath = Path.Combine(outputPath, graphFileName);
 
             try
             {
                 Parallel.Invoke(
-                    () => DataExporter.ExportMetrics(
-                        metricsFileName,
-                        outputPath ??".",
-                        result,
-                        solutionPath,
-                        format),
-                    () => DataExporter.ExportDuplications(
-                        duplicationsFileName,
-                        outputPath ?? ".",
-                        result.DuplicationMap,
-                        format),
-                    async () => await DataExporter.ExportDependencies(
-                        graphFileName,
-                        outputPath ?? ".",
-                        result.DependencyList,
-                        format)
+                    () => DataExporter.ExportMetrics(metricsFileName, outputPath, result, solutionPath, format),
+                    () => DataExporter.ExportDuplications(duplicationsFileName, outputPath, result.DuplicationMap, format),
+                    async () => await DataExporter.ExportDependencies(graphFileName, outputPath, result.DependencyList, format)
                 );
 
                 Console.WriteLine($"The data has been exported to {metricsOutputPath}");
@@ -144,6 +123,5 @@ public static void OutputDetailedMetrics(List<NamespaceMetrics> metrics, Diction
                 Console.WriteLine($"Project {projectTotal.Key} has {projectTotal.Value} total lines of code.");
             }
         }
-
     }
-}
+}