feat: upgrade WebdriverIO to v9, drop JWP capabilities#2852
Conversation
🦋 Changeset detectedLatest commit: c8d8352 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
web-padawan
force-pushed
the
feat/wdio-v9
branch
5 times, most recently
from
fbc632d to
7203a0e
Compare
web-padawan
force-pushed
the
feat/wdio-v9
branch
from
7203a0e to
e755f6b
Compare
Westbrook
left a comment
Westbrook
left a comment
There was a problem hiding this comment.
Seems fine to me. Two questions and then likely good to go.
|
|
||
| function getPlatform(c: WebDriver.DesiredCapabilities): string | undefined { | ||
| return c.platformName || c.platform; | ||
| function getPlatform(c: WebdriverIO.Capabilities): string | undefined { |
There was a problem hiding this comment.
Where does this file get this type value from? If it works, it works, but confused that there is no import. The main question is if we're doing something special somewhere else, or relying on something that I can't see, then is it visible to a more experienced WebdriverIO user in a way that would prevent their consumption of this from causing them issues?
There was a problem hiding this comment.
I tried to use this but got error "Cannot use namespace 'Capabilities' as a type".
import type { Capabilities } from '@wdio/types';
The namespace is defined here, seems like it's done for supporting custom capabilities.
There was a problem hiding this comment.
Does it run fine in other projects this way. I'm fine with whatever works, it was just surprising to see and would be annoying if this caused some undocumented requirements on consumers.
# Conflicts: # package-lock.json
3 participants
What I did
webdriverioto v9 and changed the saucelabs launcher accordinglyNote
Tested locally by running tests for
@web/test-runner-webdriverpackage. These don't currently run in CI as they were skipped long ago, in particular due to webdriverio/selenium-standalone#788 - that issue is now fixed but unfortunately the fix landed in a version that dropped Node 18 support.