у него их 620 упало

мне надо просмотреть все 800 снепшотов?))

Почему 800 то? У меня тупо только истории снапшотами тестятся

у него их 620 упало

620 уникальных страниц?

620 уникальных страниц?

страниц?

ахзахаха

там на компоненты

Так нахуя компоненты тестить на снепшоты

Почему 800 то? У меня тупо только истории снапшотами тестятся

какие ещё истории?

Это на всю страницу делается обычно=)

какие ещё истории?

Storybook

Storybook

бля

Storybook

у меня на каждый компонент есть по несколько стори

у меня на каждый компонент есть по несколько стори

Я одну делаю

сочувствую

но опять же на каждый компонент

там даже компонентов дохрена

они между собой пересекаются

я все равно не вижу смысла в снапшотах

сочувствую

например для тестирования состояния компонентов

(бабель плагины ок)

я все равно не вижу смысла в снапшотах

по факту нихуя не проверяют

упало — сам разбирайся

даже если правильно, упало

пиздец, так тесты не должны работать

по факту нихуя не проверяют

ну а как ты верстку подругому протестишь?

а что в ней тестить?

пиздец, так тесты не должны работать

Понятное дело что это не замена обычным тестам, максимум визуальные регрессии палить

да и как ты собрался тестить css modules или cssinjs

которые будут всегда падать

а что в ней тестить?

ну, что состояния рисуются, вложенные компоненты рендерятся

Понятное дело что это не замена обычным тестам, максимум визуальные регрессии палить

визуальные не спалишь, если в css сломали

ну, что состояния рисуются, вложенные компоненты рендерятся

пропсы правильные приходят

да и как ты собрался тестить css modules или cssinjs

Styled не всегда падают

ну, что состояния рисуются, вложенные компоненты рендерятся

это и так проверяется обычными тестами

пропсы правильные приходят

это обычные тесты проверяют

Styled не всегда падают

о, ну уже круто будет не всегда падать а только в части случаев на ci будет падать чуть реже чем всегда

о, ну уже круто будет не всегда падать а только в части случаев на ci будет падать чуть реже чем всегда

короче с таким подходом) не нужны они)

короче с таким подходом) не нужны они)

так блядь нормальные люди делают

ci настраивают

так блядь нормальные люди делают

я имею ввиду тестирование снепшотами

бля

о, ну уже круто будет не всегда падать а только в части случаев на ci будет падать чуть реже чем всегда

Почему им там падать? Ты же обновляешь снапшот перед коммитом

Почему им там падать? Ты же обновляешь снапшот перед коммитом

пиздец

я все равно не вижу смысла в снапшотах

сегодня выпилили нахуй

снепшоты сторибуков

заебало

99% это фиксы того что где-то чуток поменялось и по пизде все дерево цсс классов пошло

в css-modules

пиздец

ну ты же ранаешь тесты перед реквестом?

Почему им там падать? Ты же обновляешь снапшот перед коммитом

так нахуя они нужны если будут обновлены перед коммитом и там вполне может лежать падение

буду прикручивать тесты скринами

ну ты же ранаешь тесты перед реквестом?

когда как

99% это фиксы того что где-то чуток поменялось и по пизде все дерево цсс классов пошло

вот, вот блядь

буду прикручивать тесты скринами

расскажешь потом саксесс стори

ну у меня дифф скринов после деплоя отображается

расскажешь потом саксесс стори

Так там прикручиваетс на раз два

https://github.com/tsuyoshiwada/storybook-chrome-screenshot - рекомендую, только альфу не ставьте

ну у меня дифф скринов после деплоя отображается

хорошая тема

но опять же false-positive как чекать

но опять же false-positive как чекать

это что?

https://github.com/tsuyoshiwada/storybook-chrome-screenshot - рекомендую, только альфу не ставьте

а для react-cosmos есть такое?

ERROR: S client not available

Я бы сказал что единственное правильная. А то открываешь PR - ребята я тут багу поправил. А что и где поправил - пойди узнай.

Я использую plural формат. Гугли react intl plural. В readme есть пример использования. Подробнее в документации formatjs https://formatjs.io/guides/message-syntax/#plural-format

а в самом json файле переводом нельзя держать строку, которая включает другие подстроки? типа если один перевод - это конкатенация первого + слово какое-то + перевод по другому ключу

ну у меня дифф скринов после деплоя отображается

какой профит с этого ? если скрины не сойдуться, надо эталон обновлять?

это что?

когда ты сделал изменения, тесты показывают что ошибка, мол изменения как в снепшотах

а для react-cosmos есть такое?

Не подскажу, вооще скриншот сделать - 6 строк на puppeteer

когда ты сделал изменения, тесты показывают что ошибка, мол изменения как в снепшотах

ничо не пони

какой профит с этого ? если скрины не сойдуться, надо эталон обновлять?

Имхо это не должно ронять тесты, никогда.

https://github.com/tsuyoshiwada/storybook-chrome-screenshot - рекомендую, только альфу не ставьте

спасибо

когда ты сделал изменения, тесты показывают что ошибка, мол изменения как в снепшотах

у меня ci сервер делает скриншоты селениумом, и нередко бывают false-positive из-за мигающего скриншота или подобной чуши

ничо не пони

есть предыдущий скрин ты меняешь margin положение меняется скрины не совпадают ошибка или просто дифф?

когда ты сделал изменения, тесты показывают что ошибка, мол изменения как в снепшотах

Смотришь имена тестов, если какой то компонент упал который ты думал не упадет проверяешь. Но когда их 800 это трудно канешн

спасибо

Я там мастер поправил, но он зараза не релизит исправленый пакет.

есть предыдущий скрин ты меняешь margin положение меняется скрины не совпадают ошибка или просто дифф?

обычный рабочий кейс

что-то поменялось — надо чекнуть

это норм

Смотришь имена тестов, если какой то компонент упал который ты думал не упадет проверяешь. Но когда их 800 это трудно канешн

да это всегда так особенно если меняешь ui

что-то поменялось — надо чекнуть

эталон обновлять как?

руками

смотришь диффы если ок запускаешь yarn update_screenshots например

руками

как снепшоты, только не падают?

да

гуд

Зачем? Пушим скрины вместе с ПР. Смотрим на изменения вместе в code changes и молча апрувим если картинка нравиться. Задача чисто показать разницу в картинках, заместо обновления снепшота - мерж.

Зачем? Пушим скрины вместе с ПР. Смотрим на изменения вместе в code changes и молча апрувим если картинка нравиться. Задача чисто показать разницу в картинках, заместо обновления снепшота - мерж.

вот да, это было бы идеально

Зачем? Пушим скрины вместе с ПР. Смотрим на изменения вместе в code changes и молча апрувим если картинка нравиться. Задача чисто показать разницу в картинках, заместо обновления снепшота - мерж.

только надо бы это настроить

Мы так для финансовых документов делаем. Сами документы на реакте, изменения в коде как-то совсем не отражают суть. Тут же идет пара скрипшотов, по которым все понятно. Изменения кода должны finance апрувить, они програть не умеют

только надо бы это настроить

Проблема что на гит хук не повесить - скриншотилка чтука не быстрая

Проблема что на гит хук не повесить - скриншотилка чтука не быстрая

ну на ci можно ж

Но это же должно быть закомичесно в гит, чтобы светиться как часть PR. В общем мы эту проблему не решили. Просто перед открытием PR запускаем скриншоты. CI чисто даты файлов проверяет, чтобы бы не забыли их сделать

Проблема что на гит хук не повесить - скриншотилка чтука не быстрая

на препуш ок