Merge pull request #40 from intersystems/ZeroMethods

Fixed Divide By Zero error in results viewer

Author: isc-tleavitt

CHANGELOG.md

@@ -5,6 +5,11 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [3.1.1] - Unreleased
+
+### Fixed
+- #39: Fixed bug where results viewer gave divide by zero error when there were 0 executed methods in the covered code 
+
 ## [3.1.0] - 2024-07-05
 
 ### Added
@@ -25,4 +30,4 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - #25: Fix so the tool works on 2023.1
 
 ## [2.1.3] - 2022-03-30
-- Last released version before CHANGELOG existed.
+- Last released version before CHANGELOG existed.

cls/TestCoverage/UI/AggregateResultViewer.cls

@@ -25,7 +25,7 @@ XData Contents [ XMLNamespace = "http://www.intersystems.com/zen" ]
 				ROUND(100*CoveredLines/ExecutableLines,2) PercentCoverage,
 				ExecutableLines as ExecutableLines,
 				CoveredLines As LinesCovered,
-				ROUND(100*CoveredMethods/ExecutableMethods,2) MethodCoverage,
+				CASE ExecutableMethods WHEN 0 THEN NULL ELSE ROUND(100*CoveredMethods/ExecutableMethods,2) END As MethodCoverage,
 				ExecutableMethods As ExecutableMethods,
 				CoveredMethods As MethodsCovered,
 				'' as Info,
@@ -100,4 +100,3 @@ ClientMethod drawCell(value, row, col) [ Language = javascript ]
 }
 
 }
-

module.xml

@@ -2,7 +2,7 @@
 <Export generator="Cache" version="25">
 <Document name="TestCoverage.ZPM"><Module>
   <Name>TestCoverage</Name>
-  <Version>3.1.0</Version>
+  <Version>3.1.1</Version>
   <Description>Run your typical ObjectScript %UnitTest tests and see which lines of your code are executed. Includes Cobertura-style reporting for use in continuous integration tools.</Description>
   <Packaging>module</Packaging>
   <Resource Name="TestCoverage.PKG" Directory="cls" />