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

feat: Разделение ошибки и стектрейс #808

Merged

Conversation

KrapivinAndrey
Copy link
Contributor

реализация #603

сделанные изменения

  • Разделение краткой и подробной ошибки
  • Вывод подробной ошибки в трейс Allure
  • В отчет MXL подробная ошибка не выводится
  • В отчет JUnit выводится ошибка и подробное представление

@vanessa-opensource/Collaborators - просьба прокомментировать и проверить

@artbear artbear added this to the 6.7.0 milestone Nov 7, 2020
@artbear artbear merged commit bb5ddce into vanessa-opensource:develop Nov 7, 2020
@artbear
Copy link
Collaborator

artbear commented Nov 7, 2020

@KrapivinAndrey Спасибо!

@KrapivinAndrey KrapivinAndrey deleted the feature/Trace_allure branch November 7, 2020 20:21
@artbear
Copy link
Collaborator

artbear commented Mar 11, 2021

В отчет MXL подробная ошибка не выводится

вот это исправление все-таки неудобным оказалось (

запустил сегодня тесты интерактивно, тест упал,

выдалось краткое представление ошибки, абсолютно неинформативное, нет стека вызовов,

чей код виноват, непонятно (

оказалось, виноваты мои правки в плагин Данные, которые уже несколько недель как сделаны )
а тестировал я тесты в бизнес-системе и не сразу понял причину ошибки.

откатываю назад эту правку.

@KrapivinAndrey

@artbear
Copy link
Collaborator

artbear commented Mar 11, 2021

ну и в лог-файл и в консоль также желательно выводить стек ошибки, иначе непонятны причины.

разбор текущей проблемы и начался с того, что я увидел падение тестов с непонятным сообщением

image

artbear added a commit that referenced this pull request Mar 11, 2021
@artbear
Copy link
Collaborator

artbear commented Mar 11, 2021

После исправления вывод ошибки намного более понятен

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants