Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #275: Update automated testing suite to use ChromeDriver #312

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

t4k
Copy link
Contributor

@t4k t4k commented Oct 19, 2017

This PR replaces #305 and #278, and addresses #275.

Note: The version of chromedriver is locked at 2.32 right now due to php-webdriver/php-webdriver#469 which is for implementing the latest W3C specification for WebDriver.

I’ve removed all the functional and unit test files to lessen confusion because we do not have and cannot run any of those tests.

REVIEW PROCEDURE

Follow the instructions in TESTS.md on a local/test system and see if they work for you.

I’ve spent a good amount of time trying this in different environments. I have it working in Ubuntu 14.04 with PHP 5.6 and 16.04 with PHP 7.1.

Please Note: The tests will likely fail, but the testing environment should be set up properly. I will file more issues for fixes to the existing tests.

@veggiematts
Copy link
Contributor

I followed the procedure, but got:

coral@coral:~/www$ bin/chromedriver --url-base=/wd/hub --port=9515
bin/chromedriver: error while loading shared libraries: libgconf-2.so.4: cannot open shared object file: No such file or directory

After installing libgconf-2-4, all went fine, but that's maybe something worth mentioning in the install procedure.

Results of the tests:
Tests: 5, Assertions: 2, Errors: 4.

@OndraM
Copy link

OndraM commented Oct 25, 2017

FYI, just accidentally noted this issue - the chromedriver should not be affected by the W3C protocol, it should be fully working with the JsonWire protocol, so IMO you don't have to lock the chromedriver version.

@t4k
Copy link
Contributor Author

t4k commented Oct 26, 2017

@veggiematts What are your system specs? OS? PHP version?

Thanks for that note, @OndraM, I’m not sure what the issue is then. I was basing my assumption on Codeception/Codeception#4556 because my error with Codeception was:

[Facebook\WebDriver\Exception\SessionNotCreatedException] session not created exception: Chrome version must be >= 60.0.3112.0
(Driver info: chromedriver=2.33.506092 (733a02544d189eeb751fe0d7ddca79a0ee28cce4),platform=Linux 4.4.0-62-generic x86_64)

Results of chromium-browser --version:

Chromium 61.0.3163.100 Built on Ubuntu , running on Ubuntu 16.04

Everything works on Ubuntu 14.04.

@OndraM
Copy link

OndraM commented Oct 26, 2017

@t4k Not quite sure, but it works for me just fin:

$ chromedriver --version
ChromeDriver 2.33.506092 (733a02544d189eeb751fe0d7ddca79a0ee28cce4)
$ chromium --version
Chromium 61.0.3163.100 

Are you sure Selenium runs proper Chromium executable? You may also find some more output in Selenium server log.

@t4k
Copy link
Contributor Author

t4k commented Oct 27, 2017

@OndraM I’m not using it with Selenium. As the Codeception docs state at http://codeception.com/docs/03-AcceptanceTests#ChromeDriver :

ChromeDriver … can be … used as a standalone tool to drive Chrome browser.

and that’s how we’re using it in this project.

I will file a more detailed report with the Codeception project first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants