yndx-shri-reviewer / shri-2020-task-1

Задание 1. Реализуйте дизайн-систему — ШРИ, Москва, 2020
https://yandex.ru/promo/academy/shri
44 stars 103 forks source link

Ширина страницы screen-m #55

Closed bemdev closed 4 years ago

bemdev commented 4 years ago

Приветствую. Не проходят тесты

Снимок экрана 2020-01-07 в 19 41 03

Судя по всему это актуальный провал тестов. У меня вопрос.

Есть брейкпоинт 1024px при нем все блоки должны перестроиться друг под друга. Все так и сделано. Но кажется что то не так делаю.

Ширина страница screen-m это какая то переменная которая указывает ширину для блоков layout и grid ? не вижу в дереве вообще использование такого модификатора. или явного указания где то используется.

Помогите советом, куда смотреть. Написаны тесты ко всем блокам и они проходят с небольшим трешхолдом. Но при сдачи на автотесты все валится (

Просмотрел скриншоты после Ваших тестов, и такое чувство что картинки которые даны как эталон не совсем те или я делаю что то не так ,(

Возможно Вы мне скажите, что трешхолд 0.01 и мне исправить помарки.

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

stas-laniuk commented 4 years ago

Ширина страница screen-m это какая то переменная которая указывает ширину для блоков layout и grid ?

Нет, это величина, которая определяет на какой ширине вьюпорта поведение блоков изменится. Меньше 1024p - блоки строятся для отображения на мобильном устройстве, больше или равно 1024px - блоки строятся для отображения на десктопе.

Трешхолд однозначно не меньше пикселя, скорее всего - больше.

Просмотрел скриншоты после Ваших тестов, и такое чувство что картинки которые даны как эталон не совсем те или я делаю что то не так ,(

Скриншоты представляют собой три изображения, "склеенных" вместе. Левая треть - ваша верстка, центральная треть - наложение вашей верстки на эталонную, правая треть - эталон.

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

А от такой самодеятельности лучше отказаться. Сделайте так, как требует задание (даже если в задании где-то что-то не совсем правильно с точки зрения дизайна и UX).

bemdev commented 4 years ago

index_desktop

такая погрешность фатальна?

И может быть, что тесты как то залипают. Страница product была переделана, но скришот приходит старый. Причем как видно ранее я делал соответсвенно эталонному изображению из assets

product_desktop

Специально еще раз проверил, что происходит локально. И так делается как в левой части.

stas-laniuk commented 4 years ago

Если бы тесты залипали, по этой теме был бы не только ваш комментарий. Что касается фатальности погрешности, уверен, что отсутствие изображений-заглушек фатально. Думаю, как добьетесь их отображения, страница index у вас автотест пройдет.

bemdev commented 4 years ago

Если бы тесты залипали, по этой теме был бы не только ваш комментарий. Что касается фатальности погрешности, уверен, что отсутствие изображений-заглушек фатально. Думаю, как добьетесь их отображения, страница index у вас автотест пройдет.

На сколько мне известно, картинки не будут загружены при тестах. https://github.com/yndx-shri-reviewer/shri-2020-task-1/issues/34

Есть подозрение, что это очень общее описание провалов тестов. И проблему стоит искать в любых не совпадениях. Хотя кому то удалось, пройти быстро.

Вообщем интересно очень делать задание которые видоизменяется "когда захочется".

..и по секрету во втором задании куда приятнее лог ошибок.

yndx-shri-reviewer commented 4 years ago

такая погрешность фатальна?

Если автотесты не проходят, то да. И судя по тому, что красным выделены все блоки с текстом, проблема, скорее всего, именно в нём.

И может быть, что тесты как то залипают. Страница product была переделана, но скришот приходит старый.

Мы при каждом запуске тестов берем последнюю версию мастера. Скорее всего, вы не сделали пуш в удаленный репозиторий перед запуском автотестов.

Возможно Вы мне скажите, что трешхолд 0.01 и мне исправить помарки.

Трешхолд как раз равен 0.01

bemdev commented 4 years ago

Мы при каждом запуске тестов берем последнюю версию мастера. Скорее всего, вы не сделали пуш в удаленный репозиторий перед запуском автотестов.

Странно. И лыжи не едут.

Создал новый репозиторий клонирую с гита запускаю, все хорошо. Картинки отключены, страница product отрисовывается правильно.

Запускаю Ваши тесты по новой ссылке, результат падает, что и картинки есть и страница product имеет другой вид вплоть до json дерева возможно.

product_desktop (1)

index_desktop_block_product

В папке build лежит два файла и папка для картинок В папке pages лежат json файлы как в примере stub

Остальное разложено иначе чем в классическом форке, но есть assets папка где лежит и json и скриншоты, которые считались эталонными для локальных тестов. это же какая разница вроде.

Что можно сделать?

Если проблема решиться до ответа, то и ишью закрою.