@teamscale/coverage-collector 0.0.1-beta.32 → 0.0.1-beta.35

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@teamscale/coverage-collector",
-    "version": "0.0.1-beta.32",
+    "version": "0.0.1-beta.35",
     "description": "Collector for JavaScript code coverage information",
     "main": "dist/src/main.js",
     "bin": "dist/src/main.js",
@@ -43,6 +43,7 @@
         "@types/async": "^3.2.6",
         "@types/bunyan": "^1.8.8",
         "@types/jest": "^27.0.1",
+        "@types/mkdirp": "^1.0.2",
         "@types/node": "^15.0.1",
         "@types/source-map": "^0.5.7",
         "@types/tmp": "^0.2.3",

package/dist/src/main.js

@@ -122,7 +122,8 @@ class Main {
         const logfilePath = config.log_to_file.trim();
         mkdirp_1.default.sync(path_1.default.dirname(logfilePath));
         const logLevel = config.log_level;
-        return bunyan_1.default.createLogger({ name: "Instrumenter",
+        return bunyan_1.default.createLogger({
+            name: 'Instrumenter',
             streams: [
                 {
                     level: logLevel,
@@ -137,7 +138,8 @@ class Main {
                     level: logLevel,
                     path: logfilePath
                 }
-            ] });
+            ]
+        });
     }
     /**
      * Entry point of the Teamscale JavaScript Profiler.

package/dist/src/receiver/CollectingServer.d.ts

@@ -1,5 +1,5 @@
 import { IDataStorage } from '../storage/DataStorage';
-import Logger from "bunyan";
+import Logger from 'bunyan';
 /**
  * Various constants that are used to exchange data between
  * the instrumented application and the coverage collector.

package/dist/src/receiver/CollectingServer.js

@@ -133,14 +133,19 @@ class WebSocketCollectingServer {
      */
     handleCoverageMessage(session, body) {
         var _a;
-        const bodyPattern = /(?<fileId>\S+) (?<positions>((\d+:\d+)\s+)*(\d+:\d+))/;
+        const bodyPattern = /(?<fileId>\S+) (?<positions>((\d+:\d+(:\d+:\d+)?\s+)*(\d+:\d+(:\d+:\d+)?)))/;
         const matches = bodyPattern.exec(body);
         if (matches === null || matches === void 0 ? void 0 : matches.groups) {
             const fileId = matches.groups.fileId;
             const positions = ((_a = matches.groups.positions) !== null && _a !== void 0 ? _a : '').split(/\s+/);
             for (const position of positions) {
-                const [line, column] = position.split(':');
-                session.putCoverage(fileId, Number.parseInt(line), Number.parseInt(column));
+                const positionParts = position.split(':');
+                if (positionParts.length === 2) {
+                    session.putCoverage(fileId, Number.parseInt(positionParts[0]), Number.parseInt(positionParts[1]), Number.parseInt(positionParts[1]), Number.parseInt(positionParts[2]));
+                }
+                else if (positionParts.length === 4) {
+                    session.putCoverage(fileId, Number.parseInt(positionParts[0]), Number.parseInt(positionParts[1]), Number.parseInt(positionParts[2]), Number.parseInt(positionParts[3]));
+                }
             }
         }
     }

package/dist/src/receiver/Session.d.ts

@@ -1,7 +1,7 @@
 /// <reference types="node" />
 import { Socket } from 'net';
 import { IDataStorage } from '../storage/DataStorage';
-import Logger from "bunyan";
+import Logger from 'bunyan';
 /**
  * The session maintains the relevant information for a client.
  * One session is created for each client.
@@ -46,7 +46,7 @@ export declare class Session {
      * @param line - The line number within the bundle.
      * @param column - The column within the bundle.
      */
-    putCoverage(fileId: string, line: number, column: number): void;
+    putCoverage(fileId: string, startLine: number, startColumn: number, endLine: number, endColumn: number): void;
     /**
      * Map to the original file position.
      *

package/dist/src/receiver/Session.js

@@ -54,13 +54,50 @@ class Session {
      * @param line - The line number within the bundle.
      * @param column - The column within the bundle.
      */
-    putCoverage(fileId, line, column) {
-        const originalPosition = this.mapToOriginal(fileId, line, column);
-        if (originalPosition.line && originalPosition.source) {
-            this.storage.putCoverage(this.projectId, originalPosition.source, [originalPosition.line]);
-        }
-        else {
-            this.storage.signalUnmappedCoverage(this.projectId);
+    putCoverage(fileId, startLine, startColumn, endLine, endColumn) {
+        var _a, _b;
+        // Iterate over the lines to scan
+        let line = startLine;
+        while (line <= endLine) {
+            // Determine the column range to consider for this line
+            let scanFromColumn;
+            if (line === startLine) {
+                scanFromColumn = startColumn;
+            }
+            else {
+                scanFromColumn = 0;
+            }
+            let scanToColumn;
+            if (line === endLine) {
+                scanToColumn = endColumn;
+            }
+            else {
+                // Since we do not know the length of the different lines, we assume
+                // all to end in the lager of `endColumn` and `startColumn`.
+                // A better estimate (or the correct value) is supposed to be implemented
+                // in context of TS-30077.
+                scanToColumn = Math.max(endColumn, startColumn);
+            }
+            let column = scanFromColumn;
+            let lastCoveredLine = -1;
+            while (column <= scanToColumn) {
+                const originalPosition = this.mapToOriginal(fileId, line, column);
+                if (originalPosition.line && originalPosition.source) {
+                    if (lastCoveredLine !== originalPosition.line) {
+                        this.storage.putCoverage(this.projectId, originalPosition.source, [originalPosition.line]);
+                        lastCoveredLine = originalPosition.line;
+                    }
+                }
+                else {
+                    this.storage.signalUnmappedCoverage(this.projectId);
+                }
+                // Step to the next column to map back to the original.
+                // `originalPosition.name` is the token on the position, that is, if it is present
+                // we increment the column by its length.
+                column = column + Math.max(1, (_b = (_a = originalPosition.name) === null || _a === void 0 ? void 0 : _a.length) !== null && _b !== void 0 ? _b : 1);
+            }
+            // And the next line
+            line++;
         }
     }
     /**

package/dist/src/storage/DataStorage.d.ts

@@ -1,4 +1,4 @@
-import Logger from "bunyan";
+import Logger from 'bunyan';
 /**
  * Lines covered for the specified file.
  */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@teamscale/coverage-collector",
-  "version": "0.0.1-beta.32",
+  "version": "0.0.1-beta.35",
   "description": "Collector for JavaScript code coverage information",
   "main": "dist/src/main.js",
   "bin": "dist/src/main.js",
@@ -43,6 +43,7 @@
     "@types/async": "^3.2.6",
     "@types/bunyan": "^1.8.8",
     "@types/jest": "^27.0.1",
+    "@types/mkdirp": "^1.0.2",
     "@types/node": "^15.0.1",
     "@types/source-map": "^0.5.7",
     "@types/tmp": "^0.2.3",