Merge pull request #3 from snyk/percentage-feedback

Percentage feedback

Author: Felix Geelhaar

src/lib/stats/stats.ts

@@ -45,12 +45,12 @@ export const calcFileStats = (files: File[]): Stats => {
     },
     unloved: {
       ...unloved,
-      percentage: percentageToFixed(unloved.files, total.files)
+      percentage: percentageToFixed(unloved.lines, total.lines)
     },
     loved: {
       files: total.files - unloved.files,
       lines: total.lines - unloved.lines,
-      percentage: percentageToFixed(total.files - unloved.files, total.files)
+      percentage: percentageToFixed(total.lines - unloved.lines, total.lines)
     },
     owners: Array.from(ownerCount.keys()).map((owner) => {
       const counts = ownerCount.get(owner);
@@ -60,7 +60,7 @@ export const calcFileStats = (files: File[]): Stats => {
         counters: {
           files: counts ? counts.files : 0,
           lines: counts ? counts.lines : 0,
-          percentage: counts ? percentageToFixed(counts.files, total.files) : 0
+          percentage: counts ? percentageToFixed(counts.lines, total.lines) : 0
         },
       };
     }),

src/lib/stats/writer.ts

@@ -13,12 +13,12 @@ export const writer = (stats: Stats, options: { output: OUTPUT_FORMAT }, stream:
       stream.write(`${JSON.stringify(stats)}\n`);
       break;
     case(OUTPUT_FORMAT.CSV):
-      stream.write(`owner,files,lines\n`);
+      stream.write(`owner,files,lines,percentage\n`);
       stream.write(`total,${stats.total.files},${stats.total.lines},${stats.total.percentage}\n`);
-      stream.write(`loved,${stats.loved.files},${stats.loved.lines},${stats.total.percentage}\n`);
-      stream.write(`unloved,${stats.unloved.files},${stats.unloved.lines},${stats.total.percentage}\n`);
+      stream.write(`loved,${stats.loved.files},${stats.loved.lines},${stats.loved.percentage}\n`);
+      stream.write(`unloved,${stats.unloved.files},${stats.unloved.lines},${stats.unloved.percentage}\n`);
       orderedOwners.forEach((owner) => {
-        stream.write(`${owner.owner},${owner.counters.files},${owner.counters.lines}\n`);
+        stream.write(`${owner.owner},${owner.counters.files},${owner.counters.lines},${owner.counters.percentage}\n`);
       });
       break;
     default: