@teamscale/coverage-collector 0.1.0-beta.5 → 0.1.0-beta.6

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@teamscale/coverage-collector",
-    "version": "0.1.0-beta.5",
+    "version": "0.1.0-beta.6",
     "description": "Collector for JavaScript code coverage information",
     "main": "dist/src/main.js",
     "bin": "dist/src/main.js",
@@ -22,13 +22,13 @@
         "dist/**/*"
     ],
     "dependencies": {
-        "@cqse/commons": "0.1.0-beta.4",
+        "@cqse/commons": "0.1.0-beta.5",
         "argparse": "^2.0.1",
         "async": "^3.2.5",
         "axios": "^1.6.7",
         "bunyan": "^1.8.15",
         "date-and-time": "^3.1.1",
-        "dotenv": "^16.4.4",
+        "dotenv": "^16.4.5",
         "express": "^4.18.2",
         "form-data": "^4.0.0",
         "mkdirp": "^3.0.1",
@@ -50,7 +50,7 @@
         "@types/tmp": "^0.2.6",
         "@types/ws": "^8.5.10",
         "babel-jest": "^29.7.0",
-        "esbuild": "^0.20.0",
+        "esbuild": "^0.20.1",
         "jest": "^29.7.0",
         "mockttp": "^3.10.1",
         "rimraf": "^5.0.5",

package/dist/src/App.d.ts

@@ -26,9 +26,10 @@ export declare class App {
         stop: () => Promise<void>;
     };
     /**
-     * Starts a timer that dumps statistics on the number of received messages every 30s.
+     * Starts a timer that shows a message every min that no coverage
+     * was received until the opposite is the case.
      */
-    private static startStatsTimer;
+    private static startNoMessageTimer;
     /**
      * Start a timer for dumping the data, depending on the configuration.
      *

package/dist/src/App.js

@@ -103,8 +103,8 @@ class App {
         const serverState = server.start();
         // Optionally, start a timer that dumps the coverage after N seconds
         const dumpTimerState = this.maybeStartDumpTimer(config, storage, logger);
-        // Start a timer that dumps statistics every 30s.
-        const statsTimerState = this.startStatsTimer(logger, server);
+        // Start a timer that informs if no coverage was received within the last minute
+        const statsTimerState = this.startNoMessageTimer(logger, server);
         // Say bye bye on CTRL+C and exit the process
         process.on('SIGINT', async () => {
             // ... and do a final dump before.
@@ -123,13 +123,21 @@ class App {
         };
     }
     /**
-     * Starts a timer that dumps statistics on the number of received messages every 30s.
+     * Starts a timer that shows a message every min that no coverage
+     * was received until the opposite is the case.
      */
-    static startStatsTimer(logger, server) {
+    static startNoMessageTimer(logger, server) {
+        const startTime = Date.now();
         const timer = setInterval(async () => {
             const stats = server.getStatistics();
-            logger.info(`Received ${stats.totalMessages} message since start, ${stats.totalCoverageMessages} with coverage.`);
-        }, 1000 * 30);
+            if (stats.totalCoverageMessages === 0) {
+                logger.info(`No coverage received for ${((Date.now() - startTime) / 1000.0).toFixed(0)}s.`);
+            }
+            else {
+                // We can stop running the timer after we have received the first coverage.
+                clearInterval(timer);
+            }
+        }, 1000 * 60);
         return {
             stop: () => clearInterval(timer)
         };

package/dist/src/receiver/CollectingServer.js

@@ -58,12 +58,12 @@ class WebSocketCollectingServer {
         // Handle new connections from clients
         (_b = this.server) === null || _b === void 0 ? void 0 : _b.on('connection', (webSocket, req) => {
             let session = new Session_1.Session(req.socket, this.storage, this.logger);
-            this.logger.debug(`Connection from: ${req.socket.remoteAddress}`);
+            this.logger.trace(`Connection from: ${req.socket.remoteAddress}`);
             // Handle disconnecting clients
             webSocket.on('close', code => {
                 if (session) {
                     session = null;
-                    this.logger.debug(`Closing with code ${code}`);
+                    this.logger.trace(`Closing with code ${code}`);
                 }
             });
             // Handle incoming messages

package/dist/src/storage/DataStorage.js

@@ -109,7 +109,7 @@ class DataStorage {
             this.coverageByProject.set(project, projectCoverage);
         }
         coveredOriginalLines.forEach(line => projectCoverage === null || projectCoverage === void 0 ? void 0 : projectCoverage.putLine(sourceFilePath, line));
-        this.logger.debug(`Mapped Coverage: ${project} ${uniformPath} ${coveredOriginalLines}`);
+        this.logger.trace(`Mapped Coverage: ${project} ${uniformPath} ${coveredOriginalLines}`);
     }
     /**
      * Normalize the source file names provided by the Web browsers / from the
@@ -127,7 +127,7 @@ class DataStorage {
         // Currently only implemented to log the missing information.
         this.timesUnmappedCoverage++;
         if (this.timesUnmappedCoverage === 1) {
-            this.logger.debug(`Received unmapped coverage for project "${project}"`);
+            this.logger.trace(`Received unmapped coverage for project "${project}"`);
         }
     }
     /**

package/dist/src/upload/ArtifactoryUpload.d.ts

@@ -1,5 +1,5 @@
 import { ConfigParameters } from '../utils/ConfigParameters';
-import Logger from 'bunyan';
+import Logger from "bunyan";
 /**
  * Uploads a coverage file to artifactory with the provided configuration.
  */

package/dist/src/upload/ArtifactoryUpload.js

@@ -17,7 +17,7 @@ async function uploadToArtifactory(config, logger, coverageFile, lines) {
     if (lines === 0) {
         return;
     }
-    logger.info('Preparing upload to Artifactory');
+    logger.debug('Preparing upload to Artifactory');
     const form = (0, CommonUpload_1.prepareFormData)(coverageFile);
     await performArtifactoryUpload(config, form, logger);
 }

package/dist/src/upload/CommonUpload.js

@@ -27,28 +27,35 @@ exports.prepareFormData = prepareFormData;
  * Uploads a coverage file with the provided configuration.
  */
 async function performUpload(url, form, config, uploadFunction, logger) {
+    var _a, _b;
     try {
         const response = await uploadFunction(url, form, config);
-        logger.info(`Upload finished with code ${response.status}.`);
+        logger.debug(`Upload finished with code ${response.status}.`);
     }
     catch (error) {
         if (axios_1.default.isAxiosError(error)) {
+            let userMessage;
+            if (error.message) {
+                logger.error(`Upload error ${(_a = error.status) !== null && _a !== void 0 ? _a : 'UNDEFINED'}: ${error.message}`);
+            }
             if (error.response) {
-                const response = error.response;
-                if (response.status >= 400) {
-                    throw new UploadError(`Upload failed with code ${response.status}: ${response.statusText}. Response Data: ${response.data}`);
-                }
-                else {
-                    logger.info(`Upload with status code ${response.status} finished.`);
-                }
+                // The request was made and the server responded with a status code
+                // that falls out of the range of 2xx
+                logger.error('Upload error response data:', error.response.data);
+                logger.error('Upload error response status:', error.response.status);
+                logger.error('Upload error response headers:', JSON.stringify(error.response.headers));
+                userMessage = `Request failed with status ${error.response.status}: ${(_b = error.response.data) !== null && _b !== void 0 ? _b : ''}`;
             }
             else if (error.request) {
-                throw new UploadError(`Upload request did not receive a response.`);
+                // The request was made but no response was received
+                userMessage = 'No response received for the request.';
             }
-            if (error.message) {
-                logger.debug(`Something went wrong when uploading data: ${error.message}. Details of the error: ${(0, util_1.inspect)(error)}`);
-                throw new UploadError(`Something went wrong when uploading data: ${error.message}`);
+            else {
+                // Something happened in setting up the request that triggered an Error
+                userMessage = 'Request setup failed.';
             }
+            // Provide a more specific message if possible
+            throw new UploadError(`Something went wrong when uploading data: ${userMessage}`);
         }
         throw new UploadError(`Something went wrong when uploading data: ${(0, util_1.inspect)(error)}`);
     }

package/dist/src/upload/TeamscaleUpload.js

@@ -18,7 +18,7 @@ async function uploadToTeamscale(config, logger, coverageFile, lines) {
     if (lines === 0) {
         return;
     }
-    logger.info('Preparing upload to Teamscale');
+    logger.debug('Preparing upload to Teamscale');
     const form = (0, CommonUpload_1.prepareFormData)(coverageFile);
     const queryParameters = prepareQueryParameters(config);
     await performTeamscaleUpload(config, queryParameters, form, logger);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@teamscale/coverage-collector",
-  "version": "0.1.0-beta.5",
+  "version": "0.1.0-beta.6",
   "description": "Collector for JavaScript code coverage information",
   "main": "dist/src/main.js",
   "bin": "dist/src/main.js",
@@ -15,13 +15,13 @@
     "dist/**/*"
   ],
   "dependencies": {
-    "@cqse/commons": "0.1.0-beta.4",
+    "@cqse/commons": "0.1.0-beta.5",
     "argparse": "^2.0.1",
     "async": "^3.2.5",
     "axios": "^1.6.7",
     "bunyan": "^1.8.15",
     "date-and-time": "^3.1.1",
-    "dotenv": "^16.4.4",
+    "dotenv": "^16.4.5",
     "express": "^4.18.2",
     "form-data": "^4.0.0",
     "mkdirp": "^3.0.1",
@@ -43,7 +43,7 @@
     "@types/tmp": "^0.2.6",
     "@types/ws": "^8.5.10",
     "babel-jest": "^29.7.0",
-    "esbuild": "^0.20.0",
+    "esbuild": "^0.20.1",
     "jest": "^29.7.0",
     "mockttp": "^3.10.1",
     "rimraf": "^5.0.5",