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.
Использую exact версию так как в прошлый раз версия 8.14.0 сломала корректность работы hermione
There was a problem hiding this comment.
Тут обновил до 8.21, а не 8.22 (вышла сегодня). Так как в 8.22 выгасили тип matches, который у нас используется в openAndWait и добавили депрекейшн варнинги для jsonwp команд.
There was a problem hiding this comment.
Оказалось 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.
Проблема была в этом. С версии 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.
Тут пришлось использовать 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.
Отдельный тест не стал добавлять, так как эти опции нужно и просто в options прокинуть и дестрктурировать.
DudaGod
changed the title
DudaGod
force-pushed
the
HERMIONE-1242.up_wdio
branch
6 times, most recently
from
49af9f9
to
7df27f1
Compare
DudaGod
force-pushed
the
HERMIONE-1242.up_wdio
branch
3 times, most recently
from
a5d318e
to
8a12987
Compare
DudaGod
force-pushed
the
HERMIONE-1242.up_wdio
branch
2 times, most recently
from
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.
Теперь при использовании automationProtocol: devtools нужно пакет самостоятельно подносить.
There was a problem hiding this comment.
А webdriver в wdio уже встроен?
There was a problem hiding this comment.
А webdriver в wdio уже встроен?
Да.
| @@ -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.
А webdriver в wdio уже встроен?
No description provided.