handle Complexity as an int or percentage #9

This commit is contained in:
irongut
2021-09-26 00:58:44 +01:00
parent 6db46e287e
commit 9b11daeca8
4 changed files with 1095 additions and 9 deletions
+2 -2
View File
@@ -10,7 +10,7 @@ namespace CodeCoverageSummary
public double BranchRate { get; set; } public double BranchRate { get; set; }
public int Complexity { get; set; } public double Complexity { get; set; }
} }
public class CodeSummary public class CodeSummary
@@ -27,7 +27,7 @@ namespace CodeCoverageSummary
public int BranchesValid { get; set; } public int BranchesValid { get; set; }
public int Complexity { get; set; } public double Complexity { get; set; }
public List<CodeCoverage> Packages { get; set; } public List<CodeCoverage> Packages { get; set; }
+37 -6
View File
@@ -141,7 +141,7 @@ namespace CodeCoverageSummary
Name = item.Attribute("name").Value, Name = 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 = int.Parse(item.Attribute("complexity")?.Value ?? "0") Complexity = double.Parse(item.Attribute("complexity")?.Value ?? "0")
}; };
summary.Packages.Add(packageCoverage); summary.Packages.Add(packageCoverage);
summary.Complexity += packageCoverage.Complexity; summary.Complexity += packageCoverage.Complexity;
@@ -184,12 +184,27 @@ namespace CodeCoverageSummary
} }
textOutput.AppendLine($"Line Rate = {summary.LineRate * 100:N0}%, Lines Covered = {summary.LinesCovered} / {summary.LinesValid}") textOutput.AppendLine($"Line Rate = {summary.LineRate * 100:N0}%, Lines Covered = {summary.LinesCovered} / {summary.LinesValid}")
.AppendLine($"Branch Rate = {summary.BranchRate * 100:N0}%, Branches Covered = {summary.BranchesCovered} / {summary.BranchesValid}") .AppendLine($"Branch Rate = {summary.BranchRate * 100:N0}%, Branches Covered = {summary.BranchesCovered} / {summary.BranchesValid}");
.AppendLine($"Complexity = {summary.Complexity}");
if (summary.Complexity % 1 == 0)
{
textOutput.AppendLine($"Complexity = {summary.Complexity}");
}
else
{
textOutput.AppendLine($"Complexity = {summary.Complexity * 100:N0}%");
}
foreach (CodeCoverage package in summary.Packages) foreach (CodeCoverage package in summary.Packages)
{ {
textOutput.AppendLine($"{package.Name}: Line Rate = {package.LineRate * 100:N0}%, Branch Rate = {package.BranchRate * 100:N0}%, Complexity = {package.Complexity}"); if (package.Complexity % 1 == 0)
{
textOutput.AppendLine($"{package.Name}: Line Rate = {package.LineRate * 100:N0}%, Branch Rate = {package.BranchRate * 100:N0}%, Complexity = {package.Complexity}");
}
else
{
textOutput.AppendLine($"{package.Name}: Line Rate = {package.LineRate * 100:N0}%, Branch Rate = {package.BranchRate * 100:N0}%, Complexity = {package.Complexity * 100:N0}%");
}
} }
return textOutput.ToString(); return textOutput.ToString();
@@ -210,11 +225,27 @@ namespace CodeCoverageSummary
foreach (CodeCoverage package in summary.Packages) foreach (CodeCoverage package in summary.Packages)
{ {
markdownOutput.AppendLine($"{package.Name} | {package.LineRate * 100:N0}% | {package.BranchRate * 100:N0}% | {package.Complexity}"); if (package.Complexity % 1 == 0)
{
markdownOutput.AppendLine($"{package.Name} | {package.LineRate * 100:N0}% | {package.BranchRate * 100:N0}% | {package.Complexity}");
}
else
{
markdownOutput.AppendLine($"{package.Name} | {package.LineRate * 100:N0}% | {package.BranchRate * 100:N0}% | {package.Complexity * 100:N0}%");
}
} }
markdownOutput.Append($"**Summary** | **{summary.LineRate * 100:N0}%** ({summary.LinesCovered} / {summary.LinesValid}) | ") markdownOutput.Append($"**Summary** | **{summary.LineRate * 100:N0}%** ({summary.LinesCovered} / {summary.LinesValid}) | ")
.AppendLine($"**{summary.BranchRate * 100:N0}%** ({summary.BranchesCovered} / {summary.BranchesValid}) | {summary.Complexity}"); .Append($"**{summary.BranchRate * 100:N0}%** ({summary.BranchesCovered} / {summary.BranchesValid}) | ");
if (summary.Complexity % 1 == 0)
{
markdownOutput.AppendLine($"{summary.Complexity}");
}
else
{
markdownOutput.AppendLine($"{summary.Complexity * 100:N0}%");
}
return markdownOutput.ToString(); return markdownOutput.ToString();
} }
@@ -1,7 +1,8 @@
{ {
"profiles": { "profiles": {
"CodeCoverageSummary": { "CodeCoverageSummary": {
"commandName": "Project" "commandName": "Project",
"commandLineArgs": "../../../../coverage.gcovr.xml --format=md --badge=true"
}, },
"Docker": { "Docker": {
"commandName": "Docker", "commandName": "Docker",
File diff suppressed because it is too large Load Diff