Skip to content

[Bug fix] Put the same code that is in covelet.console to show the same result in 'average' (tonerdo#346) #509

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Aug 3, 2019
Merged

[Bug fix] Put the same code that is in covelet.console to show the same result in 'average' (tonerdo#346) #509

Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
c7f0179
[Bug fix]Changed to calculate based on the average coverage of the mo…
Jun 27, 2019
4c1b6a0
[Bug fix]Changed to calculate based on the average coverage of the mo…
Jun 27, 2019
2744f8e
[Bug fix] Added tests, refactoring and changed number alignment (tone…
Jun 28, 2019
492ecf8
[Bug fix] Clean code and reverted ConsoleTable (aligment), (tonerdo#346)
Jun 29, 2019
91e5a3c
[Bug fix] Refactoring: configuration data called inside the construct…
Jul 1, 2019
6522779
[Bug fix] Format code (tonerdo#346)
Jul 2, 2019
be3a72b
[Bug fix] Format code (tonerdo#346)
Jul 2, 2019
a2da885
[Bug fix] Put the same code that is in covelet.console to show the sa…
Aug 2, 2019
9c90c0a
Merge remote-tracking branch 'upstream/master'
Aug 2, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 12 additions & 4 deletions src/coverlet.msbuild.tasks/CoverageResultTask.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -153,9 +153,17 @@ public override bool Execute()
var summary = new CoverageSummary();
int numModules = result.Modules.Count;

var totalLinePercent = summary.CalculateLineCoverage(result.Modules).Percent;
var totalBranchPercent = summary.CalculateBranchCoverage(result.Modules).Percent;
var totalMethodPercent = summary.CalculateMethodCoverage(result.Modules).Percent;
var linePercentCalculation = summary.CalculateLineCoverage(result.Modules);
var branchPercentCalculation = summary.CalculateBranchCoverage(result.Modules);
var methodPercentCalculation = summary.CalculateMethodCoverage(result.Modules);

var totalLinePercent = linePercentCalculation.Percent;
var totalBranchPercent = branchPercentCalculation.Percent;
var totalMethodPercent = methodPercentCalculation.Percent;

var averageLinePercent = linePercentCalculation.AverageModulePercent;
var averageBranchPercent = branchPercentCalculation.AverageModulePercent;
var averageMethodPercent = methodPercentCalculation.AverageModulePercent;

foreach (var module in result.Modules)
{
Expand All @@ -174,7 +182,7 @@ public override bool Execute()

coverageTable.AddColumn(new[] { "", "Line", "Branch", "Method" });
coverageTable.AddRow("Total", $"{totalLinePercent}%", $"{totalBranchPercent}%", $"{totalMethodPercent}%");
coverageTable.AddRow("Average", $"{totalLinePercent / numModules}%", $"{totalBranchPercent / numModules}%", $"{totalMethodPercent / numModules}%");
coverageTable.AddRow("Average", $"{averageLinePercent}%", $"{averageBranchPercent}%", $"{averageMethodPercent}%");

Console.WriteLine(coverageTable.ToStringAlternative());

Expand Down