refactoring

coverlet-coverage · Mar 23, 2021 · 371cbba · 371cbba
1 parent 799794e
commit 371cbba
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 14 deletions.
diff --git a/src/coverlet.core/CoverageDetails.cs b/src/coverlet.core/CoverageDetails.cs
@@ -6,14 +6,14 @@ internal class CoverageDetails
     {
         private double _averageModulePercent;
 
-        public Modules Modules { get; internal set; }
+        public int ModulesCount { get; internal set; }
         public double Covered { get; internal set; }
         public int Total { get; internal set; }
         public double Percent
         {
             get
             {
-                if (Modules?.Count == 0) return 0;
+                if (ModulesCount == 0) return 0;
                 return Total == 0 ? 100D : Math.Floor((Covered / Total) * 10000) / 100;
             }
         }

diff --git a/src/coverlet.core/CoverageSummary.cs b/src/coverlet.core/CoverageSummary.cs
@@ -52,12 +52,12 @@ public CoverageDetails CalculateLineCoverage(Documents documents)
 
         public CoverageDetails CalculateLineCoverage(Modules modules)
         {
-            var details = new CoverageDetails{Modules = modules};
-            var accumPercent = 0.0D;
-
-            if (modules.Count == 0)
+            var details = new CoverageDetails{ModulesCount = modules.Count};
+            if (details.ModulesCount == 0)
                 return details;
 
+            var accumPercent = 0.0D;
+
             foreach (var module in modules)
             {
                 var moduleCoverage = CalculateLineCoverage(module.Value);
@@ -155,12 +155,12 @@ public CoverageDetails CalculateBranchCoverage(Documents documents)
 
         public CoverageDetails CalculateBranchCoverage(Modules modules)
         {
-            var details = new CoverageDetails{ Modules = modules };
-            var accumPercent = 0.0D;
-
-            if (modules.Count == 0)
+            var details = new CoverageDetails { ModulesCount = modules.Count };
+            if (details.ModulesCount == 0)
                 return details;
 
+            var accumPercent = 0.0D;
+
             foreach (var module in modules)
             {
                 var moduleCoverage = CalculateBranchCoverage(module.Value);
@@ -219,12 +219,12 @@ public CoverageDetails CalculateMethodCoverage(Documents documents)
 
         public CoverageDetails CalculateMethodCoverage(Modules modules)
         {
-            var details = new CoverageDetails{ Modules = modules };
-            var accumPercent = 0.0D;
-
-            if (modules.Count == 0)
+            var details = new CoverageDetails { ModulesCount = modules.Count };
+            if (details.ModulesCount == 0)
                 return details;
 
+            var accumPercent = 0.0D;
+
             foreach (var module in modules)
             {
                 var moduleCoverage = CalculateMethodCoverage(module.Value);