fix: added guard against empty coverage (#272)

* fix: added guard against empty coverage * fix: changed bounds check to perClassCovMap.size
forcedotcom · Jan 14, 2022 · 174bfab · 174bfab
1 parent 22c9014
commit 174bfab
Showing 1 changed file with 12 additions and 8 deletions.
diff --git a/packages/apex-node/src/tests/syncTests.ts b/packages/apex-node/src/tests/syncTests.ts
@@ -116,14 +116,18 @@ export class SyncTests {
         apexTestClassIdSet
       );
 
-      result.tests.forEach(item => {
-        const keyCodeCov = `${item.apexClass.id}-${item.methodName}`;
-        const perClassCov = perClassCovMap.get(keyCodeCov);
-        perClassCov.forEach(classCov =>
-          coveredApexClassIdSet.add(classCov.apexClassOrTriggerId)
-        );
-        item.perClassCoverage = perClassCov;
-      });
+      if (perClassCovMap.size > 0) {
+        result.tests.forEach(item => {
+          const keyCodeCov = `${item.apexClass.id}-${item.methodName}`;
+          const perClassCov = perClassCovMap.get(keyCodeCov);
+          if (perClassCov) {
+            perClassCov.forEach(classCov =>
+              coveredApexClassIdSet.add(classCov.apexClassOrTriggerId)
+            );
+            item.perClassCoverage = perClassCov;
+          }
+        });
+      }
 
       const {
         codeCoverageResults,