First implementation #1
Conversation
unlok
force-pushed
the
release/0.1.0
branch
3 times, most recently
from
260aa76
to
64e25d7
Compare
unlok
force-pushed
the
release/0.1.0
branch
2 times, most recently
from
c967b2c
to
a5b4d87
Compare
| '70.2': (test) => test.title === 'good for 70.2', | ||
| }, | ||
| filrefox: { | ||
| '80': (test) => test.title === 'good for 80', |
There was a problem hiding this comment.
нужна валидация версий относительно версий в основном конфиге hermione
There was a problem hiding this comment.
Этот плагин исключительно про установку версий браузера. Неважно, какие версии есть, важно, что мы хотим установить. Логика, где из конфига берется версия и прокидывается дальше - в другом месте и покрыта тестами. Версии браузера в конфиге может и вовсе не быть.
| return; | ||
| } | ||
|
|
||
| test.browserVersion = browserVersion; |
There was a problem hiding this comment.
c collection.eachTest есть проблема: он итерирует по копиям тестов, и если выставить какое-то поле здесь, а потом поиграться с enable/disable (например в gui), то вот это поле сбросится.
Кажется, что нужно переписать TestCollection
| ## Installation | ||
|
|
||
| ```bash | ||
| npm install hermione-browser-version-changer |
There was a problem hiding this comment.
почему бы не назвать просто hermione-browser-version?
There was a problem hiding this comment.
Ну вот как-то обсудили и так назвали. Лично я тоже был за hermione-browser-version =)
С другой стороны browser-version ни о чем не говорит, а так сразу понятно.
No description provided.