monocart-coverage-reports 2.8.4 → 2.8.5

package/README.md

@@ -506,7 +506,7 @@ const coverageOptions = {
     sourceFilter: "**/src/**"
 };
 ```
-Supports multiple patterns:
+Support multiple patterns:
 ```js
 const coverageOptions = {
     entryFilter: {
@@ -1062,8 +1062,8 @@ const coverageOptions = {
 - Node.js 20+
 - VSCode (extensions: eslint/stylelint/vue)
 ```sh
-npm install starfall-cli -g
 npm install
+npx playwright install --with-deps
 
 npm run build
 npm run test
@@ -1082,4 +1082,5 @@ npm run eol
 - [Coverage Gutters](https://github.com/ryanluker/vscode-coverage-gutters) - Display test coverage generated by lcov or xml in VSCode editor.
 
 ## Thanks
-- Special thanks to [@edumserrano](https://github.com/edumserrano)
+- [@bcoe](https://github.com/bcoe)
+- [@edumserrano](https://github.com/edumserrano)

package/README.zh-Hans.md

@@ -1066,8 +1066,8 @@ const coverageOptions = {
 - Node.js 20+
 - VSCode (extensions: eslint/stylelint/vue)
 ```sh
-npm install starfall-cli -g
 npm install
+npx playwright install --with-deps
 
 npm run build
 npm run test
@@ -1086,4 +1086,5 @@ npm run eol
 - [Coverage Gutters](https://github.com/ryanluker/vscode-coverage-gutters) - Display test coverage generated by lcov or xml in VSCode editor.
 
 ## Thanks
-- Special thanks to [@edumserrano](https://github.com/edumserrano)
+- [@bcoe](https://github.com/bcoe)
+- [@edumserrano](https://github.com/edumserrano)

package/lib/converter/collect-source-maps.js

@@ -63,9 +63,14 @@ const resolveSourcesContent = (data, url) => {
 
     const { sources, sourcesContent } = data;
 
-    let failed = false;
+    // sources [1,2,3]
+    // sourcesContent could be [null, null, "content"]
+    // some of contents could be missed
+
+    let hasSourceContent = false;
     sources.forEach((file, i) => {
         if (typeof sourcesContent[i] === 'string') {
+            hasSourceContent = true;
             return;
         }
 
@@ -79,16 +84,18 @@ const resolveSourcesContent = (data, url) => {
             const content = Util.readFileSync(path.resolve(sourcePath));
             if (typeof content === 'string') {
                 sourcesContent[i] = content;
+                hasSourceContent = true;
                 return;
             }
         }
 
-        Util.logDebug(EC.red(`failed to load source content ${file}`));
-        failed = true;
+        sourcesContent[i] = '';
+        Util.logDebug(EC.red(`failed to load source content: ${file}`));
+
     });
 
 
-    if (!failed) {
+    if (hasSourceContent) {
         return data;
     }
 };

package/lib/converter/converter.js

@@ -389,6 +389,13 @@ const checkOriginalRangeCode = (range, state, startEndMap, type) => {
         return false;
     }
 
+    // do not check text for `bytes`
+    // it could be `} ` uncovered in `try { } catch`
+    // instead, use `dedupeCountRanges` to accumulate count
+    if (type === 'bytes') {
+        return true;
+    }
+
     // invalid original code
     // Matches any character that is not a word character from the basic Latin alphabet.
     // Equivalent to [^A-Za-z0-9_]
@@ -400,12 +407,6 @@ const checkOriginalRangeCode = (range, state, startEndMap, type) => {
 
     // type: bytes, functions, branches, statements
 
-    // do not check repeated range for `bytes`
-    // instead, use `dedupeCountRanges` to accumulate count
-    if (type === 'bytes') {
-        return true;
-    }
-
     // check repeated range
     const key = `${start}_${end}`;
     if (startEndMap.has(key)) {

package/lib/index.d.ts

@@ -128,7 +128,7 @@ declare namespace MCR {
         path: string;
         type: "v8" | "istanbul";
         data: any;
-    } | undefined;
+    };
 
 
     export interface MetricsSummary {
@@ -204,7 +204,7 @@ declare namespace MCR {
         watermarks: Watermarks;
         summary: CoverageSummary;
         files: CoverageFile[];
-    } | undefined;
+    };
 
     export type LoggingType = "off" | "error" | "info" | "debug";
     export interface CoverageReportOptions {
@@ -329,7 +329,7 @@ declare namespace MCR {
         onEntry?: (entry: V8CoverageEntry) => Promise<void>;
 
         /** {function} onEnd hook */
-        onEnd?: (coverageResults: CoverageResults) => Promise<void>;
+        onEnd?: (coverageResults: CoverageResults | undefined) => Promise<void>;
 
         [key: string]: any;
     }
@@ -357,7 +357,7 @@ declare namespace MCR {
         /** add coverage data
          * 
          * @param coverageData {array} V8 format, {object} Istanbul format */
-        add: (coverageData: any[] | any) => Promise<AddedResults>;
+        add: (coverageData: any[] | any) => Promise<AddedResults | undefined>;
 
         /**
          * add V8 coverage from a dir
@@ -366,7 +366,7 @@ declare namespace MCR {
         addFromDir: (dir: string) => Promise<void>;
 
         /** generate report */
-        generate: () => Promise<CoverageResults>;
+        generate: () => Promise<CoverageResults | undefined>;
 
         /** check if cache exists */
         hasCache: () => boolean;