fix-180 #181
fix-180 #181
Conversation
show stacktrace beside the step
discard break words
allure-branding
Conflicts: allure-java/allure-java-aspects/src/test/java/ru/yandex/qatools/allure/aspects/AllureAspectUtilsTest.java
|
@lanwen ну, так и написанно - fix 180. В 180 все подробно расписано. Больше ничего содержательного нету |
|
ну просто чтобы было видно что внутри 180 не заходя в 180. А подробности уже да, в тикете пусть ищутся. |
| @@ -26,6 +27,19 @@ public void put(Step step) { | |||
| } | |||
|
|
|||
| public Step pollLast() { | |||
| return get().pollLast(); | |||
| Deque<Step> queue = get(); | |||
| Step last = queue.pollLast(); | |||
There was a problem hiding this comment.
а почему ты сначала поллишь, а потом добавляешь недостеп, в случае проблем?
There was a problem hiding this comment.
какая логика - у нас есть стек, когда степ начался, мы кладем его в стек, когда закончился - достаем последний из стека, и добавляем его в дети к предыдущему. Чтобы это работало надо поддерживать инвариант, что стек не пуст. Вначале он инициализируется с рутовым степом. Однако, если событий завершения степов произойдет больше, чем событий начала - то мы достанем со стека последний элемент, и инвариант нарушится. Поэтому я отслеживаю эту ситуацию и, если надо, добавляю еще один рут степ.
No description provided.