Success

User tests: Successful: Unsuccessful:

avatar javigomez
javigomez
9 Feb 2016

This pull:

  • Updates selenium to latest version 2.51.0.
  • To latests JoomlaBrowser 3.4.8v2 including this new function joomla-projects/joomla-browser#100
  • Forces Travis to use latest Firefox v44
  • Fixes errors raised with the new selenium and Firefox

I have tested in my local MacOS, let's see if Travis Ubuntu do not complain.

@810 I would appreciate if you could run it on Windows. Thanks

avatar javigomez javigomez - open - 9 Feb 2016
avatar yvesh
yvesh - comment - 9 Feb 2016

@javigomez Great work!

Tested, works fine with latest Firefox (44.0.1) on Linux!

avatar 810
810 - comment - 11 Feb 2016

+1

avatar pritalpatel
pritalpatel - comment - 16 Feb 2016

I am testing this PR. I am having issue with WaitForSeleniumStandaloneServer task. As shown below

waitforseleniumstandaloneserver

I think it's not related to this PR, but as per Gunjan's suggestion I commented $this->runSelenium(); function for temporary and then it works smooth, whole flow is working fine afterwords.

avatar pritalpatel
pritalpatel - comment - 16 Feb 2016

Forget to mention that I am using Ubuntu 14.04.

avatar puneet0191
puneet0191 - comment - 16 Feb 2016

@pritalpatel thanks :) I can see you have not enabled or installed php-curl extension on your Ubuntu machine. I would suggest you to enable the curl extension and then try again.

avatar yvesh
yvesh - comment - 16 Feb 2016

@puneet0191 it's not the curl php extension, it's the curl binary. ;)
@pritalpatel install curl -> apt-get install curl

avatar puneet0191
puneet0191 - comment - 16 Feb 2016

@yvesh thanks :) you rock! :guitar:

avatar gunjanpatel
gunjanpatel - comment - 17 Feb 2016

Thanks @puneet0191 php5-curl is by default enabled in my machine. You are right @yvesh I will help @pritalpatel to install Binary curl (We are using same machine :-D ) :+1:

avatar javigomez
javigomez - comment - 17 Feb 2016

Ok, given the 3 tests I'm merging. Actually there is a new release of Selenium, so I'll send the new update in a new pull.

avatar javigomez javigomez - reference | f98b43d - 17 Feb 16
avatar javigomez javigomez - merge - 17 Feb 2016
avatar javigomez javigomez - close - 17 Feb 2016

Add a Comment

Login with GitHub to post a comment