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
feat: ability to send headers when connect to browser using puppeteer #9386
feat: ability to send headers when connect to browser using puppeteer #9386
Conversation
This caused some type issues due to file refactoring on the Protractor side. You can resolve them checking into the |
1475fca
to
4609583
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Seems that there are still build issues:
|
4609583
to
0a08c31
Compare
Yeah, I see. I will ping you when all the tests pass successfully. |
14fbdf6
to
89ebc5e
Compare
89ebc5e
to
638e977
Compare
I finally fixed everything. Can you review it now? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work 👍 one question
/** | ||
* fails due to "Unable to identify the main resource" | ||
* https://github.com/webdriverio/webdriverio/issues/8541 | ||
*/ | ||
it.skip('should allow to do performance tests', async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this failing for v7
as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, with the same error. So I found that this test skipped in v8 and skip it as well for v7.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 👍
@christian-bromann can you publish new wdio@7 version to npm? Thank you. |
@DudaGod This version could not be built with ts |
Are you talking about webdriverio@7.28.0 - https://github.com/webdriverio/webdriverio/releases/tag/v7.28.0 ? |
@DudaGod I had to revert this change due to the fact that it breaks for folks using Node.js v12 and v13. Maybe this change can be applied without updating Puppeteer. |
Sorry, I did not notice that the required version of node.js has changed in puppeteer-core.
Only if I send a new PR for puppeteer-core@14, but 5 majors have already been released, so they are unlikely to agree to merge my changes. I will try to move to wdio@8 |
Proposed changes
Support changes from #9354 in wdio@7.
Reviewers: @webdriverio/project-committers