From 260e822f0ca8b7592a110f698f8784b25f5de7d7 Mon Sep 17 00:00:00 2001
From: Alex Mendes <alexanderhmendes@gmail.com>
Date: Mon, 2 Nov 2020 17:21:08 +0000
Subject: [PATCH] fix: do not print empty tables if no coverage report

---
 tests/specs/coverage.spec.js | 8 ++++++++
 tests/utils.js               | 2 +-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/tests/specs/coverage.spec.js b/tests/specs/coverage.spec.js
index 934971f..f56ddfc 100644
--- 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
index 6d6ab65..71b568e 100644
--- 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',