fix: cleanup

bgd-labs · Nov 13, 2024 · 9db3e50 · 9db3e50
1 parent 0cd0c0c
commit 9db3e50
Show file tree

Hide file tree

Showing 10 changed files with 9,358 additions and 1,133 deletions.
diff --git a/dist/action.js b/dist/action.js
diff --git a/dist/action.mjs b/dist/action.mjs
diff --git a/dist/lib.d.mts b/dist/lib.d.mts
@@ -30,6 +30,9 @@ type Lcov = {
     };
 }[];
 
-declare function generateCoverageDiff(before: Lcov, after: Lcov): string;
+type Options = {
+    rootUrl: string;
+};
+declare function generateCoverageDiff(before: Lcov, after: Lcov, { rootUrl }: Options): string;
 
 export { generateCoverageDiff };
diff --git a/dist/lib.d.ts b/dist/lib.d.ts
@@ -30,6 +30,9 @@ type Lcov = {
     };
 }[];
 
-declare function generateCoverageDiff(before: Lcov, after: Lcov): string;
+type Options = {
+    rootUrl: string;
+};
+declare function generateCoverageDiff(before: Lcov, after: Lcov, { rootUrl }: Options): string;
 
 export { generateCoverageDiff };
diff --git a/dist/lib.js b/dist/lib.js
@@ -31,7 +31,7 @@ function formatCoverage(coverage) {
   if (coverage > 0.8) color = "orange";
   if (coverage > 0.95) color = "lightgreen";
   if (coverage === 1) color = "green";
-  return "{\\color{" + color + "}" + percent + "%}";
+  return "{\\\\color{" + color + "}" + percent + "\\\\\\\\%}";
 }
 var UP = `\u2191`;
 var DOWN = `\u2193`;
@@ -42,29 +42,29 @@ function formatCoverageLine({ hit, found }, previousCoverage) {
   if (diff)
     formattedString = `^{${diff > 0 ? UP : DOWN}` + new Intl.NumberFormat("en-US", {
       maximumSignificantDigits: 2
-    }).format(diff) + "%}" + formattedString;
-  return `$${formattedString}$<br />$${hit} / ${found}$`;
+    }).format(diff) + "\\\\\\\\%}" + formattedString;
+  return `\\$${formattedString}\\$<br />\\$${hit} / ${found}\\$`;
 }
 function findFile(report, file) {
   return report.find((r) => r.file === file);
 }
 function getCoverage({ hit, found }) {
   return found == 0 ? 1 : hit / found;
 }
-function generateCoverageDiff(before, after) {
+function generateCoverageDiff(before, after, { rootUrl }) {
   let content = "| File | Line Coverage | Function Coverage | Branch Coverage |\n| --- | ---: | ---: | ---: |\n";
   for (const report of after) {
     const previousRunResult = findFile(before, report.file);
     const lineCoverage = formatCoverageLine(
       report.lines,
       previousRunResult ? getCoverage(previousRunResult.lines) : 0
     );
-    const missedLines = report.lines.details.filter((line) => line.hit === 0).map((line) => `[${line.line}](${report.file}#L${line.line})`).join(",");
+    const missedLines = report.lines.details.filter((line) => line.hit === 0).map((line) => `[${line.line}](${rootUrl}${report.file}#L${line.line})`).join(", ");
     const functionCoverage = formatCoverageLine(
       report.functions,
       previousRunResult ? getCoverage(previousRunResult.functions) : 0
     );
-    const missedFunctions = report.functions.details.filter((line) => line.hit === 0).map((line) => `[${line.name}](${report.file}#L${line.line})`).join(",");
+    const missedFunctions = report.functions.details.filter((line) => line.hit === 0).map((line) => `[${line.name}](${rootUrl}${report.file}#L${line.line})`).join("<br />");
     const branchCoverage = formatCoverageLine(
       report.branches,
       previousRunResult ? getCoverage(previousRunResult.branches) : 0

diff --git a/dist/lib.mjs b/dist/lib.mjs
@@ -7,7 +7,7 @@ function formatCoverage(coverage) {
   if (coverage > 0.8) color = "orange";
   if (coverage > 0.95) color = "lightgreen";
   if (coverage === 1) color = "green";
-  return "{\\color{" + color + "}" + percent + "%}";
+  return "{\\\\color{" + color + "}" + percent + "\\\\\\\\%}";
 }
 var UP = `\u2191`;
 var DOWN = `\u2193`;
@@ -18,29 +18,29 @@ function formatCoverageLine({ hit, found }, previousCoverage) {
   if (diff)
     formattedString = `^{${diff > 0 ? UP : DOWN}` + new Intl.NumberFormat("en-US", {
       maximumSignificantDigits: 2
-    }).format(diff) + "%}" + formattedString;
-  return `$${formattedString}$<br />$${hit} / ${found}$`;
+    }).format(diff) + "\\\\\\\\%}" + formattedString;
+  return `\\$${formattedString}\\$<br />\\$${hit} / ${found}\\$`;
 }
 function findFile(report, file) {
   return report.find((r) => r.file === file);
 }
 function getCoverage({ hit, found }) {
   return found == 0 ? 1 : hit / found;
 }
-function generateCoverageDiff(before, after) {
+function generateCoverageDiff(before, after, { rootUrl }) {
   let content = "| File | Line Coverage | Function Coverage | Branch Coverage |\n| --- | ---: | ---: | ---: |\n";
   for (const report of after) {
     const previousRunResult = findFile(before, report.file);
     const lineCoverage = formatCoverageLine(
       report.lines,
       previousRunResult ? getCoverage(previousRunResult.lines) : 0
     );
-    const missedLines = report.lines.details.filter((line) => line.hit === 0).map((line) => `[${line.line}](${report.file}#L${line.line})`).join(",");
+    const missedLines = report.lines.details.filter((line) => line.hit === 0).map((line) => `[${line.line}](${rootUrl}${report.file}#L${line.line})`).join(", ");
     const functionCoverage = formatCoverageLine(
       report.functions,
       previousRunResult ? getCoverage(previousRunResult.functions) : 0
     );
-    const missedFunctions = report.functions.details.filter((line) => line.hit === 0).map((line) => `[${line.name}](${report.file}#L${line.line})`).join(",");
+    const missedFunctions = report.functions.details.filter((line) => line.hit === 0).map((line) => `[${line.name}](${rootUrl}${report.file}#L${line.line})`).join("<br />");
     const branchCoverage = formatCoverageLine(
       report.branches,
       previousRunResult ? getCoverage(previousRunResult.branches) : 0