Repository navigation
Ability to run tests in different versions of browsers - #73
Conversation
There was a problem hiding this comment.
Что осталось:
- не забывать выгасить коммит с тестирование
- неплохо было бы тестов на limited-pool докинуть
в остальном /ok
|
|
||
| _initPool(browserId, version) { | ||
| const compositeId = buildCompositeBrowserId(browserId, version); | ||
| const freeBrowser = this.underlyingPool.freeBrowser.bind(this.underlyingPool); |
There was a problem hiding this comment.
что же ты такой любитель bind-ов, можно же спокойно без него обойтись
There was a problem hiding this comment.
я просто перенес код из конструктора)
ну и в данном случае bind явный, а если написать:
const freeBrowser = (...args) => this.underlyingPool.freeBrowser(...args);ну такоееее...
There was a problem hiding this comment.
Мы bind не юзаем, так как после него ты такую функцию ни с каким другим контекстом выполнения уже не вызовешь. Поэтому такие проблемы решаем с помощью стрелочных функций или .call
There was a problem hiding this comment.
Ну и зря не юзаете) Нужно отталкиваться от задачи. Не нужно хотеть перебиндить контекст метода, если это метод класса и он опирается на его внутренний стейт.
| } | ||
|
|
||
| const {hasFreeSlots} = options; | ||
| const hasCacheForNextRequest = this._caches[options.compositeIdForNextRequest]; |
There was a problem hiding this comment.
ты тут пулы веревками связал, так как ожидаешь, что другой пул прокинет необходимые опции
There was a problem hiding this comment.
да, по-другому не добраться до лимитов( собственно говоря, пулы сами по себе связанны по своей логике и как-то вложить по-другому их не получится.
abcc75a to
3db5690
Compare
|
|
||
| _initPool(browserId, version) { | ||
| const compositeId = buildCompositeBrowserId(browserId, version); | ||
| const freeBrowser = this.underlyingPool.freeBrowser.bind(this.underlyingPool); |
There was a problem hiding this comment.
Мы bind не юзаем, так как после него ты такую функцию ни с каким другим контекстом выполнения уже не вызовешь. Поэтому такие проблемы решаем с помощью стрелочных функций или .call
| }); | ||
| }); | ||
|
|
||
| describe('freeBrowser', () => { |
3ba5729 to
93d4e54
Compare
No description provided.