Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
chestozo opened this issue Feb 13, 2017 · 5 comments
Open

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

chestozo opened this issue Feb 13, 2017 · 5 comments
Assignees

Comments

@chestozo
Copy link
Member

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

@vitkarpov
Copy link
Member

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

@vitkarpov
Copy link
Member

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

@chestozo
Copy link
Member Author

chestozo commented Feb 13, 2017

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

@vitkarpov
Copy link
Member

vitkarpov commented Feb 13, 2017

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

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

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

@vitkarpov
Copy link
Member

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

chestozo pushed a commit that referenced this issue Feb 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants