search

yandex-ui / noscript

Noscript: JavaScript MVC Framework for building SPA
http://yandex-ui.github.io/noscript/
MIT License
34 stars 22 forks source link

Сломались тесты где-то по дороге #642

Open chestozo opened 7 years ago

chestozo commented 7 years ago

В текущем мастере вижу такое: image

vitkarpov commented 7 years ago

Хм. А ты уверен, что это ошибка, а не тест на ошибку? Просто ns.exception гадит в консоль

vitkarpov commented 7 years ago

Возможно, нужно ns.exception в тестах переопределить, чтобы ничего не писал в стандартный вывод

chestozo commented 7 years ago

Не уверен, может это ожидаемое поведение. Но в любом случае, тесты не должны такое в консоль выводить. Хочется понять ошибка / не ошибка и соответственно зафиксить / не выводить в консоль )

vitkarpov commented 7 years ago

Хочется понять ошибка / не ошибка

Ты же сам недавно делал пул, в котором была зеленая галка — https://travis-ci.org/yandex-ui/noscript/jobs/200143750

Посмотри на вывод: там те же «ошибки».

vitkarpov commented 7 years ago

Т.е. там тесты, который проверяют, что метод кидает исключение. Это ок. Правда вот гадить в консоль не кайф, может стоит затереть ns.exception?