saucelabs - npm Package Compare versions

Comparing version 4.2.1 to 4.3.0

build/constants.js

@@ -6,3 +6,3 @@ "use strict";
 });
-exports.SC_CLOSE_TIMEOUT = exports.SC_CLOSE_MESSAGE = exports.SC_READY_MESSAGE = exports.SC_PARAMS_TO_STRIP = exports.SC_CLI_PARAM_KEYS = exports.SAUCE_CONNECT_CLI_PARAMS = exports.CLI_PARAMS = exports.EPILOG = exports.USAGE = exports.TO_STRING_TAG = exports.SYMBOL_ITERATOR = exports.SYMBOL_TOSTRING = exports.SYMBOL_INSPECT = exports.REGION_MAPPING = exports.DEFAULT_OPTIONS = exports.JOB_ASSET_NAMES = exports.PARAMETERS_MAP = exports.PROTOCOL_MAP = exports.SAUCE_VERSION_NOTE = exports.SAUCE_CONNECT_DISTS = exports.SAUCE_CONNECT_BASE = exports.DEFAULT_SAUCE_CONNECT_VERSION = void 0;
+exports.SC_CLOSE_TIMEOUT = exports.SC_CLOSE_MESSAGE = exports.SC_FAILURE_MESSAGE = exports.SC_READY_MESSAGE = exports.SC_PARAMS_TO_STRIP = exports.SC_CLI_PARAM_KEYS = exports.SAUCE_CONNECT_CLI_PARAMS = exports.CLI_PARAMS = exports.EPILOG = exports.USAGE = exports.TO_STRING_TAG = exports.SYMBOL_ITERATOR = exports.SYMBOL_TOSTRING = exports.SYMBOL_INSPECT = exports.REGION_MAPPING = exports.DEFAULT_OPTIONS = exports.JOB_ASSET_NAMES = exports.PARAMETERS_MAP = exports.PROTOCOL_MAP = exports.SAUCE_VERSION_NOTE = exports.SAUCE_CONNECT_DISTS = exports.SAUCE_CONNECT_BASE = exports.DEFAULT_SAUCE_CONNECT_VERSION = void 0;
 
@@ -277,2 +277,4 @@ var _changeCase = require("change-case");
 exports.SC_READY_MESSAGE = SC_READY_MESSAGE;
+const SC_FAILURE_MESSAGE = 'Sauce Connect could not establish a connection';
+exports.SC_FAILURE_MESSAGE = SC_FAILURE_MESSAGE;
 const SC_CLOSE_MESSAGE = 'Goodbye';
@@ -279,0 +281,0 @@ exports.SC_CLOSE_MESSAGE = SC_CLOSE_MESSAGE;

build/index.js

@@ -244,3 +244,3 @@ "use strict";
      */
-    .filter(([k]) => !['_', '$0', 'sc-version', ..._constants.SC_PARAMS_TO_STRIP].includes(k))
+    .filter(([k]) => !['_', '$0', 'sc-version', 'logger', ..._constants.SC_PARAMS_TO_STRIP].includes(k))
     /**
@@ -282,2 +282,3 @@ * remove duplicate params by yargs
       cp.stdout.on('data', data => {
+        const logger = fromCLI ? process.stdout.write.bind(process.stdout) : argv.logger;
         const output = data.toString();
@@ -288,6 +289,18 @@ /**
 
-        if (fromCLI) {
-          process.stdout.write(output);
+        if (typeof logger === 'function') {
+          logger(output);
         }
+        /**
+         * fail if SauceConnect could not establish a connection
+         */
 
+
+        if (output.includes(_constants.SC_FAILURE_MESSAGE)) {
+          return reject(new Error(output));
+        }
+        /**
+         * continue if connection was established
+         */
+
+
         if (output.includes(_constants.SC_READY_MESSAGE)) {
@@ -294,0 +307,0 @@ return resolve(returnObj);

package.json

 {
   "name": "saucelabs",
-  "version": "4.2.1",
+  "version": "4.3.0",
   "author": "Christian Bromann <[e-mail]>",
@@ -86,4 +86,4 @@ "description": "A wrapper around Sauce Labs REST API",
         "functions": 100,
-        "lines": 96,
-        "statements": 96
+        "lines": 97,
+        "statements": 97
       }
@@ -90,0 +90,0 @@ },

README.md

@@ -172,2 +172,6 @@ <p align="center">
         /**
+         * you can pass in a `logger` method to print Sauce Connect log messages
+         */
+        logger: (stdout) => console.log(stdout),
+        /**
          * see all available parameters here: https://wiki.saucelabs.com/display/DOCS/Sauce+Connect+Proxy+Command-Line+Quick+Reference+Guide
@@ -174,0 +178,0 @@ * all parameters have to be applied camel cased instead of with hyphens, e.g.

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

311772

increased by0.27%

Lines of code

9201

increased by0.17%

Number of lines in readme file

241

increased by1.69%

No dependency changes