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

Add Selenium headless driver #272

Merged

Conversation

xtrasimplicity
Copy link
Contributor

@xtrasimplicity xtrasimplicity commented Sep 5, 2019

Background

The selenium_billy driver doesn't appear to support headless mode, and also (though unrelated) throws a Selenium [DEPRECATION] :profile is deprecated. Use Selenium::WebDriver::Firefox::Options#profile= instead. warning.

Summary

I've simply refactored the selenium_billy driver so that we're no longer using the deprecated method of setting a profile, and then added a selenium_headless_billy driver with headless support.

Fixes "Selenium [DEPRECATION] :profile is deprecated. Use Selenium::WebDriver::Firefox::Options#profile= instead." warning
@xtrasimplicity
Copy link
Contributor Author

Note: The CI failures aren't to do with my code -- looks like there's an issue with dependencies in the CI environment itself.

@ronwsmith
Copy link
Collaborator

@xtrasimplicity Thanks for the contribution! If you have a chance to try addressing the CI issue, please feel free, so I'll try to tackle in a few weeks.

@xtrasimplicity
Copy link
Contributor Author

Thanks, @ronwsmith. I'll take a look and will submit a new PR for the CI issue.

@xtrasimplicity
Copy link
Contributor Author

Scratch that, someone else has come up with a CI fix as part of another PR: #273 :)

@xtrasimplicity
Copy link
Contributor Author

Hi @ronwsmith, any updates on this and #273? Thanks!

@ronwsmith
Copy link
Collaborator

@xtrasimplicity I'm back! Thanks for your contribution and patience.

@ronwsmith ronwsmith merged commit 3d64d3e into oesmith:master Oct 27, 2019
@xtrasimplicity xtrasimplicity deleted the Feature/HeadlessSeleniumDriver branch October 27, 2019 02:50
@ronwsmith
Copy link
Collaborator

Released in v2.2.0

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

Successfully merging this pull request may close these issues.

2 participants