actions/code-coverage-summary
1
0
Fork 0
mirror of https://github.com/irongut/CodeCoverageSummary.git synced 2026-05-20 17:00:14 +02:00
Code Issues Packages Projects Releases Wiki Activity

improved error handling when parsing coverage files #33

Browse Source
This commit is contained in:
irongut
2022-02-19 03:25:34 +00:00
parent 9196cf3448
commit 1c5e7f1a60
1 changed files with 33 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/CodeCoverageSummary/Program.cs
+33 -5
View File
@@ -49,12 +49,16 @@ namespace CodeCoverageSummary
Console.WriteLine($"Code Coverage File: {file}"); Console.WriteLine($"Code Coverage File: {file}");
summary = ParseTestResults(file, summary); summary = ParseTestResults(file, summary);
} }
if (summary == null)
return -2; // error
summary.LineRate /= matchingFiles.Count(); summary.LineRate /= matchingFiles.Count();
summary.BranchRate /= matchingFiles.Count(); summary.BranchRate /= matchingFiles.Count();
if (summary.Packages.Count == 0) if (summary.Packages.Count == 0)
{ {
Console.WriteLine("Error: Parsing code coverage file, no packages found."); Console.WriteLine("Parsing Error: No packages found in coverage files.");
return -2; // error return -2; // error
} }
else else
@@ -97,6 +101,7 @@ namespace CodeCoverageSummary
// output // output
if (o.Output.Equals("console", StringComparison.OrdinalIgnoreCase)) if (o.Output.Equals("console", StringComparison.OrdinalIgnoreCase))
{ {
Console.WriteLine();
Console.WriteLine(output); Console.WriteLine(output);
} }
else if (o.Output.Equals("file", StringComparison.OrdinalIgnoreCase)) else if (o.Output.Equals("file", StringComparison.OrdinalIgnoreCase))
@@ -105,6 +110,7 @@ namespace CodeCoverageSummary
} }
else if (o.Output.Equals("both", StringComparison.OrdinalIgnoreCase)) else if (o.Output.Equals("both", StringComparison.OrdinalIgnoreCase))
{ {
Console.WriteLine();
Console.WriteLine(output); Console.WriteLine(output);
File.WriteAllText($"code-coverage-results.{fileExt}", output); File.WriteAllText($"code-coverage-results.{fileExt}", output);
} }
@@ -134,6 +140,9 @@ namespace CodeCoverageSummary
private static CodeSummary ParseTestResults(string filename, CodeSummary summary) private static CodeSummary ParseTestResults(string filename, CodeSummary summary)
{ {
if (summary == null)
return null;
try try
{ {
string rss = File.ReadAllText(filename); string rss = File.ReadAllText(filename);
@@ -143,19 +152,34 @@ namespace CodeCoverageSummary
var coverage = from item in xdoc.Descendants("coverage") var coverage = from item in xdoc.Descendants("coverage")
select item; select item;
if (!coverage.Any())
throw new Exception("Coverage file invalid, data not found");
var lineR = from item in coverage.Attributes() var lineR = from item in coverage.Attributes()
where item.Name == "line-rate" where item.Name == "line-rate"
select item; select item;
if (!lineR.Any())
throw new Exception("Summary Line Rate not found");
summary.LineRate += double.Parse(lineR.First().Value); summary.LineRate += double.Parse(lineR.First().Value);
var linesCovered = from item in coverage.Attributes() var linesCovered = from item in coverage.Attributes()
where item.Name == "lines-covered" where item.Name == "lines-covered"
select item; select item;
if (!linesCovered.Any())
throw new Exception("Summary Lines Covered not found");
summary.LinesCovered += int.Parse(linesCovered.First().Value); summary.LinesCovered += int.Parse(linesCovered.First().Value);
var linesValid = from item in coverage.Attributes() var linesValid = from item in coverage.Attributes()
where item.Name == "lines-valid" where item.Name == "lines-valid"
select item; select item;
if (!linesValid.Any())
throw new Exception("Summary Lines Valid not found");
summary.LinesValid += int.Parse(linesValid.First().Value); summary.LinesValid += int.Parse(linesValid.First().Value);
var branchR = from item in coverage.Attributes() var branchR = from item in coverage.Attributes()
@@ -169,12 +193,16 @@ namespace CodeCoverageSummary
var branchesCovered = from item in coverage.Attributes() var branchesCovered = from item in coverage.Attributes()
where item.Name == "branches-covered" where item.Name == "branches-covered"
select item; select item;
summary.BranchesCovered += int.Parse(branchesCovered.First().Value);
if (branchesCovered.Any())
summary.BranchesCovered += int.Parse(branchesCovered.First().Value);
var branchesValid = from item in coverage.Attributes() var branchesValid = from item in coverage.Attributes()
where item.Name == "branches-valid" where item.Name == "branches-valid"
select item; select item;
summary.BranchesValid += int.Parse(branchesValid.First().Value);
if (branchesValid.Any())
summary.BranchesValid += int.Parse(branchesValid.First().Value);
} }
// test coverage for individual packages // test coverage for individual packages
@@ -186,7 +214,7 @@ namespace CodeCoverageSummary
{ {
CodeCoverage packageCoverage = new() CodeCoverage packageCoverage = new()
{ {
Name = string.IsNullOrWhiteSpace(item.Attribute("name").Value) ? $"Package {i}" : item.Attribute("name").Value, Name = string.IsNullOrWhiteSpace(item.Attribute("name")?.Value) ? $"Package {i}" : item.Attribute("name").Value,
LineRate = double.Parse(item.Attribute("line-rate")?.Value ?? "0"), LineRate = double.Parse(item.Attribute("line-rate")?.Value ?? "0"),
BranchRate = double.Parse(item.Attribute("branch-rate")?.Value ?? "0"), BranchRate = double.Parse(item.Attribute("branch-rate")?.Value ?? "0"),
Complexity = double.Parse(item.Attribute("complexity")?.Value ?? "0") Complexity = double.Parse(item.Attribute("complexity")?.Value ?? "0")
@@ -200,7 +228,7 @@ namespace CodeCoverageSummary
} }
catch (Exception ex) catch (Exception ex)
{ {
Console.WriteLine($"Parse Error: {ex.Message}"); Console.WriteLine($"Parsing Error: {ex.Message} - {filename}");
return null; return null;
} }
} }