Отображать тесты определенного браузера #38

baymer · 2016-03-21T10:59:54Z

Скрывает тесты остальных браузеров
https://yadi.sk/i/MSyq-7E1qMWSu

j0tunn · 2016-03-21T11:36:30Z

lib/views/main.hbs

+ <option value="ie8">ie8</option>
+ <option value="chrome-latest">chrome-latest</option>
+ <option value="opera-v12.16">opera-v12.16</option>
+ <option value="firefox-latest">firefox-latest</option>


вот этот список бы не хардкодить, а из конфига gemini брать

👍, браузеры могут называться как угодно

Да, отлично было бы не хардкодить, всеми руками за. Можно вообще весь блок в шаблон вынести. Откуда список можно получить?

j0tunn · 2016-03-21T11:37:28Z

@sipayRT @SwinX глянтье плз. Меня смущает то, что все сваливается в кучу в верхнем ряду кнопок

SwinX · 2016-03-21T12:19:33Z

lib/client/controller.js

+ this._showBrowser.addEventLestener('change', function(e) {
+ var value = e.target.value;
+ byId('stylesoHideBrowsers').innerText = !value ? '' :
+ '.section[data-browser-id]:not([data-browser-id="' + value + '"]){display:none}';


Ух какая логика на селекторах. Определённо в гуй нужно добавить больше реакта.

sipayRT · 2016-03-21T12:22:22Z

@baymer положи, плз, ссылку с гифкой на какой-нибудь внешний ресурс. Нельзя оставлять ссылки внутренней сети в репозиториях внешних проектов

sipayRT · 2016-03-21T12:25:35Z

lib/client/controller.js

@@ -66,6 +67,12 @@ Controller.prototype = {
 byId('collapseAll').addEventListener('click', sections.collapseAll.bind(sections));
 byId('expandErrors').addEventListener('click', sections.expandErrors.bind(sections));

+ this._showBrowser.addEventLestener('change', function(e) {
+ var value = e.target.value;
+ byId('stylesoHideBrowsers').innerText = !value ? '' :


stylesoHideBrowsers странный ID - видимо, должно было быть stylesofHideBrowsers.
и зачем инвертировать value?

Опечатка, должно было быть stylesToHideBrowsers ("T" пропущена)

sipayRT · 2016-03-21T12:30:24Z

расскажи про реальную необходимость такого селекта? если нужно показать только один браузер, то почему не запустить просто гуй только для нужного браузера? Ну и что я должен сделать, если захочу увидеть несколько браузеров? Как будет работать мультиселект?

baymer · 2016-03-21T18:08:01Z

@sipayRT,

расскажи про реальную необходимость такого селекта?

При написании теста понадобилось смотреть и сравнивать между собой только тесты chrome, потом только тесты opera. Кажется, удобная штука. Нет?

если нужно показать только один браузер, то почему не запустить просто гуй только для нужного браузера?

Имеешь ввиду .skip(/ie|fire|oper/), чтобы увидеть только chrome?
Если да, то остальные только тесты проходить не будут, но останутся видны битые картинки.
Если имеешь ввиду передаваемый параметр при запуске (есть такой?), то в случае селекта есть возможность смотреть, то один браузер, то другой, причем довольно дешево для браузера и без пересборок.

Ну и что я должен сделать, если захочу увидеть несколько браузеров? Как будет работать мультиселект?

Мультиселект вроде и не нужен. Тут скорее кейс про "выделить конкретный браузер и сравнить акшены с плейном", например. Если нужно больше, можно выключить селект (выбрать первый дефолтный пункт).

baymer · 2016-03-21T18:09:02Z

положи, плз, ссылку с гифкой на какой-нибудь внешний ресурс

Так на внешнем же: Я.Диск.

sipayRT · 2016-03-22T08:13:48Z

Так на внешнем же: Я.Диск.

посмотри свое описание к коммиту

Имеешь ввиду .skip(/ie|fire|oper/), чтобы увидеть только chrome?
Если да, то остальные только тесты проходить не будут, но останутся видны битые картинки.
Если имеешь ввиду передаваемый параметр при запуске (есть такой?), то в случае селекта есть возможность смотреть, то один браузер, то другой, причем довольно дешево для браузера и без пересборок.

я имею ввиду опцию -b — запустить тест в конкретном браузере (при этом ты можешь заюзать эту опцию несколько раз, чтобы запустить тесты в нескольких браузерах). в результате у тебя в отчете будут только эти браузеры

https://yadi.sk/i/MSyq-7E1qMWSu

baymer · 2016-03-22T09:18:34Z

посмотри свое описание к коммиту

спасибо, поправил

имею ввиду опцию -b

Странно, в документации не нашел такой опции. Попробовал запустить gemini-gui -b chrome gemini/arrow.gemini.js и gemini-gui -b gemini/arrow.gemini.js. В обоих случаях открылись скриншоты для всех браузеров.
Если же ты имеешь ввиду npm run gemini -- -b ie8 test gemini/arrow.gemini.js, то это прогонит тесты в нужном браузере, да. Прогонит в консоли. Скриншотов я не увижу.
Если запустить npm run gemini -- -b ie8 update gemini/arrow.gemini.js; gemini-gui gemini/arrow.gemini.js и перед этим не удалять все скриншоты блока, то запуск gemini-gui покажет все скриншоты, не только по выбранному браузеру.
Если же перед запуском удалить скриншоты блока, то в результате будет такая картинка:
https://yadi.sk/i/r49npjcZqNzJv

Требуется именно в сверить скриншоты по определенному браузеру на этапе их создания или обновления. При этом можно выбрать то один браузер, то другой, без пересборки.

Если это никому не нужно, то ок. Мне понадобилось, я написал. Может еще кому пригодится:sunglasses:

a-x- · 2016-05-14T17:49:51Z

Очень нужное изменение, по крайней мере, ещё и мне!

a-x- · 2016-05-14T17:50:48Z

Кстати, в результатах прогонов в CI вообще нельзя отфильтровать по бро, а временами хочется.

sipayRT · 2016-05-16T10:31:19Z

Кстати, в результатах прогонов в CI вообще нельзя отфильтровать по бро, а временами хочется.

расскажи почему этого хочется? может мы просто найдем единое верное решение

a-x- · 2016-05-16T12:26:19Z

@sipayRT
Например, когда все браузеры ведут себя заведомо одинаково для какой-то функциональности и нужно быстро просмотреть диффы, не тратя времени на браузерные дубликаты.

В качестве универсального решения вот как можно было бы сделать с "рокетной наукой": если между браузерами скриншоты для конкретного стейта одинаковы, то кластеризовать с текстовой пометкой.

levonet added the review label Mar 21, 2016

j0tunn reviewed Mar 21, 2016
View reviewed changes

SwinX reviewed Mar 21, 2016
View reviewed changes

sipayRT reviewed Mar 21, 2016
View reviewed changes

Отображать тесты определенного браузера.

10164ba

https://yadi.sk/i/MSyq-7E1qMWSu

перенес в showBrowser в шаблон

821f6bd

baymer force-pushed the show-broswer branch from 355825a to 821f6bd Compare March 22, 2016 10:12

fix class name of style control

dd87da7

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Отображать тесты определенного браузера #38

Отображать тесты определенного браузера #38

baymer commented Mar 21, 2016

j0tunn Mar 21, 2016

SwinX Mar 21, 2016

baymer Mar 21, 2016

j0tunn commented Mar 21, 2016

SwinX Mar 21, 2016

sipayRT commented Mar 21, 2016

sipayRT Mar 21, 2016

baymer Mar 21, 2016

sipayRT commented Mar 21, 2016

baymer commented Mar 21, 2016

baymer commented Mar 21, 2016

sipayRT commented Mar 22, 2016

baymer commented Mar 22, 2016

a-x- commented May 14, 2016

a-x- commented May 14, 2016

sipayRT commented May 16, 2016

a-x- commented May 16, 2016 •

edited

Loading

Отображать тесты определенного браузера #38

Are you sure you want to change the base?

Отображать тесты определенного браузера #38

Conversation

baymer commented Mar 21, 2016

j0tunn Mar 21, 2016

Choose a reason for hiding this comment

SwinX Mar 21, 2016

Choose a reason for hiding this comment

baymer Mar 21, 2016

Choose a reason for hiding this comment

j0tunn commented Mar 21, 2016

SwinX Mar 21, 2016

Choose a reason for hiding this comment

sipayRT commented Mar 21, 2016

sipayRT Mar 21, 2016

Choose a reason for hiding this comment

baymer Mar 21, 2016

Choose a reason for hiding this comment

sipayRT commented Mar 21, 2016

baymer commented Mar 21, 2016

baymer commented Mar 21, 2016

sipayRT commented Mar 22, 2016

baymer commented Mar 22, 2016

a-x- commented May 14, 2016

a-x- commented May 14, 2016

sipayRT commented May 16, 2016

a-x- commented May 16, 2016 • edited Loading

a-x- commented May 16, 2016 •

edited

Loading