You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cordova.apache.org by alsorokin <gi...@git.apache.org> on 2016/02/19 16:05:32 UTC

[GitHub] cordova-medic pull request: CB-10652 Added the instruction on how ...

GitHub user alsorokin opened a pull request:

    https://github.com/apache/cordova-medic/pull/81

    CB-10652 Added the instruction on how to prepare a slave for Appium t…

    …ests
    
    https://issues.apache.org/jira/browse/CB-10652

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/MSOpenTech/cordova-medic CB-10652

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cordova-medic/pull/81.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #81
    
----
commit 70bd723bf8ec35b3a673a91ffccfcb693efafaed
Author: Alexander Sorokin <al...@akvelon.com>
Date:   2016-02-19T15:03:42Z

    CB-10652 Added the instruction on how to prepare a slave for Appium tests

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org


[GitHub] cordova-medic pull request: CB-10652 Added the instruction on how ...

Posted by alsorokin <gi...@git.apache.org>.
Github user alsorokin commented on the pull request:

    https://github.com/apache/cordova-medic/pull/81#issuecomment-186250451
  
    @dblotsky please take a look


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org


[GitHub] cordova-medic pull request: CB-10652 Added the instruction on how ...

Posted by dblotsky <gi...@git.apache.org>.
Github user dblotsky commented on the pull request:

    https://github.com/apache/cordova-medic/pull/81#issuecomment-187511248
  
    Will the iOS steps always be necessary, or will they become unneeded later? For Android, do you have a link to the bug that when fixed will obsolete this Android setup?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org


[GitHub] cordova-medic pull request: CB-10652 Added the instruction on how ...

Posted by dblotsky <gi...@git.apache.org>.
Github user dblotsky commented on a diff in the pull request:

    https://github.com/apache/cordova-medic/pull/81#discussion_r53733144
  
    --- Diff: SLAVES.md ---
    @@ -27,6 +27,17 @@ Two special NPM packages are also required for Medic builds on iOS: `ios-sim` an
     
         npm install -g ios-deploy ios-sim
     
    +To run Appium tests on real iOS devices you may need to install ios-webkit-debug-proxy using the command:
    +
    + ``` center
    + # The first command is only required if you don't have brew installed.
    + > ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"
    --- End diff --
    
    Please point the docs to the official Homebrew installation page, since this command might change.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org


[GitHub] cordova-medic pull request: CB-10652 Added the instruction on how ...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/cordova-medic/pull/81


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org


[GitHub] cordova-medic pull request: CB-10652 Added the instruction on how ...

Posted by alsorokin <gi...@git.apache.org>.
Github user alsorokin commented on the pull request:

    https://github.com/apache/cordova-medic/pull/81#issuecomment-188248990
  
    Also, added the links to the Appium issues describing the problem and the workaround for the chromedriver issue.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org


[GitHub] cordova-medic pull request: CB-10652 Added the instruction on how ...

Posted by dblotsky <gi...@git.apache.org>.
Github user dblotsky commented on a diff in the pull request:

    https://github.com/apache/cordova-medic/pull/81#discussion_r54035101
  
    --- Diff: SLAVES.md ---
    @@ -38,6 +49,12 @@ To make the Android commands available on the command line, set the following en
     - `ANDROID_HOME`, equal to the absolute path to the Android SDK directory
     - `PATH`, extended to contain `ANDROID_HOME/tools` and `ANDROID_HOME/platform_tools`
     
    +To run Appium tests, you may also need to download the [latest chromedriver](https://sites.google.com/a/chromium.org/chromedriver/downloads) and set the following environment variable:
    +
    +- `CHROMEDRIVER_EXECUTABLE`, equal to the absolute path to the chromedriver executable
    +
    +This is needed to avoid the bug in older chromedriver versions that was preventing Appium tests to switch contexts. There are a bunch of github issues created for this bug in Appium project: [1](https://github.com/appium/appium/issues/4429), [2](https://github.com/appium/appium/issues/5526), [3](https://github.com/appium/appium/issues/5616).
    --- End diff --
    
    Please make the links reference links.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org


[GitHub] cordova-medic pull request: CB-10652 Added the instruction on how ...

Posted by alsorokin <gi...@git.apache.org>.
Github user alsorokin commented on the pull request:

    https://github.com/apache/cordova-medic/pull/81#issuecomment-188232798
  
    @dblotsky iOS steps will always be necessary only if you want to run the tests on real device.
    You don't need to run these steps to run the tests on the simulator.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org