-
-
Notifications
You must be signed in to change notification settings - Fork 683
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
Add retry messages and fix flaky (fixes #413) #414
Conversation
baev
commented
Jun 15, 2017
•
edited
edited
.withTime(retried.getTime()) | ||
); | ||
latest.getStatusDetailsSafe().setFlaky(true); | ||
latest.getStatusDetailsSafe().setFlaky(!statuses.isEmpty()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently if a test fails 100% times, it will be displayed flaky.
It is also wrong, I think
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👎
BTW flacky -> flaky
@@ -3,7 +3,8 @@ | |||
{{#each retries}} | |||
<div class="retry-row" data-uid="{{uid}}"> | |||
<span class="label label_status_{{status}}">{{status}}</span> | |||
{{date time.start}} at {{time time.start}} | |||
<span>{{date time.start}} at {{time time.start}}</span> | |||
<div class="retry__message">{{statusDetails}}</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if you will use <p>
, then you can avoid extra styles, because <p>
provides margins by default