-
Notifications
You must be signed in to change notification settings - Fork 228
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
WebdriverIO v5 #349
Comments
It would be awesome to get webdriver 5 supported as it comes with significantly improved typescript typings. |
Hey 👋 , this is Christian from the WebdriverIO team. Is there any chance I can help with this issue? @eostrom ? Cheers |
Hi @christian-bromann. I've moved on from the project I was using Spectron for, but if you want to pursue the WebdriverIO upgrade you are more than welcome to use the branch above as a starting point. There's just one commit, and it's mostly renaming things... plus it looks like I simplified client initialization using a promise returned from There may be mistakes, I wasn't able to test thoroughly. |
@christian-bromann, if this is an issue that is being picked up and worked on - myself and I know a few others would love to help out as well. This fix would help streamline to have our webpage tests line up with our electron tests, which we would love to see! |
@codebytere is this being planned to get picked up or is community involvement expected? |
I can try to help but unfortunately the core team does not have the bandwidth to proritize this right now. |
#### Description of changes We only use the webdriverio package for the sake of getting typings to use with Spectron, so we want to keep its major version pinned to the major version spectron uses (v4) until spectron updates (see electron-userland/spectron#349 ) Verified that the updated file passes the [dependabot config file validator](https://dependabot.com/docs/config-file/validator/). #### Pull request checklist - [n/a] Addresses an existing issue: Fixes #0000 - [n/a] Added relevant unit test for your changes. (`yarn test`) - [n/a] Verified code coverage for the changes made. Check coverage report at: `<rootDir>/test-results/unit/coverage` - [x] Ran precheckin (`yarn precheckin`) - [n/a] (UI changes only) Added screenshots/GIFs to description above - [n/a] (UI changes only) Verified usability with NVDA/JAWS
Is there any progress on this? |
+1 to this, would love to see spectron upgraded to the latest webdriverio --> v6 just came out a few days ago. https://www.npmjs.com/package/webdriverio Right now I get npm install warnings since spectron has a dependency chain to an outdated versions of core-js, request, and other packages through webdriverio. I even get a warning about webdriverio being outdated: |
It would be very appreciated to move to WebDriverIO v5 or v6 since the v4 website is no longer online anymore. Fortunately, the API page is still available. For now, adding |
PR is now up for updating to v5: |
WebdriverIO 5.0 is out. I started modifying Spectron to use it. I didn't get very far, but if anyone else wants to press forward, my work is in https://github.com/eostrom/spectron/tree/webdriverio-5.
The text was updated successfully, but these errors were encountered: