-
Notifications
You must be signed in to change notification settings - Fork 64
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
chore: update webdriverio to 8.21.0 #803
Conversation
package.json
Outdated
@@ -74,7 +74,7 @@ | |||
"urijs": "^1.19.11", | |||
"url-join": "^4.0.1", | |||
"uuid": "^9.0.1", | |||
"webdriverio": "8.13.4", | |||
"webdriverio": "8.22.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.
Использую exact версию так как в прошлый раз версия 8.14.0 сломала корректность работы hermione
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.
Тут обновил до 8.21, а не 8.22 (вышла сегодня). Так как в 8.22 выгасили тип matches
, который у нас используется в openAndWait
и добавили депрекейшн варнинги для jsonwp команд.
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.
Оказалось 8.21 тоже немогу поставить (выехал только 6 дней назад). Ставлю 8.20.4
@@ -157,7 +157,7 @@ module.exports = class ExistingBrowser extends Browser { | |||
...this._getSessionOptsFromConfig(OPTIONAL_SESSION_OPTS), | |||
...detectedSessionEnvFlags, | |||
...this._config.sessionEnvFlags, | |||
options: _.pick(sessionOpts, "automationProtocol"), | |||
options: sessionOpts, |
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.
Проблема была в этом. С версии 8.14.0 опции для подключения к хосту теперь берутся из options
.
|
||
beforeEach(() => { | ||
({ mkExistingBrowser_: mkBrowser_ } = proxyquire("../utils", { | ||
webdriverio: { attach: wdioAttachStub, "@global": true }, | ||
"./client-bridge": { build: sandbox.stub().resolves(), "@global": 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.
Тут пришлось использовать global
, иначе нужно было бы писать вложенные proxyquire
-ы. Решил, что это меньшее из зол.
assert.calledWithMatch(webdriverio.attach, { foo: "bar" }); | ||
await initBrowser_(mkBrowser_(), { sessionOpts }); | ||
|
||
assert.calledWithMatch(webdriverio.attach, { ...sessionOpts, options: sessionOpts }); |
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.
Отдельный тест не стал добавлять, так как эти опции нужно и просто в options
прокинуть и дестрктурировать.
49af9f9
to
7df27f1
Compare
a5d318e
to
8a12987
Compare
94fb291
to
cf619df
Compare
@@ -56,6 +56,7 @@ | |||
"clear-require": "^1.0.1", | |||
"date-fns": "^2.29.3", | |||
"debug": "^2.6.9", | |||
"devtools": "8.21.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.
Теперь при использовании automationProtocol: devtools
нужно пакет самостоятельно подносить.
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.
А webdriver в wdio уже встроен?
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.
А webdriver в wdio уже встроен?
Да.
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.
/ok
@@ -56,6 +56,7 @@ | |||
"clear-require": "^1.0.1", | |||
"date-fns": "^2.29.3", | |||
"debug": "^2.6.9", | |||
"devtools": "8.21.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.
А webdriver в wdio уже встроен?
No description provided.