Github user robpaveza commented on a diff in the pull request:
https://github.com/apache/cordova-medic/pull/47#discussion_r29370199
--- Diff: bin/medic.js ---
@@ -19,18 +19,44 @@
* under the License.
*/
-"use strict";
+/* jshint node: true */
+
+// "use strict";
// node dependencies
-var fs = require("fs");
-var os = require("os");
+var fs = require("fs");
+var os = require("os");
+var path = require("path");
// external dependencies
var shelljs = require("shelljs");
var optimist = require("optimist");
+// internal dependencies
+var testwait = require("./lib/testwait");
+var testcheck = require("./lib/testcheck");
+
// constants
-var MAX_REMOVAL_ATTEMPTS = 3;
+var MAX_REMOVAL_ATTEMPTS = 3;
+var CORDOVA_MEDIC_DIR = "cordova-medic";
+var DEFAULT_APP_PATH = "mobilespec";
+var DEFAULT_APP_ENTRY = "index.html";
+var ANDROID_TIMEOUT = 120000;
--- End diff --
Android timeout is really, really long if minute is 60... Maybe rewrite
ANDROID_TIMEOUT in context of ONE_MINUTE?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]