-
Notifications
You must be signed in to change notification settings - Fork 18
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
Падает тест junit #147
Comments
ИМХО это зарегистированный баг 1скрипта, когда в Линуксе и Винде формируются разные закрывающие теги :( на винде, в аппвейор тесты проходят на ура! Хотя бы в твоем свежем ПР #148 А у тебя на Линуксе падают? |
это не баг оскрипта, это баг тестов. XML в обоих случаях валидный и проверять его надо по XSD/чтением в объектной технике, а не сравнением строк. |
@artbear как раз на винде у меня и падают эти тесты. Имхо, Никита прав - и тот, и тот xml валидный и баг как раз в тесте. Будет время - переделаю, хочется полностью зелёную полосу ) |
Пришла идея - заменять лишний пробел
для приемочных тестов вполне себе рабочий вариант |
я называю это багом движка, потому что на разных платформах при одном исходном коде мы получаем разный текст файла :( тест переделать можно и нужно. |
Закрываю, исправлено и влито. |
В фиче core/junit-xml.feature падают все три сценария. Отладочный лог: https://gist.github.com/zeratulayuris/1ba579cfbf09e7d3c2cef5bbc59c93ac
Судя по всему, ошибка в пробелах перед закрывающим тег слешем - в фиче они есть, в отчете их нет.
osciprt 1.0.21.1, win10 pro.
The text was updated successfully, but these errors were encountered: