@testomatio/reporter 2.0.0-beta-esm → 2.0.0-beta.1-xml

package/src/reporter.js

@@ -1,10 +1,25 @@
-import TestomatClient from './client.js';
-import * as TRConstants from './constants.js';
+// import TestomatClient from './client.js';
+// import * as TRConstants from './constants.js';
 import { services } from './services/index.js';
 import reporterFunctions from './reporter-functions.js';
 
+export const artifact = reporterFunctions.artifact;
+export const log = reporterFunctions.log;
+export const logger = services.logger;
+export const meta = reporterFunctions.keyValue;
+export const step = reporterFunctions.step;
+
+/**
+ * @typedef {import('./reporter-functions.js')} artifact
+ * @typedef {import('./reporter-functions.js')} log
+ * @typedef {import('./services/index.js')} logger
+ * @typedef {import('./reporter-functions.js')} meta
+ * @typedef {import('./reporter-functions.js')} step
+ */
 export default {
-  // TODO: deprecate in future; use log or testomat.log
+  /**
+   * @deprecated Use `log` or `testomat.log`
+   */
   testomatioLogger: services.logger,
 
   artifact: reporterFunctions.artifact,
@@ -13,12 +28,6 @@ export default {
   meta: reporterFunctions.keyValue,
   step: reporterFunctions.step,
 
-  TestomatClient,
-  TRConstants,
+  // TestomatClient,
+  // TRConstants,
 };
-
-export const artifact = reporterFunctions.artifact;
-export const log = reporterFunctions.log;
-export const logger = services.logger;
-export const meta = reporterFunctions.keyValue;
-export const step = reporterFunctions.step;

package/src/services/key-values.js

@@ -18,7 +18,7 @@ class KeyValueStorage {
 
   /**
    * Stores key-value pair and passes it to reporter
-   * @param {{key: string}} keyValue - key-value pair(s) as object
+   * @param {{[key: string]: string}} keyValue - key-value pair(s) as object
    * @param {*} context - full test title
    */
   put(keyValue, context = null) {

package/src/services/logger.js

@@ -45,8 +45,10 @@ class Logger {
   logLevel = process?.env?.LOG_LEVEL?.toUpperCase() || 'ALL';
 
   constructor() {
-    if (!dataStorage.isFileStorage ||
-      (process.env.TESTOMATIO_INTERCEPT_CONSOLE_LOGS&& process.env.TESTOMATIO_INTERCEPT_CONSOLE_LOGS !== 'false'))
+    if (
+      !dataStorage.isFileStorage ||
+      (process.env.TESTOMATIO_INTERCEPT_CONSOLE_LOGS && process.env.TESTOMATIO_INTERCEPT_CONSOLE_LOGS !== 'false')
+    )
       this.intercept(console);
   }
 
@@ -91,7 +93,6 @@ class Logger {
         logs.push(arg.join(' '));
       } else {
         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);
@@ -121,7 +122,7 @@ class Logger {
           current +
           // strings are splitted by args when use tagged template, thus we add arg after each string
           // it looks like: `string1 arg1 string2 arg2 string3`
-          (args[index] !== undefined // eslint-disable-line no-nested-ternary
+          (args[index] !== undefined
             ? typeof args[index] === 'string'
               ? args[index] // add arg as it is
               : this.#stringifyLogs(args[index]) // stringify arg