Commit 7dd4b0e

committed

Fix Appium tests by tweaking timeouts.

- Switch to Node 6.4.0 for Travis. - Add explicit timeouts in appium test initial waits. - Get rid of explicit platform add.

1 parent 4fa1649 commit 7dd4b0eCopy full SHA for 7dd4b0e

File tree

+15

-10

lines changed

+15

-10

lines changed

Lines changed: 6 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`env:`
`2`	`2`	`global:`
`3`		`- - NODE_VERSION=4.4.7`
	`3`	`+ - NODE_VERSION=6.4.0`
`4`	`4`	`- DATE=$(date +%Y-%m-%d)`
`5`	`5`	`- PACKAGE_VERSION=$DATE-$TRAVIS_BUILD_NUMBER`
`6`	`6`
`@@ -22,7 +22,7 @@ before_cache:`
`22`	`22`
`23`	`23`	`cache:`
`24`	`24`	`directories:`
`25`		`- - .nvm`
	`25`	`+ - $HOME/.nvm`
`26`	`26`	`- $HOME/.gradle/caches/`
`27`	`27`	`- $HOME/.gradle/wrapper/`
`28`	`28`
`@@ -41,14 +41,14 @@ install:`
`41`	`41`
`42`	`42`	`before_script:`
`43`	`43`	`- echo no \| android create avd --force -n test -t android-19 -b armeabi-v7a`
`44`		`-- emulator -avd test -no-audio -no-window &`
`45`		`-- android-wait-for-emulator`
	`44`	`+- emulator -memory 1024 -avd test -no-audio -no-window &`
`46`	`45`
`47`	`46`	`script:`
`48`		`-- tns platform add android`
`49`	`47`	`- # DISABLE KARMA TESTS - hanging the build on Travis`
`50`	`48`	`- # tns test android --emulator --justlaunch`
`51`		`-- npm run appium-android`
	`49`	`+- tns build android`
	`50`	`+- android-wait-for-emulator`
	`51`	`+- npm run run-appium-android`
`52`	`52`
`53`	`53`	`before_deploy:`
`54`	`54`	`- cd ../nativescript-angular`

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,7 @@`
`2`	`2`	`var nsAppium = require("nativescript-dev-appium");`
`3`	`3`
`4`	`4`	`describe("multi page routing", function () {`
`5`		`- this.timeout(120000);`
	`5`	`+ this.timeout(360000);`
`6`	`6`	`var driver;`
`7`	`7`
`8`	`8`	`before(function () {`
`@@ -19,6 +19,7 @@ describe("multi page routing", function () {`
`19`	`19`
`20`	`20`	`it("loads default path", function () {`
`21`	`21`	`return driver`
	`22`	`+ .waitForElementByAccessibilityId("first-multi-page", 300000)`
`22`	`23`	`.elementByAccessibilityId("first-multi-page")`
`23`	`24`	`.should.eventually.exist`
`24`	`25`	`.text().should.eventually.equal("First: multi-page")`
`@@ -31,6 +32,7 @@ describe("multi page routing", function () {`
`31`	`32`	`"second.destroy"] // <-- back (first component is reused)`
`32`	`33`	`.join(",");`
`33`	`34`	`return driver`
	`35`	`+ .waitForElementByAccessibilityId("first-navigate-multi-page", 300000)`
`34`	`36`	`.elementByAccessibilityId("first-navigate-multi-page")`
`35`	`37`	`.should.eventually.exist`
`36`	`38`	`.tap()`

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,7 @@`
`2`	`2`	`var nsAppium = require("nativescript-dev-appium");`
`3`	`3`
`4`	`4`	`describe("single page routing", function () {`
`5`		`- this.timeout(120000);`
	`5`	`+ this.timeout(360000);`
`6`	`6`	`var driver;`
`7`	`7`
`8`	`8`	`before(function () {`
`@@ -19,6 +19,7 @@ describe("single page routing", function () {`
`19`	`19`
`20`	`20`	`it("loads default path", function () {`
`21`	`21`	`return driver`
	`22`	`+ .waitForElementByAccessibilityId("first-single-page", 300000)`
`22`	`23`	`.elementByAccessibilityId("first-single-page")`
`23`	`24`	`.should.eventually.exist`
`24`	`25`	`.text().should.eventually.equal("First: single-page")`
`@@ -33,6 +34,7 @@ describe("single page routing", function () {`
`33`	`34`	`"first.init"].join(",");`
`34`	`35`
`35`	`36`	`return driver`
	`37`	`+ .waitForElementByAccessibilityId("first-single-page", 300000)`
`36`	`38`	`.elementByAccessibilityId("first-navigate-single-page")`
`37`	`39`	`.should.eventually.exist`
`38`	`40`	`.tap()`

Lines changed: 3 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,8 @@`
`64`	`64`	`},`
`65`	`65`	`"scripts": {`
`66`	`66`	`"updateTests": "grunt updateTests",`
`67`		`- "appium-android": "tns build android && nativescript-dev-appium android",`
	`67`	`+ "appium-android": "tns build android && npm run run-appium-android",`
	`68`	`+ "run-appium-android": "nativescript-dev-appium android",`
`68`	`69`	`"appium-ios-simulator": "tns build ios && nativescript-dev-appium ios-simulator"`
`69`	`70`	`}`
`70`		`-}`
	`71`	`+}`

Comments

(0)