yndx-shri-reviewer / shri-2020-task-2

Задание 2. Напишите линтер — ШРИ, Москва, 2020
https://yandex.ru/promo/academy/shri
13 stars 41 forks source link

Валидация текстовых блоков #31

Closed victor-kugay closed 4 years ago

victor-kugay commented 4 years ago

@yndx-shri-reviewer, добрый день))

Я написал большое количество тестов для кейсов, связанных с валидацей текстовых блоков, валидировал json-ы из задания shri-1, однако, автотесты показывают 0 из 8 пройденных тестов.

Судя по всему мой код падает с некоторой ошибкой, т.к. после постановки try {} catch { return []; } как минимум 3 теста уже проходят.

Спасибо.

nat-k-dev commented 4 years ago

У меня тоже самое. Написано 26 тестов на заголовки, в том числе тесты из задания. И тоже по заголовкам 0 из 8, остальные проходят. При этом когда был отправлен алгоритм с ошибками, в т.ч. использование необъявленной переменной, все автотесты прошли успешно.

victor-kugay commented 4 years ago

@appalse Мой вариант падает с ошибкой, т.к. после try catch хотя бы три теста поднялись, а вот такой вариант, с referenceError - уже более интересно))

nat-k-dev commented 4 years ago

@ViktorKugay а вы запускали на двух json'ах из 1 задания?

victor-kugay commented 4 years ago

@appalse Да. Страницу product у меня валидируется без ошибок, index содержит некоторые ошибки, но не связанные с текстовыми блоками, т.к. там нет ни одного заголовка.

frontstall commented 4 years ago

@appalse Да. Страницу product у меня валидируется без ошибок, index содержит некоторые ошибки, но не связанные с текстовыми блоками, т.к. там нет ни одного заголовка.

Такая же история. 0 из 8 при локально зеленых тестах и 3 из 8 при отсутствии валидации вообще :)

nat-k-dev commented 4 years ago

Видимо, надо поискать, где может быть обращение в полям объекта в состоянии undefined

nat-k-dev commented 4 years ago

icons8-костыль-50 wanted

yndx-shri-reviewer commented 4 years ago

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