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 2016/07/15 09:07:52 UTC
cordova-paramedic git commit: CB-11584 Don't try to run Appium tests
when --justBuild'ing
Repository: cordova-paramedic
Updated Branches:
refs/heads/master c53e22e6e -> bbd6f46b3
CB-11584 Don't try to run Appium tests when --justBuild'ing
Project: http://git-wip-us.apache.org/repos/asf/cordova-paramedic/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-paramedic/commit/bbd6f46b
Tree: http://git-wip-us.apache.org/repos/asf/cordova-paramedic/tree/bbd6f46b
Diff: http://git-wip-us.apache.org/repos/asf/cordova-paramedic/diff/bbd6f46b
Branch: refs/heads/master
Commit: bbd6f46b3c442c2ca5acc026d6d13ef92ea526b8
Parents: c53e22e
Author: Alexander Sorokin <al...@akvelon.com>
Authored: Fri Jul 15 12:07:40 2016 +0300
Committer: Alexander Sorokin <al...@akvelon.com>
Committed: Fri Jul 15 12:07:40 2016 +0300
----------------------------------------------------------------------
lib/ParamedicConfig.js | 2 +-
lib/paramedic.js | 5 +++++
2 files changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cordova-paramedic/blob/bbd6f46b/lib/ParamedicConfig.js
----------------------------------------------------------------------
diff --git a/lib/ParamedicConfig.js b/lib/ParamedicConfig.js
index 0233eb2..e81f87c 100644
--- a/lib/ParamedicConfig.js
+++ b/lib/ParamedicConfig.js
@@ -44,7 +44,7 @@ ParamedicConfig.prototype.getDefaultSaucePlatformVersion = function () {
ParamedicConfig.parseFromArguments = function (argv) {
return new ParamedicConfig({
platform: argv.platform,
- action: !!argv.justbuild ? 'build' : 'run',
+ action: !!(argv.justbuild || argv.justBuild) ? 'build' : 'run',
args: (!!argv.browserify ? '--browserify ' : ''),
plugins: Array.isArray(argv.plugin) ? argv.plugin : [argv.plugin],
useTunnel: !!argv.useTunnel,
http://git-wip-us.apache.org/repos/asf/cordova-paramedic/blob/bbd6f46b/lib/paramedic.js
----------------------------------------------------------------------
diff --git a/lib/paramedic.js b/lib/paramedic.js
index 5e30914..f31cade 100644
--- a/lib/paramedic.js
+++ b/lib/paramedic.js
@@ -229,6 +229,7 @@ ParamedicRunner.prototype.runLocalTests = function () {
self.waitForTests().then(resolve);
});
}
+ return util.TEST_PASSED; // if we're not waiting for a test result, just report tests as passed
}, function(output) {
// this trace is automatically available in verbose mode
// so we check for this flag to not trace twice
@@ -242,6 +243,10 @@ ParamedicRunner.prototype.runLocalTests = function () {
ParamedicRunner.prototype.runAppiumTests = function (useSauce) {
var platform = this.config.getPlatformId();
var self = this;
+ if (self.config.getAction() === 'build') {
+ logger.normal('Just building, so skipping Appium tests...');
+ return Q(util.TEST_PASSED);
+ }
if (platform !== 'android' && platform !== 'ios') {
logger.warn('Unsupported platform for Appium test run: ' + platform);
// just skip Appium tests
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org