You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by al...@apache.org on 2017/07/07 09:24:11 UTC

cordova-paramedic git commit: Better default Sauce Labs job name

Repository: cordova-paramedic
Updated Branches:
  refs/heads/master 3b32f80bf -> e3367b83c


Better default Sauce Labs job name


Project: http://git-wip-us.apache.org/repos/asf/cordova-paramedic/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-paramedic/commit/e3367b83
Tree: http://git-wip-us.apache.org/repos/asf/cordova-paramedic/tree/e3367b83
Diff: http://git-wip-us.apache.org/repos/asf/cordova-paramedic/diff/e3367b83

Branch: refs/heads/master
Commit: e3367b83c336c16fd91b369a97138682911c83b3
Parents: 3b32f80
Author: Alexander Sorokin <al...@akvelon.com>
Authored: Fri Jul 7 12:23:57 2017 +0300
Committer: Alexander Sorokin <al...@akvelon.com>
Committed: Fri Jul 7 12:23:57 2017 +0300

----------------------------------------------------------------------
 lib/ParamedicConfig.js |  4 ++--
 lib/paramedic.js       | 16 ++++++++--------
 2 files changed, 10 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cordova-paramedic/blob/e3367b83/lib/ParamedicConfig.js
----------------------------------------------------------------------
diff --git a/lib/ParamedicConfig.js b/lib/ParamedicConfig.js
index 65b186d..c6b2360 100644
--- a/lib/ParamedicConfig.js
+++ b/lib/ParamedicConfig.js
@@ -159,7 +159,7 @@ ParamedicConfig.prototype.setShouldUseSauce = function (sus) {
 };
 
 ParamedicConfig.prototype.getBuildName = function () {
-    return this._config.buildName || DEFAULT_BUILD_NAME;
+    return this._config.buildName || this.getDefaultBuildName();
 };
 
 ParamedicConfig.prototype.setBuildName = function (buildName) {
@@ -167,7 +167,7 @@ ParamedicConfig.prototype.setBuildName = function (buildName) {
 };
 
 ParamedicConfig.prototype.getDefaultBuildName = function () {
-    return DEFAULT_BUILD_NAME;
+    return DEFAULT_BUILD_NAME + ' ' + Date.now();
 };
 
 ParamedicConfig.prototype.getSauceUser = function () {

http://git-wip-us.apache.org/repos/asf/cordova-paramedic/blob/e3367b83/lib/paramedic.js
----------------------------------------------------------------------
diff --git a/lib/paramedic.js b/lib/paramedic.js
index 732a48b..5b19b69 100644
--- a/lib/paramedic.js
+++ b/lib/paramedic.js
@@ -106,7 +106,7 @@ ParamedicRunner.prototype.run = function () {
                     self.killEmulatorProcess();
                 });
         }
-        return self.displaySauceDetails();
+        return self.displaySauceDetails(self.sauceBuildName);
     })
     .fin(function () {
         self.cleanUpProject();
@@ -698,10 +698,13 @@ ParamedicRunner.prototype.getAppName = function () {
     return appName;
 };
 
-ParamedicRunner.prototype.displaySauceDetails = function () {
+ParamedicRunner.prototype.displaySauceDetails = function (buildName) {
     if (!this.config.shouldUseSauce()) {
         return Q();
     }
+    if (!buildName) {
+        buildName = this.config.getBuildName();
+    }
 
     var self = this;
     var d = Q.defer();
@@ -713,14 +716,10 @@ ParamedicRunner.prototype.displaySauceDetails = function () {
         password: self.config.getSauceKey()
     });
 
-    if (self.config.getBuildName() === self.config.getDefaultBuildName()) {
-        logger.warn('Build name is not specified, showing all sauce jobs with default name...');
-    }
-
     sauce.getJobs(function (err, jobs) {
         var found = false;
         for (var job in jobs) {
-            if (jobs.hasOwnProperty(job) && jobs[job].name && jobs[job].name.indexOf(self.config.getBuildName()) === 0) {
+            if (jobs.hasOwnProperty(job) && jobs[job].name && jobs[job].name.indexOf(buildName) === 0) {
                 var jobUrl = 'https://saucelabs.com/beta/tests/' + jobs[job].id;
                 logger.normal('============================================================================================');
                 logger.normal('Job name: ' + jobs[job].name);
@@ -746,8 +745,9 @@ ParamedicRunner.prototype.displaySauceDetails = function () {
 };
 
 ParamedicRunner.prototype.getSauceCaps = function () {
+    this.sauceBuildName = this.sauceBuildName || this.config.getBuildName();
     var caps = {
-        name: this.config.getBuildName(),
+        name: this.sauceBuildName,
         browserName: '',
         appiumVersion: this.config.getSauceAppiumVersion(),
         deviceOrientation: 'portrait',


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org