fix: do not print empty tables if no coverage report

alexandermendes · Nov 2, 2020 · 260e822 · 260e822
1 parent 8afa0fd
commit 260e822
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/tests/specs/coverage.spec.js b/tests/specs/coverage.spec.js
@@ -402,4 +402,12 @@ describe('Coverage', () => {
     expect(lines).toContain('|[src/one.js](../blob/abc123/src/one.js)|100|100|100|100|:white_check_mark:|');
     expect(lines).toContain('|[src/two.js](../blob/abc123/src/two.js)|100|100|100|100|:white_check_mark:|');
   });
+
+  it('does not report anything if no coverage reported for the files changed', async () => {
+    mockFs();
+
+    await coverage();
+
+    expect(getMarkdownReport()).toBeUndefined();
+  });
 });
diff --git a/tests/utils.js b/tests/utils.js
@@ -36,7 +36,7 @@ export const getFileXml = (filePath, metrics, lines = []) => `
   </file>
 `;
 
-export const getMarkdownReport = () => markdown.mock.calls[0][0];
+export const getMarkdownReport = () => markdown.mock.calls?.[0]?.[0];
 
 export const translateMetric = (metric) => ({
   statements: 'statements',