Skip to content

Conversation

SvetoslavTsenov
Copy link
Contributor

No description provided.

@vchimev vchimev changed the title chore: readme docs: update README.md Nov 22, 2017

Install it with:
The `nativescript-dev-appium` plugin requires:
* latest version of [Appium](https://github.com/appium/appium/releases)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which is the latest version? I mean now is 'x' but tomorrow will the new one be compatible with the plugin. Maybe we should commit to current one.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We commit that the new one will be compatible with the plugin and avoid a specific version on purpose.

README.md Outdated
├── package.json
├── tsconfig.json

> Note: To avaid any incompatibilities between the source of `e2e` tests and the source of the application, we recommend to exclude the `e2e` folder from the application's `tsconfig.json` file: `exclude": [ "e2e" ]`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo 'avoid'. Not sure why I have to exclude the e2e folder.

README.md Outdated
2. Find elements findElementByText, findElementsByXPath, findElementByAccessibilityId etc...
3. Gesture support: swipe, scroll, drag, scrollTo
4. Action support: tap, click, doubleTap, hold,
5. Element characteristics: lcation, exists, size, isDisplayed
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo 'location'

README.md Outdated
## Troubleshooting

Use the `--verbose` option to get error details:

```
$ npm run appium --runType=android --verbose
$ npm run appium --runType android25 --verbose
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

npm run e2e -- --runType android25 --verbose

@@ -126,17 +161,30 @@ $ npm run e2e --runType android25 --sauceLab --appLocation demo.apk --capsLocati

```
Copy link
Contributor

@zbranzov zbranzov Nov 22, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One more line 160 has to be edited:
npm run e2e -- --runType android25 --sauceLab --appPath demo.apk --capsLocation "/e2e-tests/config"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@vchimev vchimev merged commit 9a9fb3c into master Nov 22, 2017
@vchimev vchimev deleted the readme branch November 22, 2017 14:49
@lock
Copy link

lock bot commented Aug 26, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Aug 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants