monocart-coverage-reports 2.0.7 → 2.0.9

package/lib/utils/dedupe.js

@@ -62,11 +62,6 @@ const dedupeRanges = (ranges) => {
 // apply to js count ranges
 const dedupeCountRanges = (ranges) => {
 
-    // v8 ranges format
-    // { startOffset: 0, endOffset: 6, count: 0 },
-    // { startOffset: 0, endOffset: 6, count: 1 },
-    // { startOffset: 0, endOffset: 297, count: 1 }
-
     // count ranges format
     // { start: 0, end: 6, count: 0 }
 

package/lib/utils/util.js

@@ -243,6 +243,19 @@ const Util = {
         visitor(rootNode);
     },
 
+    updateOffsetToLocation: (locator, loc) => {
+        const sLoc = locator.offsetToLocation(loc.start);
+        loc.start = {
+            line: sLoc.line,
+            column: sLoc.column
+        };
+        const eLoc = locator.offsetToLocation(loc.end);
+        loc.end = {
+            line: eLoc.line,
+            column: eLoc.column
+        };
+    },
+
     forEachFile: function(dir, extList, callback) {
         if (!fs.existsSync(dir)) {
             return;

package/lib/v8/v8-summary.js

@@ -1,32 +1,7 @@
-// https://playwright.dev/docs/api/class-coverage
-
-// url, text, ranges: [ {start, end} ]
-const getCssSummaryBytes = (item) => {
-
-    const source = item.source;
-    const total = source.length;
-
-    let covered = 0;
-    const bytes = item.data.bytes;
-    if (bytes) {
-        bytes.forEach((range) => {
-            if (range.count > 0) {
-                covered += range.end - range.start;
-            }
-        });
-    }
-
-    // no functions for css
-    return {
-        total,
-        covered
-    };
-
-};
-
-// url, source, ranges:[{start,end, count}]
-const getJsSummaryBytes = (item) => {
+const Util = require('../utils/util.js');
 
+// both css and js
+const getV8SummaryBytes = (item) => {
     const source = item.source;
     const total = source.length;
 
@@ -36,6 +11,10 @@ const getJsSummaryBytes = (item) => {
 
     let endPos = 0;
     uncoveredBytes.forEach((range) => {
+        if (range.ignored) {
+            return;
+        }
+
         const { start, end } = range;
 
         if (start > endPos) {
@@ -63,13 +42,85 @@ const getJsSummaryBytes = (item) => {
     };
 };
 
-const getV8SummaryBytes = (item) => {
-    if (item.type === 'css') {
-        return getCssSummaryBytes(item);
-    }
-    return getJsSummaryBytes(item);
+// calculate uncovered, pct, status
+const calculatePctAndStatus = (item, watermarks) => {
+    Object.keys(item).forEach((k) => {
+        const indicateData = item[k];
+        indicateData.uncovered = indicateData.total - indicateData.covered;
+
+        let pct = '';
+        let status = 'unknown';
+
+        if (indicateData.total) {
+            pct = Util.PNF(indicateData.covered, indicateData.total, 2);
+            status = Util.getStatus(pct, watermarks[k]);
+        }
+
+        indicateData.pct = pct;
+        indicateData.status = status;
+
+    });
+};
+
+const getV8Summary = (v8list, watermarks) => {
+
+    // get bytes summary
+    v8list.forEach((entry) => {
+        entry.summary.bytes = getV8SummaryBytes(entry);
+    });
+
+    // overall summary
+    const summary = {
+        bytes: {
+            total: 0,
+            covered: 0
+        },
+        functions: {
+            total: 0,
+            covered: 0
+        },
+        branches: {
+            total: 0,
+            covered: 0
+        },
+        lines: {
+            total: 0,
+            covered: 0,
+            blank: 0,
+            comment: 0
+        }
+    };
+
+    v8list.forEach((entry) => {
+        const entrySummary = entry.summary;
+        calculatePctAndStatus(entrySummary, watermarks);
+
+        // do NOT add debug file
+        if (entry.debug) {
+            return;
+        }
+
+        Object.keys(entrySummary).forEach((k) => {
+            const indicateData = entrySummary[k];
+            if (!indicateData) {
+                return;
+            }
+            summary[k].total += indicateData.total;
+            summary[k].covered += indicateData.covered;
+
+            if (k === 'lines') {
+                summary[k].blank += indicateData.blank;
+                summary[k].comment += indicateData.comment;
+            }
+        });
+    });
+
+    // calculate overall summary
+    calculatePctAndStatus(summary, watermarks);
+
+    return summary;
 };
 
 module.exports = {
-    getV8SummaryBytes
+    getV8Summary
 };

package/lib/v8/v8.js

@@ -1,6 +1,6 @@
 const path = require('path');
 const Util = require('../utils/util.js');
-const { getV8SummaryBytes } = require('./v8-summary.js');
+const { getV8Summary } = require('./v8-summary.js');
 const { dedupeRanges } = require('../utils/dedupe.js');
 const { getSourcePath } = require('../utils/source-path.js');
 const { mergeScriptCovs } = require('../packages/monocart-coverage-vendor.js');
@@ -221,27 +221,8 @@ const saveV8HtmlReport = async (reportData, options, v8HtmlOptions) => {
     return htmlPath;
 };
 
-// calculate uncovered, pct, status
-const calculatePctAndStatus = (item, watermarks) => {
-    Object.keys(item).forEach((k) => {
-        const indicateData = item[k];
-        indicateData.uncovered = indicateData.total - indicateData.covered;
-
-        let pct = '';
-        let status = 'unknown';
-
-        if (indicateData.total) {
-            pct = Util.PNF(indicateData.covered, indicateData.total, 2);
-            status = Util.getStatus(pct, watermarks[k]);
-        }
-
-        indicateData.pct = pct;
-        indicateData.status = status;
-
-    });
-};
-
 const saveV8Report = async (v8list, options) => {
+
     const defaultWatermarks = {
         bytes: [50, 80],
         functions: [50, 80],
@@ -250,51 +231,7 @@ const saveV8Report = async (v8list, options) => {
     };
     const watermarks = Util.resolveWatermarks(defaultWatermarks, options.watermarks);
 
-    // init summary
-    v8list.forEach((entry) => {
-        entry.summary.bytes = getV8SummaryBytes(entry);
-    });
-
-    // overall summary
-    const summaryList = v8list.map((entry) => entry.summary);
-    const summary = {
-        bytes: {
-            total: 0,
-            covered: 0
-        },
-        functions: {
-            total: 0,
-            covered: 0
-        },
-        branches: {
-            total: 0,
-            covered: 0
-        },
-        lines: {
-            total: 0,
-            covered: 0,
-            blank: 0,
-            comment: 0
-        }
-    };
-    summaryList.forEach((item) => {
-        calculatePctAndStatus(item, watermarks);
-        Object.keys(item).forEach((k) => {
-            const indicateData = item[k];
-            if (!indicateData) {
-                return;
-            }
-            summary[k].total += indicateData.total;
-            summary[k].covered += indicateData.covered;
-
-            if (k === 'lines') {
-                summary[k].blank += indicateData.blank;
-                summary[k].comment += indicateData.comment;
-            }
-        });
-    });
-
-    calculatePctAndStatus(summary, watermarks);
+    const summary = getV8Summary(v8list, watermarks);
 
     const reportData = {
         name: options.name,
@@ -338,7 +275,9 @@ const saveV8Report = async (v8list, options) => {
         return outputPath;
     });
 
-    const report = {
+    const summaryList = v8list.map((entry) => entry.summary);
+
+    const coverageResults = {
         type: 'v8',
         reportPath,
         name: options.name,
@@ -347,7 +286,7 @@ const saveV8Report = async (v8list, options) => {
         files: summaryList
     };
 
-    return report;
+    return coverageResults;
 };
 
 module.exports = {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "monocart-coverage-reports",
-    "version": "2.0.7",
+    "version": "2.0.9",
     "description": "Monocart coverage reports",
     "main": "lib/index.js",
     "exports": {
@@ -17,10 +17,11 @@
         "build-docs": "node ./scripts/build-docs.js",
         "build-test": "node ./scripts/build-test.js",
         "build": "sf lint && sf b -p && npm run build-test",
-        "test-node": "npm run test-node-env && npm run test-node-ins && npm run test-node-api && npm run test-node-cdp",
-        "test-node-env": "cross-env NODE_V8_COVERAGE=.temp/v8-coverage node ./test/test-node-env.js && node ./test/generate-node-report.js",
+        "test-node": "npm run test-node-env && npm run test-node-api && npm run test-node-fgc && npm run test-node-ins && npm run test-node-cdp",
+        "test-node-env": "cross-env NODE_V8_COVERAGE=.temp/v8-coverage-env node ./test/test-node-env.js && node ./test/generate-node-report.js",
+        "test-node-api": "cross-env NODE_V8_COVERAGE=.temp/v8-coverage-api node ./test/test-node-api.js",
+        "test-node-fgc": "node ./test/test-node-fgc.js",
         "test-node-ins": "node ./test/test-node-ins.js",
-        "test-node-api": "cross-env NODE_V8_COVERAGE=.temp/v8-coverage node ./test/test-node-api.js",
         "test-node-cdp": "node --inspect=9229 ./test/test-node-cdp.js",
         "test-browser": "node ./test/test.js",
         "test": "npm run test-browser && npm run test-node && npm run build-docs",