Merge pull request #17 from jeankedotcom/summary-error

tonerdo · web-flow · commit afd49c9a6ca3 · 2018-03-26T15:19:58.000+02:00
Check for totalLines to be 0 in CoverageSummary.
diff --git a/src/coverlet.core/CoverageSummary.cs b/src/coverlet.core/CoverageSummary.cs
@@ -32,7 +32,7 @@ public CoverageSummaryResult CalculateSummary()
                     }
                 }
 
-                result.Add(System.IO.Path.GetFileNameWithoutExtension(mod.Key), (linesCovered * 100) / totalLines);
+                result.Add(System.IO.Path.GetFileNameWithoutExtension(mod.Key), totalLines == 0 ? totalLines : (linesCovered * 100) / totalLines);
             }
 
             return result;

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ public CoverageSummaryResult CalculateSummary()`
`32`	`32`	`}`
`33`	`33`	`}`
`34`	`34`
`35`		`- result.Add(System.IO.Path.GetFileNameWithoutExtension(mod.Key), (linesCovered * 100) / totalLines);`
	`35`	`+ result.Add(System.IO.Path.GetFileNameWithoutExtension(mod.Key), totalLines == 0 ? totalLines : (linesCovered * 100) / totalLines);`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`return result;`