-
-
Notifications
You must be signed in to change notification settings - Fork 121
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(core):change flakiness overhead from ms to readable format #793
feat(core):change flakiness overhead from ms to readable format #793
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #793 +/- ##
=============================================
- Coverage 59.26% 59.25% -0.02%
- Complexity 791 793 +2
=============================================
Files 213 213
Lines 4380 4378 -2
Branches 689 689
=============================================
- Hits 2596 2594 -2
Misses 1462 1462
Partials 322 322
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
7bd9eca
to
f8394ed
Compare
…com:Vacxe/marathon into feature-core-update-flakiness-overhead-time
5c470e0
to
3fff539
Compare
@@ -72,7 +72,7 @@ data class ExecutionReport( | |||
val rawDuration = rawTests | |||
//Incomplete tests mess up the calculations of time since their end time is 0 and duration is, hence, years | |||
//We filter here for unavailable time just to be safe | |||
.filter { it.startTime != 0L && it.endTime != 0L } |
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.
Or will be better to filter endTime - startTime > 0
? WDYT @Malinskiy
3fff539
to
78e478b
Compare
No description provided.