Fix Appium tests by tweaking timeouts.

hdeshev · hdeshev · commit 7dd4b0e94010 · 2016-08-30T16:03:53.000+03:00
- Switch to Node 6.4.0 for Travis.
- Add explicit timeouts in appium test initial waits.
- Get rid of explicit platform add.
diff --git a/.travis.yml b/.travis.yml
@@ -1,6 +1,6 @@
 env:
     global:
-        - NODE_VERSION=4.4.7
+        - NODE_VERSION=6.4.0
         - DATE=$(date +%Y-%m-%d)
         - PACKAGE_VERSION=$DATE-$TRAVIS_BUILD_NUMBER
 
@@ -22,7 +22,7 @@ before_cache:
 
 cache:
   directories:
-        - .nvm
+        - $HOME/.nvm
         - $HOME/.gradle/caches/
         - $HOME/.gradle/wrapper/
 
@@ -41,14 +41,14 @@ install:
 
 before_script:
 - echo no | android create avd --force -n test -t android-19 -b armeabi-v7a
-- emulator -avd test -no-audio -no-window &
-- android-wait-for-emulator
+- emulator -memory 1024 -avd test -no-audio -no-window &
 
 script:
-- tns platform add android
 - # DISABLE KARMA TESTS - hanging the build on Travis
 - # tns test android --emulator --justlaunch
-- npm run appium-android
+- tns build android
+- android-wait-for-emulator
+- npm run run-appium-android
 
 before_deploy:
 - cd ../nativescript-angular
diff --git a/tests/e2e-tests/multi-page-routing.js b/tests/e2e-tests/multi-page-routing.js
@@ -2,7 +2,7 @@
 var nsAppium = require("nativescript-dev-appium");
 
 describe("multi page routing", function () {
-    this.timeout(120000);
+    this.timeout(360000);
     var driver;
 
     before(function () {
@@ -19,6 +19,7 @@ describe("multi page routing", function () {
 
     it("loads default path", function () {
         return driver
+            .waitForElementByAccessibilityId("first-multi-page", 300000)
             .elementByAccessibilityId("first-multi-page")
                 .should.eventually.exist
             .text().should.eventually.equal("First: multi-page")
@@ -31,6 +32,7 @@ describe("multi page routing", function () {
             "second.destroy"] // <-- back (first component is reused)
             .join(",");
         return driver
+            .waitForElementByAccessibilityId("first-navigate-multi-page", 300000)
             .elementByAccessibilityId("first-navigate-multi-page")
                 .should.eventually.exist
             .tap()
diff --git a/tests/e2e-tests/single-page-routing.js b/tests/e2e-tests/single-page-routing.js
@@ -2,7 +2,7 @@
 var nsAppium = require("nativescript-dev-appium");
 
 describe("single page routing", function () {
-    this.timeout(120000);
+    this.timeout(360000);
     var driver;
 
     before(function () {
@@ -19,6 +19,7 @@ describe("single page routing", function () {
 
     it("loads default path", function () {
         return driver
+            .waitForElementByAccessibilityId("first-single-page", 300000)
             .elementByAccessibilityId("first-single-page")
                 .should.eventually.exist
             .text().should.eventually.equal("First: single-page")
@@ -33,6 +34,7 @@ describe("single page routing", function () {
             "first.init"].join(",");
 
         return driver
+            .waitForElementByAccessibilityId("first-single-page", 300000)
             .elementByAccessibilityId("first-navigate-single-page")
                 .should.eventually.exist
             .tap()
diff --git a/tests/package.json b/tests/package.json
@@ -64,7 +64,8 @@
     },
     "scripts": {
         "updateTests": "grunt updateTests",
-        "appium-android": "tns build android && nativescript-dev-appium android",
+        "appium-android": "tns build android && npm run run-appium-android",
+        "run-appium-android": "nativescript-dev-appium android",
         "appium-ios-simulator": "tns build ios && nativescript-dev-appium ios-simulator"
     }
-}
+}
