feat: use bluebird instead q #155
Conversation
DudaGod
force-pushed
the
feature/use-bluebird
branch
from
42afab8
to
93cc7bb
Compare
| }; | ||
|
|
||
| function emitBrowserEvent(emitter, browser, event) { | ||
| return emitter.emitAndWait(event, browser.publicAPI, {browserId: browser.id}); | ||
| const browserSession = Object.assign(browser.publicAPI, {browserId: browser.id}); |
There was a problem hiding this comment.
В текущей реализации passthrough-emitter-а - https://github.com/gemini-testing/gemini-core/blob/3979b225ff19b10475ae96a1de879a4019e141ee/lib/passthrough-emitter/index.js
ожидается, что метод emitAndWait будет вызываться с двумя аргументами. В данном случае SESSION_START и SESSION_END эмитится с 3 аргументами. Мое предложение объединть wd и id браузера в один объект. Сейчас я модифицирую wd, т.к. у нас сейчас старая версия лодаша. В отдельном PR-е перееду на последнюю версию лодаша и тогда тут можно будет юзнуть так:
const browserSession = _.extend({browserId: browser.id}, browser.publicAPI);Так же полистал по плагинам, третий аргумент юзают только stat-reporter и url-decorator.
There was a problem hiding this comment.
фигню какую-то написал, так работать не будет. Так как лодашовский extend не добавит к новому объекту все методы инстанса webdriverio. Поэтому пока оставлю текущий вариант:
const browserSession = Object.assign(browser.publicAPI, {browserId: browser.id});
DudaGod
force-pushed
the
feature/use-bluebird
branch
3 times, most recently
from
f5c823d
to
2c6d66a
Compare
lib/constants/runner-events.js
Outdated
| let events = _.assign(getSyncEvents(), getAsyncEvents()); | ||
|
|
||
| Object.defineProperty(events, 'getSync', {value: getSyncEvents, enumerable: false}); | ||
| Object.defineProperty(events, 'getAsync', {value: getAsyncEvents, enumerable: false}); | ||
| Object.defineProperty(events, 'getAll', {value: getAllEvents, enumerable: false}); |
DudaGod
force-pushed
the
feature/use-bluebird
branch
from
57c80a3
to
de252cb
Compare
DudaGod
force-pushed
the
feature/use-bluebird
branch
from
de252cb
to
dd27d14
Compare
|
надо рибейзить |
|
@DudaGod Здесь кажется проще новый PR создать, чем этот отрибейзить |
And also use 'AsyncEmitter', 'PassthroughEmitter', 'promiseUtils' from gemini-core. Instead of qemitter and q-promise-utils libraries