Merge pull request #307 from dkhunt27/diffToFixed

handled diff.ToFixed too

README.md

@@ -102,7 +102,7 @@ The report is based on the coverage report generated by your test runner.
 - `npm run all`
 - `git commit/push changes`
 - make PR back to main
-- wait for pipelines to finish (test will always finish with an error since this isn't a nx monorepo)
+- wait for pipelines to finish; then merge
 - `git checkout main`
 - `git pull`
 - `git tag v1`

src/comment.ts

@@ -14,7 +14,12 @@ export const buildComment = ({results}: BuildCommentInputs): string => {
     let arrow = ''
     let diffHtml = ''
 
-    if (result.diff !== null) {
+    // when no tests, not sure if output is undefined or 'Unknown'; TODO: add test case
+    if (
+      result.diff !== undefined &&
+      result.diff !== null &&
+      (result.diff as unknown) !== 'Unknown'
+    ) {
       if (result.diff < 0) {
         arrow = '▾'
       } else if (result.diff > 0) {
@@ -35,8 +40,17 @@ export const buildComment = ({results}: BuildCommentInputs): string => {
 
     const htmlResults = tabulate(result.details)
 
-    const coverage =
-      result.coverage === undefined ? 'unknown' : result.coverage.toFixed(2)
+    // when no tests, not sure if is undefined or 'Unknown'; TODO: add test case
+    let coverage
+    if (
+      result.coverage === undefined ||
+      result.coverage === null ||
+      (result.coverage as unknown) === 'Unknown'
+    ) {
+      coverage = 'unknown'
+    } else {
+      coverage = result.coverage.toFixed(2)
+    }
 
     return `${table(
       tbody(tr(th(result.app), th(coverage, '%'), diffHtml))