[GitHub] cordova-plugin-camera pull request #244: Android Appium test tweaks

2016-12-23 Thread alsorokin
Github user alsorokin commented on a diff in the pull request:


https://github.com/apache/cordova-plugin-camera/pull/244#discussion_r93733873
  
--- Diff: appium-tests/android/android.spec.js ---
@@ -21,10 +21,13 @@
  *
 */
 
-// these tests are meant to be executed by Cordova Medic Appium runner
-// you can find it here: https://github.com/apache/cordova-medic/
+// these tests are meant to be executed by Cordova ParaMedic Appium runner
+// you can find it here: https://github.com/apache/cordova-paramedic/
 // it is not necessary to do a full CI setup to run these tests
-// just run "node cordova-medic/medic/medic.js appium --platform android 
--plugins cordova-plugin-camera"
+// for android 5.1, run:
+//  node cordova-paramedic/main.js --platform android --plugin 
cordova-plugin-camera --skipMainTests --saucePlatformVersion "5.1" --target 
api22
--- End diff --

I think `--saucePlatformVersion` is meant to be used together with 
`--shouldUseSauce`
For a local test run, I think it makes sense to use `--target` arg, which 
specifies the emulator name to run on.


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

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



[GitHub] cordova-plugin-camera pull request #244: Android Appium test tweaks

2016-12-23 Thread alsorokin
Github user alsorokin commented on a diff in the pull request:


https://github.com/apache/cordova-plugin-camera/pull/244#discussion_r93737605
  
