Fix coverage to use line coverage instead of method coverage when cre…

…ating badge for coverage results
xoofx · Feb 9, 2024 · 6437a11 · 6437a11
1 parent 7a09f53
commit 6437a11
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/dotnet-releaser/ReleaserApp.BuildAndTests.cs b/src/dotnet-releaser/ReleaserApp.BuildAndTests.cs
@@ -81,12 +81,12 @@ private async Task<bool> BuildAndTest(IDevHosting? devHosting, BuildInformation
             {
                 if (_config.Coverage.Enable)
                 {
-                    var coverageResult = LoadAndDisplayCoverageResults();
+                    var lineCoverageResult = LoadAndDisplayCoverageResults();
 
                     // Publish badge if requested
                     if (devHosting is not null)
                     {
-                        await PublishCoverageToGist(devHosting, buildInfo, coverageResult);
+                        await PublishCoverageToGist(devHosting, buildInfo, lineCoverageResult);
                     }
                 }
 
@@ -160,7 +160,7 @@ private HitCoverage LoadAndDisplayCoverageResults()
 
         _logger.InfoMarkup("Coverage Results:", table);
 
-        return totalMethodRate;
+        return totalLineRate;
     }
 
     private async Task<bool> Build(string projectFile, bool isTestProject)

diff --git a/src/dotnet-releaser/ReleaserApp.Coverage.cs b/src/dotnet-releaser/ReleaserApp.Coverage.cs
@@ -13,7 +13,7 @@ namespace DotNetReleaser;
 
 public partial class ReleaserApp
 {
-    private async Task PublishCoverageToGist(IDevHosting devHosting, BuildInformation buildInfo, HitCoverage coverage)
+    private async Task PublishCoverageToGist(IDevHosting devHosting, BuildInformation buildInfo, HitCoverage lineCoverage)
     {
         if (!_config.Coverage.BadgeUploadToGist || !buildInfo.IsPush) return;
 
@@ -24,7 +24,7 @@ private async Task PublishCoverageToGist(IDevHosting devHosting, BuildInformatio
             return;
         }
 
-        var rate = (int)Math.Round((double)coverage.Rate * 100);
+        var rate = (int)Math.Round((double)lineCoverage.Rate * 100);
 
         // TODO: We could make many of these things configurable (colors, size of the badge, etc.)
         var color = rate switch