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 tunnel #19

Closed
csnover opened this issue Jul 29, 2015 · 3 comments
Closed

Add Selenium tunnel #19

csnover opened this issue Jul 29, 2015 · 3 comments

Comments

@csnover
Copy link
Member

csnover commented Jul 29, 2015

Add a new tunnel provider that downloads and runs Selenium (and configurable optional dependencies like ChromeDriver, IEDriverServer, etc.) on the local machine.

This should be pretty easy since TestingBot already uses a Java application so that code can pretty much just be taken and lightly edited into a new tunnel.

@sholladay
Copy link

This would be fantastic. A couple of "food for thought" notes:

  • Another project that does this well is grunt-selenium-webdriver, which I have traditionally used because of its reliability and caching of the binary. It has some intelligence to make things "just work", even in edge cases. Might be worth looking at when implementing this.

  • I often override the default Selenium port of 4444, because OS X Server hogs it (which leads to complaints from other developers about EADDRINUSE errors). Ideally, in my Intern config, I would be able to say something like:

    tunnelOptions : {
        port : 0  // or 'auto', etc.
    }

    ... and this would use an available port.

@rodneyrehm
Copy link

I'd like to see this feature as well. As of now I have external work-arounds (e.g. PhantomJS2) to spin up a browser instance to run intern against, and quit the browser afterwards.

@jason0x43
Copy link
Member

This was added in d289ec9 and will be included in the next release of Dig Dug. Thanks @devpaul!

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

Successfully merging a pull request may close this issue.

4 participants