--- Diff: appium-tests/android/android.spec.js ---
@@ -373,7 +384,6 @@ describe('Camera tests Android.', function () {
 return driver
 
.waitForElementByXPath('//android.widget.TextView[@text="Gallery"]', 2)
 
.elementByXPath('//android.widget.TextView[@text="Gallery"]')
-
.elementByXPath('//android.widget.TextView[@text="Gallery"]')
--- End diff --

Please do not delete duplicate element selections.
They are needed to work around the following bug: on Android emulator under 
Windows, element selection doesn't work the first time on some elements. And 
sometimes even a second or third time - wrong elements are being clicked.
The only way to work around this issue I've found is to select the element 
multiple times like this.
Maybe selecting element by accessibility id could resolve this issue. I'll 
make a note to myself to look into it a bit later, but for now please leave it 
as is.


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

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



[GitHub] cordova-windows issue #219: CB-12189: Add support for WinMD and DLL combinat...

2016-12-23 Thread matrosov-nikita
Github user matrosov-nikita commented on the issue:

https://github.com/apache/cordova-windows/pull/219
  
@vladimir-kotikov, @daserge, updated.


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

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



[GitHub] cordova-windows issue #219: CB-12189: Add support for WinMD and DLL combinat...

2016-12-23 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the issue:

https://github.com/apache/cordova-windows/pull/219
  
Rebased on top of master and force-pushed, will merge once tests pass


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

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



[GitHub] cordova-windows pull request #219: CB-12189: Add support for WinMD and DLL c...

2016-12-23 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cordova-windows/pull/219


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

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



[GitHub] cordova-lib pull request #513: CB-12189 windows: Add `implementation` attrib...

2016-12-23 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cordova-lib/pull/513


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

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



[GitHub] cordova-docs pull request #671: CB-12189 windows: Document `implementation` ...

2016-12-23 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cordova-docs/pull/671


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

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



[GitHub] cordova-plugin-camera pull request #244: Android Appium test tweaks

2016-12-23 Thread filmaj
Github user filmaj commented on a diff in the pull request:


https://github.com/apache/cordova-plugin-camera/pull/244#discussion_r93797685
  
--- Diff: appium-tests/android/android.spec.js ---
@@ -21,10 +21,13 @@
  *
 */
 
-// these tests are meant to be executed by Cordova Medic Appium runner
-// you can find it here: https://github.com/apache/cordova-medic/
+// these tests are meant to be executed by Cordova ParaMedic Appium runner
+// you can find it here: https://github.com/apache/cordova-paramedic/
 // it is not necessary to do a full CI setup to run these tests
-// just run "node cordova-medic/medic/medic.js appium --platform android 
--plugins cordova-plugin-camera"
+// for android 5.1, run:
+//  node cordova-paramedic/main.js --platform android --plugin 
cordova-plugin-camera --skipMainTests --saucePlatformVersion "5.1" --target 
api22
--- End diff --

So, while that may be how paramedic right now was designed, that's actually 
counter-productive when working with appium. Appium will actually select the 
correct emulator (and even start it for you) based on the `platformName`, 
`platformVersion` and `deviceName` capabilities you provide. These capabilities 
apply regardless if you are running the test locally or on Sauce Labs.

I think, moving forward, for a paramedic refactor I am gathering use cases 
for / noting issues to address, we would want to expose the appium/selenium 
capabilities more completely. This would also reduce the difference between 
local test executions and executions on Sauce Labs, by allowing Appium to worry 
about managing the emulator, rather than managing it in paramedic.

I will change the instructions back, but just a heads up that I think we 
should change this behaviour in the future.


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

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



[GitHub] cordova-plugin-camera pull request #244: Android Appium test tweaks

2016-12-23 Thread filmaj
Github user filmaj commented on a diff in the pull request:


https://github.com/apache/cordova-plugin-camera/pull/244#discussion_r93798887
  
--- Diff: appium-tests/android/android.spec.js ---
@@ -373,7 +384,6 @@ describe('Camera tests Android.', function () {
 return driver
 
.waitForElementByXPath('//android.widget.TextView[@text="Gallery"]', 2)
 
.elementByXPath('//android.widget.TextView[@text="Gallery"]')
-
.elementByXPath('//android.widget.TextView[@text="Gallery"]')
--- End diff --

Do you have any details around the emulator bug on Windows? Ideally:
 - any Appium issues filed that sound similar?
 - any Android issues filed relevant?
 - Windows version?
 - Appium version?
 - Android emulator version?

I'm also up for looking into it more closely.


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

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



[GitHub] cordova-plugin-camera issue #244: Android Appium test tweaks

2016-12-23 Thread filmaj
Github user filmaj commented on the issue:

https://github.com/apache/cordova-plugin-camera/pull/244
  
@alsorokin thanks for the feedback, I've addressed your points. Let me know 
if you have any other concerns or if you are good with merging it.


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

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



[GitHub] cordova-plugin-camera issue #244: Android Appium test tweaks

2016-12-23 Thread cordova-qa
Github user cordova-qa commented on the issue:

https://github.com/apache/cordova-plugin-camera/pull/244
  
Cordova CI Build has completed successfully.

**Commit** - 
[Link](https://github.com/apache/cordova-plugin-camera/pull/244/commits/4e26e2a04ae256e4305419a9d469771dd4e0f8c8)
**Dashboard** - 
[Link](http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113/)

| Builder Name  | Console Output | Test Report | Device Logs  |
| :---: | :---:  |   :---: | :---:|
| [Windows 8.1 Store]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=windows-8.1-store/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=windows-8.1-store/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=windows-8.1-store/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=windows-8.1-store/artifact/)
 |
| [Windows 10  Store]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=windows-10-store/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=windows-10-store/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=windows-10-store/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=windows-10-store/artifact/)
 |
| [Windows 8.1 Phone]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=windows-8.1-phone/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=windows-8.1-phone/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=windows-8.1-phone/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=windows-8.1-phone/artifact/)
 |
| [iOS]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=ios/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=ios/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=ios/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=ios/artifact/)
 |
| [Android]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=android/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=android/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=android/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera-pr/113//PLATFORM=android/artifact/)
 |
 



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

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



Need help

2016-12-23 Thread Subasish Chakraborty
Hello,

I am Subasish chakraborty currently working on Cordova. I am running
Cordova in NETBEANS 8.0.1. Can you send me a simple project to access
my android phone contacts or Camera. I have trying for so long time
but I failed. Installation and everything is done only not getting how
to use the plugins. Installation of the plugin has also been done. Any
help from your side would be helpful.


Thanks and Regards,

Subasish Chakraborty

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