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 support to use WSEndpoint on Puppeteer #1350

Merged
merged 1 commit into from
Dec 19, 2018
Merged

Add support to use WSEndpoint on Puppeteer #1350

merged 1 commit into from
Dec 19, 2018

Conversation

gabrielcaires
Copy link

Hello,
With this PR will be possible to use remote chrome instances like https://www.browserless.io/ or docker containers like https://github.com/joelgriffith/browserless.

I saw that currently the project is not mocking Puppeteer and we don't have tests to _startBrowser, so I don't know how test it. Should I mock Puppeteer?

@@ -59,7 +59,8 @@ const consoleLogStore = new Console();
*
* ```js
* "chrome": {
* "executablePath" : "/path/to/Chrome"
* "executablePath" : "/path/to/Chrome",
* "browserWSEndpoint": "ws://localhost:3000"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks. Can you add here an example here for using CodeceptJS with browserless.io?

@DavertMik
Copy link
Contributor

DavertMik commented Nov 23, 2018

I think for such case we are fine without a test
Just update the docs and we are fine!

@DavertMik DavertMik merged commit 15c7595 into codeceptjs:master Dec 19, 2018
@DavertMik
Copy link
Contributor

Ok, I will update docs on my own

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