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

Падает тест junit #147

Closed
zeratulayuris opened this issue Mar 30, 2019 · 7 comments
Closed

Падает тест junit #147

zeratulayuris opened this issue Mar 30, 2019 · 7 comments
Labels
Milestone

Comments

@zeratulayuris
Copy link
Contributor

zeratulayuris commented Mar 30, 2019

В фиче core/junit-xml.feature падают все три сценария. Отладочный лог: https://gist.github.com/zeratulayuris/1ba579cfbf09e7d3c2cef5bbc59c93ac

Судя по всему, ошибка в пробелах перед закрывающим тег слешем - в фиче они есть, в отчете их нет.

osciprt 1.0.21.1, win10 pro.

@artbear
Copy link
Owner

artbear commented Apr 4, 2019

ИМХО это зарегистированный баг 1скрипта, когда в Линуксе и Винде формируются разные закрывающие теги :(

EvilBeaver/OneScript#817

на винде, в аппвейор тесты проходят на ура!

Хотя бы в твоем свежем ПР #148

А у тебя на Линуксе падают?

@nixel2007
Copy link
Contributor

это не баг оскрипта, это баг тестов. XML в обоих случаях валидный и проверять его надо по XSD/чтением в объектной технике, а не сравнением строк.

@zeratulayuris
Copy link
Contributor Author

@artbear как раз на винде у меня и падают эти тесты. Имхо, Никита прав - и тот, и тот xml валидный и баг как раз в тесте. Будет время - переделаю, хочется полностью зелёную полосу )

@zeratulayuris
Copy link
Contributor Author

И, кстати, про аппвейор в том самом моём ПР:
image

@artbear artbear added this to the next milestone Apr 5, 2019
@artbear
Copy link
Owner

artbear commented Apr 5, 2019

Пришла идея - заменять лишний пробел

СтрЗаменить(СтрокаХМЛ, " />" , "/>")

для приемочных тестов вполне себе рабочий вариант

@artbear
Copy link
Owner

artbear commented Apr 5, 2019

я называю это багом движка, потому что на разных платформах при одном исходном коде мы получаем разный текст файла :(

тест переделать можно и нужно.
просто такое поведение движка повышает сложность его использования :(

@zeratulayuris
Copy link
Contributor Author

Закрываю, исправлено и влито.

@artbear artbear modified the milestones: next, 1.10 Apr 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants