@testomatio/reporter 1.1.0-beta → 1.2.0-beta

package/lib/_ArtifactStorageOld.js

@@ -7,9 +7,8 @@ const { TESTOMAT_ARTIFACT_SUFFIX } = require('./constants');
 const { specificTestInfo } = require('./util');
 
 class ArtifactStorage {
-
+  
   constructor(params) {
-    this._tmpPrefix = "tsmt_reporter";
     this.isFile = false;
     this.isMemory = true;
     this.storage = params?.toFile;
@@ -21,7 +20,7 @@ class ArtifactStorage {
       // so the solution is make it opposite to isMemory (which will be retrieved from global)
       this.isFile = true;
       this.isMemory = false;
-      this.tmpDirFullpath = this.createTestomatTmpDir(this._tmpPrefix);
+      this.tmpDirFullpath = this.createTestomatTmpDir(ArtifactStorage._tmpPrefix);
       debug('SAVE to tmp folder mode enabled!');
     }
   }
@@ -113,7 +112,7 @@ class ArtifactStorage {
 
   // no need to use multiple dirs; we can implement it later if required
   static tmpTestomatDirNames() {
-    const subname = this._tmpPrefix || "tsmt_reporter";
+    const subname = ArtifactStorage._tmpPrefix || "tsmt_reporter";
 
     return fs.readdirSync(os.tmpdir(), { withFileTypes: true })
       .filter((item) => item.isDirectory())
@@ -138,4 +137,6 @@ class ArtifactStorage {
   }
 }
 
+ArtifactStorage._tmpPrefix = "tsmt_reporter";
+
 module.exports = ArtifactStorage;

package/lib/logger.js

@@ -1,7 +1,18 @@
+const chalk = require('chalk');
 const debug = require('debug')('@testomatio/reporter:logger');
 const { DataStorage } = require('./dataStorage');
 
 const LOG_METHODS = ['assert', 'debug', 'error', 'info', 'log', 'trace', 'warn'];
+const LEVELS = {
+  ALL: { severity: 1, color: '' },
+  VERBOSE: { severity: 3, color: 'grey' },
+  TRACE: { severity: 5, color: 'grey' },
+  DEBUG: { severity: 7, color: 'cyan' },
+  INFO: { severity: 9, color: 'black' },
+  LOG: { severity: 11, color: 'black' },
+  WARN: { severity: 13, color: 'yellow' },
+  ERROR: { severity: 15, color: 'red' },
+};
 
 /**
  * Logger allows to:
@@ -14,7 +25,11 @@ class Logger {
   // _loggerToIntercept intercepted and reassigned immediately when added
 
   constructor(params = {}) {
+    // set default logger to be used in log, warn, error, etc methods
+    this._originalUserLogger = { ...console };
+
     this.dataStorage = new DataStorage('log', params);
+    this.logLevel = process?.env?.LOG_LEVEL?.toUpperCase() || 'ALL';
 
     // commented because prefer to use "intercept" method
     // if (params?.logger) this._loggerToIntercept = params.logger;
@@ -65,22 +80,38 @@ class Logger {
           // it looks like: `string1 arg1 string2 arg2 string3`
           (args[index] !== undefined // eslint-disable-line no-nested-ternary
             ? typeof args[index] === 'string'
-              ? // add arg as it is
-                args[index]
-              : // stringify arg
-                this._strinfifyLogs(args[index])
-            : // add space if no arg after string
-              ' '),
+              ? args[index] // add arg as it is
+              : this._strinfifyLogs(args[index]) // stringify arg
+            : ' '), // add space if no arg after string
         // initial accumulator value
         '',
       );
     } else {
       // this block means arguments syntax is used (syntax like $('text', someVar))
+      // in this case strings represents just a first argument
       logs = this._strinfifyLogs(strings, ...args);
     }
     this.dataStorage.putData(logs);
   }
 
+  /**
+   * Allows you to define a step inside a test. Step name is attached to the report and
+   * helps to understand the test flow.
+   * @param {*} strings 
+   * @param  {...any} values 
+   */
+  step(strings, ...values) {
+    let logs = '';
+    for (let i = 0; i < strings.length; i++) {
+      logs += strings[i];
+      if (i < values.length) {
+        logs += values[i];
+      }
+    }
+    logs = chalk.blue(`> ${logs}`);
+    this.dataStorage.putData(logs);
+  }
+
   /**
    *
    * @param {*} context testId or test context from test runner
@@ -97,13 +128,23 @@ class Logger {
       if (typeof arg === 'string') {
         logs.push(arg);
       } else {
-        logs.push(JSON.stringify(arg));
+        try {
+          // eslint-disable-next-line no-unused-expressions
+          this.prettyObjects ? logs.push(JSON.stringify(arg, null, 2)) : logs.push(JSON.stringify(arg));
+        } catch (e) {
+          debug('Error while stringify object', e);
+          logs.push(arg);
+        }
       }
     }
     return logs.join(' ');
   }
 
   assert(...args) {
+    const level = 'ERROR';
+    const severity = LEVELS[level].severity;
+    if (severity < LEVELS[this.logLevel]?.severity) return;
+
     const logs = this._strinfifyLogs(...args);
     this.dataStorage.putData(logs);
     try {
@@ -114,8 +155,14 @@ class Logger {
   }
 
   debug(...args) {
+    const level = 'DEBUG';
+    const severity = LEVELS[level].severity;
+    if (severity < LEVELS[this.logLevel]?.severity) return;
+
+    if (this.logLevel === 'error' || this.logLevel === 'warn') return;
     const logs = this._strinfifyLogs(...args);
-    this.dataStorage.putData(logs);
+    const colorizedLogs = chalk[LEVELS[level].color](logs);
+    this.dataStorage.putData(colorizedLogs);
     try {
       this._originalUserLogger.debug(...args);
     } catch (e) {
@@ -124,8 +171,13 @@ class Logger {
   }
 
   error(...args) {
+    const level = 'ERROR';
+    const severity = LEVELS[level].severity;
+    if (severity < LEVELS[this.logLevel]?.severity) return;
+
     const logs = this._strinfifyLogs(...args);
-    this.dataStorage.putData(logs);
+    const colorizedLogs = chalk[LEVELS[level].color](logs);
+    this.dataStorage.putData(colorizedLogs);
     try {
       this._originalUserLogger.error(...args);
     } catch (e) {
@@ -134,6 +186,10 @@ class Logger {
   }
 
   info(...args) {
+    const level = 'INFO';
+    const severity = LEVELS[level].severity;
+    if (severity < LEVELS[this.logLevel]?.severity) return;
+
     const logs = this._strinfifyLogs(...args);
     this.dataStorage.putData(logs);
     try {
@@ -144,6 +200,10 @@ class Logger {
   }
 
   log(...args) {
+    const level = 'INFO';
+    const severity = LEVELS[level].severity;
+    if (severity < LEVELS[this.logLevel]?.severity) return;
+
     const logs = this._strinfifyLogs(...args);
     this.dataStorage.putData(logs);
     try {
@@ -154,8 +214,13 @@ class Logger {
   }
 
   trace(...args) {
+    const level = 'TRACE';
+    const severity = LEVELS[level].severity;
+    if (severity < LEVELS[this.logLevel]?.severity) return;
+
     const logs = this._strinfifyLogs(...args);
-    this.dataStorage.putData(logs);
+    const colorizedLogs = chalk[LEVELS[level].color](logs);
+    this.dataStorage.putData(colorizedLogs);
     try {
       this._originalUserLogger.trace(...args);
     } catch (e) {
@@ -164,8 +229,13 @@ class Logger {
   }
 
   warn(...args) {
+    const level = 'WARN';
+    const severity = LEVELS[level].severity;
+    if (severity < LEVELS[this.logLevel]?.severity) return;
+
     const logs = this._strinfifyLogs(...args);
-    this.dataStorage.putData(logs);
+    const colorizedLogs = chalk[LEVELS[level].color](logs);
+    this.dataStorage.putData(colorizedLogs);
     try {
       this._originalUserLogger.warn(...args);
     } catch (e) {
@@ -202,6 +272,20 @@ class Logger {
       this._loggerToIntercept[method] = (...args) => this[method](...args);
     }
   }
+
+  /**
+   * Allows to configure logger. Make sure you do it before the logger usage in your code.
+   *
+   * @param {Object} [config={}] - The configuration object.
+   * @param {string} [config.logLevel] - The desired log level. Valid values are 'DEBUG', 'INFO', 'WARN', and 'ERROR'.
+   * @param {boolean} [config.prettyObjects] - Specifies whether to enable pretty printing of objects.
+   * @returns {void}
+   */
+  configure(config = {}) {
+    if (!config) return;
+    if (config.prettyObjects) this.prettyObjects = config.prettyObjects;
+    if (config.logLevel) this.logLevel = config.logLevel.toUpperCase();
+  }
 }
 
 Logger.instance = null;
@@ -210,12 +294,8 @@ Logger.instance = null;
 module.exports = new Logger();
 
 // TODO: consider using fs promises instead of writeSync/appendFileSync to prevent blocking and improve performance
-// TODO: try to use method generator *
-// TODO: understand captureStackTrace in output (check on .error) (NOT SUCH IMPORTANT)
-// TODO: handle log levels to colorize logs later on UI
-// TODO: parse passed arguments as {level: 'str', message: 'str'} because some loggers use such syntax
-// TODO: add setLevel method - allows skip messages with the lower level
+// TODO: parse passed arguments as {level: 'str', message: 'str'} because some loggers use such syntax;
+// upd: did not face such loggers, but still could be useful
+
 // TODO: in case of unset _originalUserLogger, logger.{method} will not provide console output,
 // need to add some logger by default
-
-// TODO step method (blue color)

package/lib/reporter.js

@@ -4,10 +4,12 @@ const TRConstants = require('./constants');
 const TRArtifacts = require('./_ArtifactStorageOld');
 
 const log = logger._log.bind(logger);
+const step = logger.step.bind(logger);
 
 module.exports = {
   logger,
   log,
+  step,
   TestomatClient,
   TRConstants,
   TRArtifacts,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testomatio/reporter",
-  "version": "1.1.0-beta",
+  "version": "1.2.0-beta",
   "description": "Testomatio Reporter Client",
   "main": "./lib/reporter.js",
   "typings": "typings/index.d.ts",