Skip to content

Commit

Permalink
add some logging to make it easier to follow ios startup flow
Browse files Browse the repository at this point in the history
  • Loading branch information
jlipps committed Mar 12, 2014
1 parent b2a782a commit 775fb8f
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
7 changes: 6 additions & 1 deletion lib/devices/ios/ios.js
Expand Up @@ -387,6 +387,7 @@ IOS.prototype.onUnexpectedInstrumentsExit = function (code, traceDir) {
};

IOS.prototype.setXcodeFolder = function (cb) {
logger.info("Setting Xcode folder");
helpers.getXcodeFolder(function (err, xcodeFolder) {
if (err) {
logger.error("Could not determine Xcode folder");
Expand All @@ -397,6 +398,7 @@ IOS.prototype.setXcodeFolder = function (cb) {
};

IOS.prototype.setXcodeVersion = function (cb) {
logger.info("Setting Xcode version");
helpers.getXcodeVersion(function (err, versionNumber) {
if (err) {
logger.error("Could not determine Xcode version");
Expand All @@ -412,6 +414,7 @@ IOS.prototype.setXcodeVersion = function (cb) {
};

IOS.prototype.setiOSSDKVersion = function (cb) {
logger.info("Setting iOS SDK Version");
helpers.getiOSSDKVersion(function (err, versionNumber) {
if (err) {
logger.error("Could not determine iOS SDK version");
Expand Down Expand Up @@ -464,7 +467,7 @@ IOS.prototype.setLocale = function (cb) {
}
}.bind(this));
} else {
msg = "Could not set locale information because the ios-sim-local was not found at " + iosSimLocalePath;
msg = "Could not set locale information because the ios-sim-locale was not found at " + iosSimLocalePath;
logger.error(msg);
cb(new Error(msg));
}
Expand Down Expand Up @@ -612,6 +615,7 @@ IOS.prototype.setSafariPrefs = function () {
};

IOS.prototype.detectTraceTemplate = function (cb) {
logger.info("Detecting automation tracetemplate");
var msg;
if (!this.args.automationTraceTemplatePath) {
helpers.getXcodeFolder(function (err, xcodeFolderPath) {
Expand Down Expand Up @@ -661,6 +665,7 @@ IOS.prototype.detectUdid = function (cb) {
cb(new Error("Timed out trying to detect udid."));
});
} else {
logger.info("Not auto-detecting udid, running on sim");
cb();
}
};
Expand Down
2 changes: 2 additions & 0 deletions test/README.md
Expand Up @@ -8,3 +8,5 @@ Here's a list of things that aren't tested in this suite:
* node install in appium.app (for appium-instruments)
* android with --udid (even if emulator)
* {launch: false} caps
* setting locale
* --avd / avdName

0 comments on commit 775fb8f

Please sign in to comment.