-
-
Notifications
You must be signed in to change notification settings - Fork 413
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: add additionalWebviewBundleIds capability #1117
Conversation
lib/desired-caps.js
Outdated
@@ -192,6 +192,9 @@ let desiredCapConstraints = _.defaults({ | |||
includeSafariInWebviews: { | |||
isBoolean: true | |||
}, | |||
additionalWebviewBundleIds: { | |||
isArray: true |
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.
I'm thinking about making it a comma separated string instead. I see people sometimes having problems poviding arrays to caps, especially in Java
@@ -70,6 +70,7 @@ extensions.getNewRemoteDebugger = async function getNewRemoteDebugger () { | |||
} | |||
return createRemoteDebugger({ | |||
bundleId: this.opts.bundleId, | |||
additionalBundleIds: this.opts.additionalWebviewBundleIds, |
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.
this.opts.additionalWebviewBundleIds || []
?
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.
This is done on the remote debugger side.
e969516
to
5950d23
Compare
@@ -192,6 +192,10 @@ let desiredCapConstraints = _.defaults({ | |||
includeSafariInWebviews: { | |||
isBoolean: true | |||
}, | |||
additionalWebviewBundleIds: { | |||
// recognize the capability | |||
// but validate in driver#validateDesiredCaps |
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.
👀
* feat: add additionalWebviewBundleIds capability * fix: allow JSON array for cap
Pass additional webview bundle identifiers along to the remote debugger.