fix: switch do statement to a for statement and start random number c…

…ount at 1
mcarvin8 · Mar 6, 2024 · 1c29323 · 1c29323
1 parent 8bc11e5
commit 1c29323
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/src/helpers/convertToGenericCoverageReport.ts b/src/helpers/convertToGenericCoverageReport.ts
@@ -33,12 +33,16 @@ export function convertToGenericCoverageReport(data: CoverageData, dxDirectory:
 
       for (const coveredLine of coveredLines) {
         if (coveredLine > totalLines) {
-          let randomLineNumber;
-          do {
-            randomLineNumber = Math.floor(Math.random() * totalLines) + 1;
-          } while (coveredLines.includes(randomLineNumber) || uncoveredLines.includes(randomLineNumber) || randomLines.includes(randomLineNumber));
-          randomLines.push(randomLineNumber);
-          xml += `\t\t<lineToCover lineNumber="${randomLineNumber}" covered="true"/>\n`;
+          for (let randomLineNumber = 1; randomLineNumber <= totalLines; randomLineNumber++) {
+            if (
+              !uncoveredLines.includes(randomLineNumber) &&
+              !coveredLines.includes(randomLineNumber) &&
+              !randomLines.includes(randomLineNumber)
+            ) {
+              xml += `\t\t<lineToCover lineNumber="${randomLineNumber}" covered="true"/>\n`;
+              randomLines.push(randomLineNumber);
+            }
+          }
         } else {
           xml += `\t\t<lineToCover lineNumber="${coveredLine}" covered="true"/>\n`;
         }