added emojies

Author: marinakalyuzhnaya

dist/main.js

@@ -6024,6 +6024,28 @@ function uncovered(file, options) {
         .join(", ");
 }
 
+/**
+ * Compares two arrays of objects and returns with unique lines update
+ * @param {number} pdiff value from diff percentage
+ * @returns {string} emoji string for negative/positive pdiff
+ */
+const renderEmoji = pdiff => {
+    if (pdiff < 0) return "🔴";
+    if (pdiff >= 0) return "🟢";
+};
+
+/**
+ * Compares two arrays of objects and returns with unique lines update
+ * @param {Array} otherArray
+ * @returns {Function} function with filtering non original lines
+ */
+const comparer = otherArray => current =>
+    otherArray.filter(
+        other =>
+            other.lines.found === current.lines.found &&
+            other.lines.hit === current.lines.hit,
+    ).length === 0;
+
 /**
  * Github comment for monorepo
  * @param {Array<{packageName, lcovPath}>} lcovArrayForMonorepo
@@ -6046,15 +6068,14 @@ function commentForMonorepo(
         const plus = pdiff > 0 ? "+" : "";
         const arrow = pdiff === 0 ? "" : pdiff < 0 ? "▾" : "▴";
 
-        const getColor = () => {
-        	if (pdiff === 0) return 'inherit';
-        	if (pdiff < 0) return 'red';
-        	if (pdiff > 0) return 'green';
-				};
-
-        const diffTh = `<div style="color:${getColor};"}>${th(arrow, " ", plus, pdiff.toFixed(2), "%")}</div>`;
+        const pdiffHtml = baseLcov ? th(renderEmoji(pdiff), " ", arrow, " ", plus, pdiff.toFixed(2), "%") : "";
+				let report = lcovObj.lcov;
 
-        const pdiffHtml = baseLcov ? diffTh : "";
+        if (baseLcov) {
+            const onlyInLcov = lcovObj.lcov.filter(comparer(baseLcov));
+            const onlyInBefore = baseLcov.filter(comparer(lcovObj.lcov));
+            report = onlyInBefore.concat(onlyInLcov);
+        }
 
         return `${table(
             tbody(
@@ -6066,7 +6087,7 @@ function commentForMonorepo(
             ),
         )} \n\n ${details(
             summary("Coverage Report"),
-            tabulate(lcovObj.lcov, options),
+            tabulate(report, options),
         )} <br/>`;
     });
 
@@ -6088,15 +6109,23 @@ function comment(lcov, before, options) {
     const plus = pdiff > 0 ? "+" : "";
     const arrow = pdiff === 0 ? "" : pdiff < 0 ? "▾" : "▴";
 
-    const pdiffHtml = before ? th(arrow, " ", plus, pdiff.toFixed(2), "%") : "";
+		const pdiffHtml = before ? th(renderEmoji(pdiff), " ", arrow, " ", plus, pdiff.toFixed(2), "%") : "";
+
+    let report = lcov;
+
+    if (before) {
+        const onlyInLcov = lcov.filter(comparer(before));
+        const onlyInBefore = before.filter(comparer(lcov));
+        report = onlyInBefore.concat(onlyInLcov);
+    }
 
     return fragment(
         `Coverage after merging ${b(options.head)} into ${b(
             options.base,
         )} <p></p>`,
         table(tbody(tr(th(percentage(lcov).toFixed(2), "%"), pdiffHtml))),
         "\n\n",
-        details(summary("Coverage Report"), tabulate(lcov, options)),
+        details(summary("Coverage Report"), tabulate(report, options)),
     );
 }
 
@@ -6314,7 +6343,7 @@ async function main() {
         commit: context.payload.pull_request.head.sha,
         prefix: `${process.env.GITHUB_WORKSPACE}/`,
         head: context.payload.pull_request.head.ref,
-        base: context.payload.pull_request.base.ref,
+        base: context.payload.pull_request.base.ref
     };
 
     const lcov = !monorepoBasePath && (await parse$1(raw));

src/comment.js

@@ -2,6 +2,28 @@ import { details, summary, b, fragment, table, tbody, tr, th } from "./html";
 import { percentage } from "./lcov";
 import { tabulate } from "./tabulate";
 
+/**
+ * Compares two arrays of objects and returns with unique lines update
+ * @param {number} pdiff value from diff percentage
+ * @returns {string} emoji string for negative/positive pdiff
+ */
+const renderEmoji = pdiff => {
+    if (pdiff < 0) return "🔴";
+    if (pdiff >= 0) return "🟢";
+};
+
+/**
+ * Compares two arrays of objects and returns with unique lines update
+ * @param {Array} otherArray
+ * @returns {Function} function with filtering non original lines
+ */
+const comparer = otherArray => current =>
+    otherArray.filter(
+        other =>
+            other.lines.found === current.lines.found &&
+            other.lines.hit === current.lines.hit,
+    ).length === 0;
+
 /**
  * Github comment for monorepo
  * @param {Array<{packageName, lcovPath}>} lcovArrayForMonorepo
@@ -24,15 +46,14 @@ export function commentForMonorepo(
         const plus = pdiff > 0 ? "+" : "";
         const arrow = pdiff === 0 ? "" : pdiff < 0 ? "▾" : "▴";
 
-        const getColor = () => {
-        	if (pdiff === 0) return 'inherit';
-        	if (pdiff < 0) return 'red';
-        	if (pdiff > 0) return 'green';
-				};
-
-        const diffTh = `<div style="color:${getColor};"}>${th(arrow, " ", plus, pdiff.toFixed(2), "%")}</div>`;
+        const pdiffHtml = baseLcov ? th(renderEmoji(pdiff), " ", arrow, " ", plus, pdiff.toFixed(2), "%") : "";
+				let report = lcovObj.lcov;
 
-        const pdiffHtml = baseLcov ? diffTh : "";
+        if (baseLcov) {
+            const onlyInLcov = lcovObj.lcov.filter(comparer(baseLcov));
+            const onlyInBefore = baseLcov.filter(comparer(lcovObj.lcov));
+            report = onlyInBefore.concat(onlyInLcov);
+        }
 
         return `${table(
             tbody(
@@ -44,7 +65,7 @@ export function commentForMonorepo(
             ),
         )} \n\n ${details(
             summary("Coverage Report"),
-            tabulate(lcovObj.lcov, options),
+            tabulate(report, options),
         )} <br/>`;
     });
 
@@ -66,15 +87,23 @@ export function comment(lcov, before, options) {
     const plus = pdiff > 0 ? "+" : "";
     const arrow = pdiff === 0 ? "" : pdiff < 0 ? "▾" : "▴";
 
-    const pdiffHtml = before ? th(arrow, " ", plus, pdiff.toFixed(2), "%") : "";
+		const pdiffHtml = before ? th(renderEmoji(pdiff), " ", arrow, " ", plus, pdiff.toFixed(2), "%") : "";
+
+    let report = lcov;
+
+    if (before) {
+        const onlyInLcov = lcov.filter(comparer(before));
+        const onlyInBefore = before.filter(comparer(lcov));
+        report = onlyInBefore.concat(onlyInLcov);
+    }
 
     return fragment(
         `Coverage after merging ${b(options.head)} into ${b(
             options.base,
         )} <p></p>`,
         table(tbody(tr(th(percentage(lcov).toFixed(2), "%"), pdiffHtml))),
         "\n\n",
-        details(summary("Coverage Report"), tabulate(lcov, options)),
+        details(summary("Coverage Report"), tabulate(report, options)),
     );
 }
 

src/index.js

@@ -108,7 +108,7 @@ async function main() {
         commit: context.payload.pull_request.head.sha,
         prefix: `${process.env.GITHUB_WORKSPACE}/`,
         head: context.payload.pull_request.head.ref,
-        base: context.payload.pull_request.base.ref,
+        base: context.payload.pull_request.base.ref
     };
 
     const lcov = !monorepoBasePath && (await parse(raw));