-
Notifications
You must be signed in to change notification settings - Fork 15k
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
docs: Updated docs on testing Electron using WebdriverIO #39380
docs: Updated docs on testing Electron using WebdriverIO #39380
Conversation
007cf4f
to
e5963b7
Compare
9eb98b4
to
19d08b3
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.
@christian-bromann - someone should be round to review this more thoroughly soon, but in the meantime lint is failing due to this issue:
docs/tutorial/automated-testing.md:43:69 MD049/emphasis-style Emphasis style should be consistent [Expected: asterisk; Actual: underscore]
checks should go green once that's addressed!
notes: Updated docs on testing Electron using WebdriverIO
19d08b3
to
a55d39d
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.
First pass on the content
chromedriver: { | ||
logFileName: 'wdio-chromedriver.log' | ||
}, | ||
electronVersion: '23.1.0' |
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.
If we're reading the package.json
, shouldn't this be derived from the project's devDependencies
?
// ... | ||
} | ||
``` | ||
|
||
The Electron service takes care of downloading the right Chromedriver version for you, updating | ||
your capabilities to point to your Electron application as well as offers useful testing features |
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.
your capabilities to point to your Electron application as well as offers useful testing features | |
your [ChromeDriver capabilities](https://chromedriver.chromium.org/capabilities) to point to your Electron application as well as offers useful testing features |
We didn't do this in the previous version of the docs either, but I think it would be helpful to link to ChromeDriver's definition of what capabilities
are.
Closing in favor of #40083 |
Description of Change
Recommend users a better setup for testing Electron Applications with WebdriverIO.
Checklist
npm test
passesRelease Notes
Notes: Updated docs on testing Electron using WebdriverIO