wdio-reportportal-reporter - npm Package Compare versions

Comparing version 0.0.9 to 0.0.10

.github/ISSUE_TEMPLATE.md

@@ -0,0 +0,0 @@ ## The problem

build/reporter.js

@@ -22,3 +22,3 @@ 'use strict';
 const { testItemStatuses, events, entityType } = require('./constants');
-const { promiseErrorHandler, logger, isEmpty, limit, sendToReporter } = require('./utils');
+const { promiseErrorHandler, Logger, isEmpty, limit, sendToReporter } = require('./utils');
 
@@ -31,2 +31,3 @@ const { PASSED, FAILED, SKIPPED } = testItemStatuses;
     this.parents = {};
+    this.logger = new Logger(options.debug);
     this.testStartRequestsPromises = {};
@@ -40,3 +41,4 @@ this.lastFailedTestRequestPromises = {};
       screenshotsLogLevel: 'info',
-      enableRetriesWorkaround: false
+      enableRetriesWorkaround: false,
+      debug: false
     }, options);
@@ -122,3 +124,2 @@
     }
-
     const parent = this.getParent(test.cid);
@@ -153,2 +154,6 @@ if (parent.type === entityType.TEST && this.options.enableRetriesWorkaround) {
   testPending(test) {
+    const parent = this.getParent(test.cid);
+    if (parent && parent.type === entityType.SUITE) {
+      this.testStart(test);
+    }
     this.testFinished(test, SKIPPED, { issue_type: 'NOT_ISSUE' });
@@ -243,3 +248,3 @@ }
     if (!(await this.waitForFailedTest(cid, 2000, 10))) {
-      logger.warn('Attempt to send file to failed item fails. There is no failed test yet.');
+      this.logger.warn('Attempt to send file to failed item fails. There is no failed test yet.');
       return;
@@ -264,3 +269,3 @@ }
     if (!(await this.waitForFailedTest(cid, 2000, 10))) {
-      logger.warn('Attempt to send log to failed item fails. There is no failed test yet.');
+      this.logger.warn('Attempt to send log to failed item fails. There is no failed test yet.');
       return;
@@ -267,0 +272,0 @@ }

build/utils.js

@@ -17,3 +17,3 @@ 'use strict';
 
-/* eslint-disable no-console,no-param-reassign */
+/* eslint-disable no-console,no-param-reassign,class-methods-use-this */
 const stringify = require('json-stringify-safe');
@@ -27,14 +27,23 @@
 
-const logger = {
+class Logger {
+  constructor(debug = false) {
+    this.debug = debug;
+  }
+
   info(msg) {
     console.log(msg);
-  },
+  }
+
   error(msg) {
     console.error(msg);
-  },
+  }
+
   warn(msg) {
-    console.warn(msg);
+    if (this.debug) {
+      console.warn(msg);
+    }
   }
-};
+}
 
+const logger = new Logger();
 const promiseErrorHandler = promise => {
@@ -121,3 +130,3 @@ promise.catch(err => {
 module.exports = {
-  promiseErrorHandler, logger, isEmpty, limit, sendToReporter
+  promiseErrorHandler, Logger, isEmpty, limit, sendToReporter
 };

lib/reporter.js

@@ -6,3 +6,3 @@ /* eslint-disable object-curly-newline */
 const { testItemStatuses, events, entityType } = require('./constants');
-const { promiseErrorHandler, logger, isEmpty, limit, sendToReporter } = require('./utils');
+const { promiseErrorHandler, Logger, isEmpty, limit, sendToReporter } = require('./utils');
 
@@ -15,2 +15,3 @@ const { PASSED, FAILED, SKIPPED } = testItemStatuses;
     this.parents = {};
+    this.logger = new Logger(options.debug);
     this.testStartRequestsPromises = {};
@@ -25,2 +26,3 @@ this.lastFailedTestRequestPromises = {};
       enableRetriesWorkaround: false,
+      debug: false,
     }, options);
@@ -110,3 +112,2 @@
     }
-
     const parent = this.getParent(test.cid);
@@ -145,2 +146,6 @@ if (parent.type === entityType.TEST && this.options.enableRetriesWorkaround) {
   testPending(test) {
+    const parent = this.getParent(test.cid);
+    if (parent && parent.type === entityType.SUITE) {
+      this.testStart(test);
+    }
     this.testFinished(test, SKIPPED, { issue_type: 'NOT_ISSUE' });
@@ -235,3 +240,3 @@ }
     if (!(await this.waitForFailedTest(cid, 2000, 10))) {
-      logger.warn('Attempt to send file to failed item fails. There is no failed test yet.');
+      this.logger.warn('Attempt to send file to failed item fails. There is no failed test yet.');
       return;
@@ -256,3 +261,3 @@ }
     if (!(await this.waitForFailedTest(cid, 2000, 10))) {
-      logger.warn('Attempt to send log to failed item fails. There is no failed test yet.');
+      this.logger.warn('Attempt to send log to failed item fails. There is no failed test yet.');
       return;
@@ -259,0 +264,0 @@ }

lib/utils.js

@@ -1,2 +0,2 @@
-/* eslint-disable no-console,no-param-reassign */
+/* eslint-disable no-console,no-param-reassign,class-methods-use-this */
 const stringify = require('json-stringify-safe');
@@ -10,14 +10,23 @@
 
-const logger = {
+class Logger {
+  constructor(debug = false) {
+    this.debug = debug;
+  }
+
   info(msg) {
     console.log(msg);
-  },
+  }
+
   error(msg) {
     console.error(msg);
-  },
+  }
+
   warn(msg) {
-    console.warn(msg);
-  },
-};
+    if (this.debug) {
+      console.warn(msg);
+    }
+  }
+}
 
+const logger = new Logger();
 const promiseErrorHandler = (promise) => {
@@ -103,3 +112,3 @@ promise.catch((err) => {
 module.exports = {
-  promiseErrorHandler, logger, isEmpty, limit, sendToReporter,
+  promiseErrorHandler, Logger, isEmpty, limit, sendToReporter,
 };

package.json

 {
     "name": "wdio-reportportal-reporter",
-    "version": "0.0.9",
+    "version": "0.0.10",
     "description": "A WebdriverIO plugin. Report results to Report Portal.",
@@ -10,10 +10,5 @@ "main": "build/reporter.js",
         "compile": "babel lib/ -d build/",
-        "eslint": "eslint ./lib test/",
-        "release": "np patch",
-        "release:patch": "np patch",
-        "release:minor": "np minor",
-        "release:major": "np major",
+        "eslint": "eslint ./lib",
         "test": "run-s eslint test:unit",
-        "test:unit": "mocha",
-        "watch": "npm run compile -- --watch"
+        "test:unit": "mocha"
     },
@@ -63,3 +58,2 @@ "repository": {
         "node-static": "^0.7.10",
-        "np": "~2.18.0",
         "npm-run-all": "~4.1.0",
@@ -66,0 +60,0 @@ "rimraf": "^2.6.2",

No alert changes

Improved metrics

Total package byte prevSize

173024

increased by384.97%

Dev dependency count

23

decreased by-4.17%

Number of package files

16

increased by14.29%

Lines of code

779

increased by3.18%

No dependency changes